Wireshark · Wireshark-commits: [Wireshark-commits] rev 52490: / /trunk-1.10/epan/dissectors/: packet-openwire.c

[eapache@xxxxxxxxxxxxx]

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=52490

User: eapache
Date: 2013/10/10 01:08 PM

Log:
 Copy over from trunk to pacify the fuzzbot and so Michael's patch backports
 cleanly
 
   ------------------------------------------------------------------------
   r52458 | eapache | 2013-10-08 19:16:53 -0400 (Tue, 08 Oct 2013) | 10 lines
   Changed paths:
      M /trunk/epan/dissectors/packet-openwire.c
 
   Hacky fix for https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9248
   Just break out of the loop if offset doesn't go up.
 
   There's almost certainly a better fix - the dissector is weird, and I'm not sure
   if all the _length_remaining() checks are important or legacy, and what affect
   they have on this issue.
 
   At the very least this will pacify the fuzzbots until somebody has time to
   figure it out properly.
 
   ------------------------------------------------------------------------

Directory: /trunk-1.10/epan/dissectors/
  Changes    Path                 Action
  +3 -0      packet-openwire.c    Modified