diff options
| author | Jeff Morriss <jeff.morriss.ws@gmail.com> | 2013-08-08 17:53:11 +0000 |
|---|---|---|
| committer | Jeff Morriss <jeff.morriss.ws@gmail.com> | 2013-08-08 17:53:11 +0000 |
| commit | 45d2989e8b61c1c38d36a8a789e412bc97fdd020 (patch) | |
| tree | 170a4f63be5bf8914caf07afa1e69f5c39288c2a /epan/dissectors/packet-rsvp.c | |
| parent | 30e199e2a17bb376af82dceeab13fa98c287eed8 (diff) | |
From Francesco Fondelli via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9024 :
In dissect_rsvp_lsp_attributes() there is a check for invalid Attributes
TLV length that is no more correct in light of changes described in
Section 14 of RFC 5420. This patch fixes that.
svn path=/trunk/; revision=51224
Diffstat (limited to 'epan/dissectors/packet-rsvp.c')
| -rw-r--r-- | epan/dissectors/packet-rsvp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-rsvp.c b/epan/dissectors/packet-rsvp.c index a094ea9676..bb1f8f6617 100644 --- a/epan/dissectors/packet-rsvp.c +++ b/epan/dissectors/packet-rsvp.c @@ -5253,7 +5253,7 @@ dissect_rsvp_lsp_attributes(proto_tree *ti, proto_tree *rsvp_object_tree, tlv_type = tvb_get_ntohs(tvb, offset+tlv_off); tlv_len = tvb_get_ntohs(tvb, offset+tlv_off+2); - if ((tlv_len == 0) || (tlv_off+tlv_len > (obj_length-4))) { + if ((tlv_len == 0) || (tlv_off+tlv_len > obj_length)) { proto_tree_add_text(rsvp_object_tree, tvb, offset+tlv_off+2, 2, "Invalid length"); return; |