Wireshark-commits: [Wireshark-commits] master-2.4 2cd4058: 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:56:15 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=2cd40589d886fc356a1cd5b8f66ccf8b943079f5
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master-2.4
Repository: wireshark

Commits:

2cd4058 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 338ce1b67217e5e5f9ee81540748e34766fd85c2)
    Reviewed-on: https://code.wireshark.org/review/33496
    

Actions performed:

    from  4fe658b   Fix the dissection of Find First2 Query EA Info information.
     add  2cd4058   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(+)