Wireshark-bugs: [Wireshark-bugs] [Bug 7726] nfsv4.1 needs to parse SEQUENCE status flags

Date: Mon, 17 Sep 2012 20:42:59 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7726

Michael Mann <mmann78@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #9161|0                           |1
        is obsolete|                            |
   Attachment #9161|review_for_checkin?         |
              Flags|                            |
   Attachment #9162|0                           |1
        is obsolete|                            |
   Attachment #9162|review_for_checkin?         |
              Flags|                            |
   Attachment #9163|0                           |1
        is obsolete|                            |
   Attachment #9163|review_for_checkin?         |
              Flags|                            |
   Attachment #9164|0                           |1
        is obsolete|                            |
   Attachment #9164|review_for_checkin?         |
              Flags|                            |
   Attachment #9166|                            |review_for_checkin?
              Flags|                            |

--- Comment #12 from Michael Mann <mmann78@xxxxxxxxxxxx> 2012-09-17 20:42:58 PDT ---
Created attachment 9166
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=9166
All NFS patches in one

J. Bruce,

I combined all of your patches into one, cleaning up along the way.  Can you
verify this new patch is what you're expecting?

I also noticed a bunch of proto_tree_add_text() calls within the dissector that
could be changed to proto_tree_add_item calls so that the fields are
filterable.  Thanks for the current patches and feel free to continue to update
the NFS dissector in that regard much like you did here (but as a separate bug
report).

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.