Wireshark · Wireshark-commits: [Wireshark-commits] master-2.2 6721868: BOOTP: Fix V-I Vendor-Specific Information Option

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=6721868766fd6f0df0859c3cf36777da69a2b3ef
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

6721868 by Jaap Keuter (jaap.keuter@xxxxxxxxx):

    BOOTP: Fix V-I Vendor-Specific Information Option
    
    Apply the same value checks to the vendor generic suboption dissection
    as is done for the Cable lab and ADSL forum ones.
    See https://ask.wireshark.org/questions/57695 for an example issue.
    
    Change-Id: I4fe07d07cf0a93f4693e5ff54dd70c008701cf41
    Reviewed-on: https://code.wireshark.org/review/18999
    Reviewed-by: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    (cherry picked from commit 9ca313cfbe4993a0a36520d216a3e4282b0b7b99)
    Reviewed-on: https://code.wireshark.org/review/19020
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  a512ff6   Make sure config.h is included before inet_addr.h.
    adds  6721868   BOOTP: Fix V-I Vendor-Specific Information Option


Summary of changes:
 epan/dissectors/packet-bootp.c |   50 +++++++++++++++++++++++++++++-----------
 1 file changed, 37 insertions(+), 13 deletions(-)