diff options
author | Anders Broman <anders.broman@ericsson.com> | 2011-10-28 06:06:20 +0000 |
---|---|---|
committer | Anders Broman <anders.broman@ericsson.com> | 2011-10-28 06:06:20 +0000 |
commit | fb1736120faf6d12fa02be25ca4df0ecb5bc1d8d (patch) | |
tree | adf5fee04f964ec48c4d901a6d3546091501aed4 /epan/dissectors/packet-rtps2.c | |
parent | 83aed00facb852b27419fde04afb4c59ac30d01f (diff) |
From Adam Mitz:
The RTPS2 dissector doesn't handle octetsToNextHeader properly in INFO_RELY submessages.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6506
svn path=/trunk/; revision=39646
Diffstat (limited to 'epan/dissectors/packet-rtps2.c')
-rw-r--r-- | epan/dissectors/packet-rtps2.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/epan/dissectors/packet-rtps2.c b/epan/dissectors/packet-rtps2.c index 7b1ba27ebb..ce600290c5 100644 --- a/epan/dissectors/packet-rtps2.c +++ b/epan/dissectors/packet-rtps2.c @@ -7408,18 +7408,18 @@ static void dissect_INFO_REPLY(tvbuff_t *tvb, int min_len; rtps_util_decode_flags(tree, tvb, offset + 1, flags, INFO_REPLY_FLAGS); - min_len = 8; - if ((flags & FLAG_INFO_REPLY_M) != 0) min_len += 8; + min_len = 4; + if ((flags & FLAG_INFO_REPLY_M) != 0) min_len += 4; - if (octets_to_next_header != min_len) { + if (octets_to_next_header < min_len) { proto_tree_add_uint_format(tree, hf_rtps_sm_octets_to_next_header, tvb, offset+2, 2, octets_to_next_header, - "octetsToNextHeader: %u (Error: should be == %u)", + "octetsToNextHeader: %u (Error: should be >= %u)", octets_to_next_header, min_len); return; |