aboutsummaryrefslogtreecommitdiffstats
path: root/epan
diff options
context:
space:
mode:
authorBill Meier <wmeier@newsguy.com>2012-02-20 20:23:56 +0000
committerBill Meier <wmeier@newsguy.com>2012-02-20 20:23:56 +0000
commit8fce627622f7f0bb2c0207ab9886d595e5658c62 (patch)
treee8d18fd2b55a35badcfea5ee9aebf9d30c2bc048 /epan
parentc7f1a431d23e17a15777652b1252e139f182b0e6 (diff)
Thou shalt not stop dissecting after doing processing for COL_INFO ....
svn path=/trunk/; revision=41083
Diffstat (limited to 'epan')
-rw-r--r--epan/dissectors/packet-elcom.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/epan/dissectors/packet-elcom.c b/epan/dissectors/packet-elcom.c
index 6bd12feae0..04eccd952d 100644
--- a/epan/dissectors/packet-elcom.c
+++ b/epan/dissectors/packet-elcom.c
@@ -407,7 +407,7 @@ dissect_datarequest(proto_item *ti_arg, gint ett_arg, tvbuff_t *tvb, gint arg_of
}
/* XXX: Are all the tests against tvb_length() really the right way to handle invalid fields ?
- * It seems to me that invalid fields should should just add an expert item
+ * It seems to me that invalid fields should just add an expert item
* or cause a "Malformed" exception.
*/
static void
@@ -478,8 +478,6 @@ dissect_elcom(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
col_append_str(pinfo->cinfo, COL_INFO, " Response");
break;
}
-
- return;
}
if (!tree)