Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 6768] Buildbot crash output: fuzz-2012-01-25-22599.pcap

[bugzilla-daemon@xxxxxxxxxxxxx]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6768

Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #7837|                            |review_for_checkin?
              Flags|                            |

--- Comment #1 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> 2012-02-13 19:08:05 PST ---
Created attachment 7837
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=7837
Patch that fixes the core

The attached patch fixes the problem for me but I'm not sure it's correct. 
Should there always be pairs of mux elements (so would a dissector assertion
that current_me be valid be the right thing to do)?  Or should the first
(handed to us) one be counted (it's not currently)?

The code has been like this since this dissector was first submitted 6+ years
ago, so I'm leaving this for someone familiar with the protocol.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.