Wireshark-commits: [Wireshark-commits] master 1fc558b: NFS: Fixing again stateid display of other_h
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: Fri, 25 Oct 2019 03:24:52 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1fc558b82bdf4bc707e9e42d1232c327052aecb3
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

1fc558b by Jorge Mora (jmora1300@xxxxxxxxx):

    NFS: Fixing again stateid display of other_hash
    
    Initially added feature to match linux kernel hash value of
    stateid to the wireshark in
      commit 05c47d755a82891fd73b2dfad7e128cd3c966ded
      change id I70c6d2b88822b6f735e8bc506a1bfcb421f6ddb9
      [NFS] adding stateid hash based on stateid.other
    
    This has been broken by
      commit b913bcc13a9a6213902691a8d73bba1427ab407b
      change id I3bf7caefc3341887a4c9137500dfeac0115af8cf
      NFSv4: Fix for duplicate StateId hashes
    
    This patch fixes it to again calculate a crc32 hash to
    make it to the linux kernel output.
    
    Change-Id: Ibb29516de61bd9bdf2dd1164da4b14412aa7a71c
    Reviewed-on: https://code.wireshark.org/review/34842
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  dda619f   nas-5gs: NAS-5GS 5GS network feature support incorrectly dissected
     add  1fc558b   NFS: Fixing again stateid display of other_hash


Summary of changes:
 epan/dissectors/packet-nfs.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)