Wireshark-bugs: [Wireshark-bugs] [Bug 5185] Bootp option 125 sub 5

Date: Tue, 19 Jul 2011 18:26:29 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5185

Michael Mann <mmann78@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mmann78@xxxxxxxxxxxx

--- Comment #11 from Michael Mann <mmann78@xxxxxxxxxxxx> 2011-07-19 18:26:27 PDT ---
The parsing of the fields which the original patch(es) intended have already
been applied to the trunk (from elsewhere), but the display filter
functionality requested in comment #6 has not due to the "incompatibility" of
combining of options 125 and 60 (see dissect_docsis_cm_cap()).

Not sure if this warrants closing or ideas in keeping options 125 and 60
combined and still use proto_item_add_item().  It would be possible to do
proto_tree_add_uint_format() (and similar), which I think is better than the
current proto_item_append_text(), but it's not as clean as I would like. I'd
prefer to integrate into bootp_handle_basic_types() (my original intention to
"update" the current patch(es))

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