Wireshark-bugs: [Wireshark-bugs] [Bug 6570] UCP dissector bug of operation 61
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Mon, 14 Nov 2011 12:05:19 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6570

Chris Maynard <christopher.maynard@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #2 from Chris Maynard <christopher.maynard@xxxxxxxxx> 2011-11-14 12:05:19 PST ---
(In reply to comment #1)
> so shouldn't hf_ucp_parm_OPID be an FT_UINT16 instead of an FT_UINT8?

No, I guess it doesn't matter since it's 2 characters that comprise the OPID,
not 2 bytes, and the expected values are either 00 or 39, both of which fit in
an FT_UINT8.  Also, the field is added using proto_tree_add_uint() in
ucp_handle_int() with the number of bytes highlighted determined by the
difference in current offset to the offset of the next '/' separator.

Fixed in r39834 and scheduled for 1.6.4 and 1.4.11.

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