diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2017-06-17 09:03:05 +0200 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-06-17 13:17:28 +0000 |
commit | 6462560b30635b79abc7d248dbf53179f31d84a7 (patch) | |
tree | 9ea17da332ffd5c70c59c59e72f8dd412e339edf /epan/dissectors/packet-smpp.c | |
parent | b2fa26ff7a270dc6a0f70c06cdc3e186f0c9d29f (diff) |
smpp: don't THROW() an exception from a dissector
If the pdu length is too short, we can simply stop dissection and return
the number of bytes we processed.
Change-Id: I11581daa3fdb80b3d5a07754039ec1b640945b2e
Reviewed-on: https://code.wireshark.org/review/22187
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Petri-Dish: Martin Kaiser <wireshark@kaiser.cx>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-smpp.c')
-rw-r--r-- | epan/dissectors/packet-smpp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-smpp.c b/epan/dissectors/packet-smpp.c index 48b1ba8517..a717406499 100644 --- a/epan/dissectors/packet-smpp.c +++ b/epan/dissectors/packet-smpp.c @@ -2385,7 +2385,7 @@ dissect_smpp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data) tvbuff_t *pdu_tvb; if (pdu_len < 1) - THROW(ReportedBoundsError); + return offset; if (pdu_real_len <= 0) return offset; |