Wireshark-commits: [Wireshark-commits] master-3.0 c5c740b: 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 14:47:56 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c5c740bce9551c0b1d3165c6fb49a21039efffda
Submitter: "Pascal Quantin <pascal@xxxxxxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

c5c740b 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/35944
    Petri-Dish: Pascal Quantin <pascal@xxxxxxxxxxxxx>
    

Actions performed:

    from  f4fa13f   artnet: Fix ArtPollReply universe calculation
     add  c5c740b   rdm: Remove incorrect dissection of ack overflow data


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