Wireshark-commits: [Wireshark-commits] master 9ca313c: BOOTP: Fix V-I Vendor-Specific Information O
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 2 Dec 2016 14:48:01 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9ca313cfbe4993a0a36520d216a3e4282b0b7b99
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

9ca313c 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>
    

Actions performed:

    from  3b5d5ad   autotools: fix typo.
    adds  9ca313c   BOOTP: Fix V-I Vendor-Specific Information Option


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