Wireshark-commits: [Wireshark-commits] rev 52491: / /trunk-1.8/epan/dissectors/: packet-openwire.c

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: eapache@xxxxxxxxxxxxx
Date: Thu, 10 Oct 2013 13:10:47 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=52491

User: eapache
Date: 2013/10/10 01:10 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.8/epan/dissectors/
  Changes    Path                 Action
  +3 -0      packet-openwire.c    Modified