Wireshark-commits: [Wireshark-commits] master be113e8: DHCP/BOOTP: Option 125 Suboption: (1) option
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: Sat, 17 Jan 2015 22:26:13 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=be113e8806796faf048d6c95c1141525ab5da29e
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

be113e8 by Alexis La Goutte (alexis.lagoutte@xxxxxxxxx):

    DHCP/BOOTP: Option 125 Suboption: (1) option-len always expects 1 but specification allows for more
    
    We believe this is a bug in Wireshark with Option 125 Suboption: (1)  option-len where Wireshark always expects 1 but specification is more flexible.
    Re: http://www.cablelabs.com/wp-content/uploads/specdocs/CL-SP-CANN-DHCP-Reg-I10-1308081.pdf page 15 part 4.2.2 'The DHCPv4 Option Request Option'
    
    Issue reported by Steve Krebs
    
    Bug:10784
    Change-Id: I8eced7a70877d13d925b81c0f47885609baa7633
    Reviewed-on: https://code.wireshark.org/review/6510
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  506409d   X509AF: dissect Certificate Serial Number as INT64 instead of INT32
    adds  be113e8   DHCP/BOOTP: Option 125 Suboption: (1) option-len always expects 1 but specification allows for more


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