diff options
| author | Michael Mann <mmann78@netscape.net> | 2016-11-11 21:58:45 -0500 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2016-11-13 23:30:52 +0000 |
| commit | 63776db384f75b2dc793cab46cf27250d8913711 (patch) | |
| tree | 5ab3d7781329c14d002c3bdf3076367be9ff405a | |
| parent | 4fd9872d11d3d575d8f261097c51a7d30b36ee40 (diff) | |
| download | wireshark-63776db384f75b2dc793cab46cf27250d8913711.tar.gz | |
packet-dtn.c break loop if evaluate_sdnv doesn't succeed.
Bug: 13097
Change-Id: If6128f5a3c940b4713c396e1a4a8f002a6bd03b8
Reviewed-on: https://code.wireshark.org/review/18764
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
| -rw-r--r-- | epan/dissectors/packet-dtn.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/epan/dissectors/packet-dtn.c b/epan/dissectors/packet-dtn.c index 881f5fd4c6..8a12f9b0f5 100644 --- a/epan/dissectors/packet-dtn.c +++ b/epan/dissectors/packet-dtn.c @@ -1558,11 +1558,13 @@ display_extension_block(proto_tree *tree, tvbuff_t *tvb, packet_info *pinfo, int for (i = 0; i < num_eid_ref; i++) { - evaluate_sdnv(tvb, offset, &sdnv_length); + if (evaluate_sdnv(tvb, offset, &sdnv_length) < 0) + break; offset += sdnv_length; block_overhead += sdnv_length; - evaluate_sdnv(tvb, offset, &sdnv_length); + if (evaluate_sdnv(tvb, offset, &sdnv_length) < 0) + break; offset += sdnv_length; block_overhead += sdnv_length; } |