diff options
author | etxrab <etxrab@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-10-28 06:06:20 +0000 |
---|---|---|
committer | etxrab <etxrab@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-10-28 06:06:20 +0000 |
commit | a57476728df00ddcfea9dfbdca53896d982653ad (patch) | |
tree | adf5fee04f964ec48c4d901a6d3546091501aed4 /epan/dissectors/packet-rtps2.c | |
parent | 34c9bd4b847e051a0b99c6d34f1c5fbcaf969fde (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
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@39646 f5534014-38df-0310-8fa8-9805f1628bb7
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; |