Wireshark-commits: [Wireshark-commits] rev 35722: /trunk/epan/dissectors/ /trunk/epan/dissectors/:

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: wmeier@xxxxxxxxxxxxx
Date: Mon, 31 Jan 2011 15:27:29 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=35722

User: wmeier
Date: 2011/01/31 07:27 AM

Log:
 From Roberto Morro: Fix for Bug #5628
 
 This patch (against revision 35716) resolves the following bug:
 [introduced in patch attached to Bug #5518]
 "Dissector bug, protocol RSVP, in packet 78:
 More than 1000000 items in the tree -- possible infinite loop"
 
 Actually the bug is resolved by the "return" statement that was erased by
 mistake. The patch fixes another issue: the code didn't take care of the fact
 that TLVs must be padded to a length multiple of 4.
 
 See:
 https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5518
 https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5628
 
 From me:
  Change two bytestring_to_str(tvb_get_ptr(...),...) to tvb_bytes_to_str(tvb,...)

Directory: /trunk/epan/dissectors/
  Changes    Path             Action
  +11 -4     packet-rsvp.c    Modified