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.
