Wireshark-bugs: [Wireshark-bugs] [Bug 3884] Assertion caused by fuzz test file
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Mon, 17 Aug 2009 13:42:27 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3884


Mike Morrin <wireshark@xxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |wireshark@xxxxxxxxxxxxxxx




--- Comment #4 from Mike Morrin <wireshark@xxxxxxxxxxxxxxx>  2009-08-17 13:42:26 PDT ---
Sorry about the flurry of confusion...  I managed to add the patch and comments
to the wrong bug.

The patch for this bug just throws a ReportedBoundError when the length of the
field does not match the length of the expected integer type.  I know it is not
really the intended use of this excpetion, but it gives sensible behaviour in
practice.

I would have rather called expert_add_info_format() with a new warning type
(without throwing an exception) to allow PDU decoding to continue, but that
scheme foundered because proto_tree_new_item() has no access to packet_info,
and I couldn't see a reasonable way around that problem.


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