Wireshark-commits: [Wireshark-commits] master-3.2 05939eb: rdm: Remove incorrect dissection of ack
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, 25 Jan 2020 13:47:19 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=05939ebb7c02ac29fd7de969633dcdd0eee4087e
Submitter: "Pascal Quantin <pascal@xxxxxxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

05939eb by Erwin Rol (erwin@xxxxxxxxxxxx):

    rdm: Remove incorrect dissection of ack overflow data
    
    Ack overflow data was incorrectly dissected causing checksum errors. For now
    just display raw data.
    
    Change-Id: Icdd858bdbeeb4dd40e48c45fc46e5e188d53be69
    Signed-off-by: Erwin Rol <erwin@xxxxxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/35915
    Reviewed-by: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Pascal Quantin <pascal@xxxxxxxxxxxxx>
    (cherry picked from commit 4f462bb6e3470ccb09b8da1efd4f133169aef03d)
    Reviewed-on: https://code.wireshark.org/review/35943
    Petri-Dish: Pascal Quantin <pascal@xxxxxxxxxxxxx>
    

Actions performed:

    from  aa3be87   Bugfix SMB IOCTL response with BUFFER_OVERFLOW
     add  05939eb   rdm: Remove incorrect dissection of ack overflow data


Summary of changes:
 epan/dissectors/packet-rdm.c | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)