ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 3454] Feature Request: Add protocol sub-field to the bootp

Date: Fri, 8 Jul 2011 05:16:21 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3454

Michael Mann <mmann78@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #6593|review_for_checkin?         |
               Flag|                            |
   Attachment #6593|0                           |1
        is obsolete|                            |
   Attachment #6635|                            |review_for_checkin?
               Flag|                            |

--- Comment #7 from Michael Mann <mmann78@xxxxxxxxxxxx> 2011-07-08 05:16:18 PDT ---
Created an attachment (id=6635)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=6635)
More BOOTP filters

Went even farther with replacing many proto_tree_add_text() calls with
proto_tree_add_item(), so the BOOTP dissector is considerably more filterable  
Its definitely "scope creep" relative to this bug, but I didn't think a
separate bug/enhancement entry was necessary.  I also wasn't sure I'd ever get
to expanding beyond just the "basic type" BOOTP options when I submitted the
original patch.

The remaining proto_tree_add_text() calls are mostly in "vendor specific"
functionality, as well as a few others that require a little more understanding
of the protocol itself.

Fuzztested with (to be) attached trace, but the trace was generated as a "hack"
to test my changes.

-- 
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.