Wireshark-commits: [Wireshark-commits] master 05c47d7: [NFS] adding stateid hash based on stateid.o

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: Fri, 18 Dec 2015 13:23:07 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=05c47d755a82891fd73b2dfad7e128cd3c966ded
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

05c47d7 by Olga Kornievskaia (kolga@xxxxxxxxxx):

    [NFS] adding stateid hash based on stateid.other
    
    In NFS protocol, stateid's seq# changes within the stateid and therefore
    it changes the calculated hash of the stateid displayed by the wireshark.
    It makes it inconvenient to filter packets based on such value.
    
    This patches adds additional field (instead of replacing old) to display
    the CRC32 hash of the stateid.other field.
    
    Bug:11895
    Change-Id: I70c6d2b88822b6f735e8bc506a1bfcb421f6ddb9
    Signed-off-by: Olga Kornievskaia <kolga@xxxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/12536
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  df8dd74   Removed local definition of offsetof().
    adds  05c47d7   [NFS] adding stateid hash based on stateid.other


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