diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2015-08-13 22:32:43 +0200 |
---|---|---|
committer | Martin Kaiser <wireshark@kaiser.cx> | 2015-08-17 20:25:45 +0000 |
commit | c78d7c8aa84361bf13fa0816dfe16f9a46fccb3a (patch) | |
tree | eee9a188cb19d5a31fbafa1107b414824163924e /epan/dissectors/packet-h1.c | |
parent | ec1d25d884fbaed26f4ee18935567b044c6256a5 (diff) |
h1: don't THROW() an exception from a dissector
leaving the parsing loop should be enough in this case...
Change-Id: Ic250961aeb4d3cfcd74ee8caacb59657c32444de
Reviewed-on: https://code.wireshark.org/review/10078
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Diffstat (limited to 'epan/dissectors/packet-h1.c')
-rw-r--r-- | epan/dissectors/packet-h1.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-h1.c b/epan/dissectors/packet-h1.c index fdb9b05dd8..9e2e6f72ce 100644 --- a/epan/dissectors/packet-h1.c +++ b/epan/dissectors/packet-h1.c @@ -226,8 +226,8 @@ static gboolean dissect_h1(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, and try the next dissector */ return FALSE; } - if (tvb_get_guint8(tvb,offset + position + 1) < 1) - THROW(ReportedBoundsError); + if (tvb_get_guint8(tvb,offset + position + 1) == 0) + break; position += tvb_get_guint8(tvb,offset + position + 1); /* Goto next section */ } /* ..while */ next_tvb = tvb_new_subset_remaining(tvb, offset+tvb_get_guint8(tvb,offset+2)); |