Wireshark-commits: [Wireshark-commits] master c2b7a74: nfs: Fixing OFFLOAD_STATUS reply decoding
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: Wed, 21 Sep 2016 20:43:38 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c2b7a74c149241a5fa490d9201c04a987e3d3f63
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

c2b7a74 by Olga Kornievskaia (olga.kornievskaia@xxxxxxxxx):

    nfs: Fixing OFFLOAD_STATUS reply decoding
    
    commit b5aa0ff1a4e84c15f6c159e4900e84bd563570b4
    
        nfs: Add NFSv4.2 ops OFFLOAD_CANCEL and OFFLOAD_STATUS
    
    As coded, the reply of the OFFLOAD_STATUS decodes a stateid.
    However, in the spec, the reply is count and an array of
    statuses. I propose the following fix to match the spec
    for the OFFLOAD_STATUS.
    
    Change-Id: Ibaddba96446b8d9b520ca977f0b1ed66749d3388
    Reviewed-on: https://code.wireshark.org/review/17805
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  eafa0a0   dmp: Restore checksum incorrect info
    adds  c2b7a74   nfs: Fixing OFFLOAD_STATUS reply decoding


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