Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-bugs: [Wireshark-bugs] [Bug 6570] UCP dissector bug of operation 61

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.