Wireshark-commits: [Wireshark-commits] master-3.0 847d015: Don't assume padding is present at the e

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 04 Jun 2019 22:55:36 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=847d015826c33466784fdabcc2d8dd4b0409a818
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

847d015 by Guy Harris (guy@xxxxxxxxxxxx):

    Don't assume padding is present at the end of UNIX Info2.
    
    Check whether the byte count includes the padding before skipping it; it
    may not be present (at least not if this is at the end of the byte
    parameters).
    
    Change-Id: I4385a4713cb6813a6e8519005288d6ef5a28f028
    Reviewed-on: https://code.wireshark.org/review/33493
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit cf95ff8c29fb4d6576fd4f089389ff802ad542f8)
    Reviewed-on: https://code.wireshark.org/review/33494
    

Actions performed:

    from  79e5de6   Fix the dissection of Find First2 Query EA Info information.
     add  847d015   Don't assume padding is present at the end of UNIX Info2.


Summary of changes:
 epan/dissectors/packet-smb.c | 1 +
 1 file changed, 1 insertion(+)