diff options
| author | Michael Mann <mmann78@netscape.net> | 2016-10-10 23:34:41 -0400 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2016-10-11 06:54:05 +0000 |
| commit | 71232b69fda6372f1853cf7c7fadd989353adc69 (patch) | |
| tree | a6fbf6d436c3fa8e32df47dfa41be0ac510876d6 /epan/dissectors/packet-fix.c | |
| parent | 203f12dca0bde081abf45cdce108082abbfc95db (diff) | |
fix: break loop on failed string->number conversion
Bug: 12998
Change-Id: I3e9739dd9863e615c2ca0fc91f043c8562e4203e
Reviewed-on: https://code.wireshark.org/review/18142
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-fix.c')
| -rw-r--r-- | epan/dissectors/packet-fix.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-fix.c b/epan/dissectors/packet-fix.c index 6d8ebe99ce..111c7146e7 100644 --- a/epan/dissectors/packet-fix.c +++ b/epan/dissectors/packet-fix.c @@ -292,7 +292,7 @@ dissect_fix_packet(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* da if (!ws_strtou32(tvb_get_string_enc(wmem_packet_scope(), tvb, field_offset, tag->tag_len, ENC_ASCII), NULL, &tag_value)) { proto_tree_add_expert(fix_tree, pinfo, &ei_fix_tag_invalid, tvb, field_offset, tag->tag_len); - continue; + break; } if (tag->value_len < 1) { proto_tree *field_tree; |