diff options
author | etxrab <etxrab@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-07-06 04:13:54 +0000 |
---|---|---|
committer | etxrab <etxrab@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-07-06 04:13:54 +0000 |
commit | 1c7b739d7dc60179bd2e3be6efea6c3a2c7219cb (patch) | |
tree | d54968c08c6b9498169d365b9b967efe80967f86 /epan/dissectors/packet-rpc.c | |
parent | 0bb4c56fec008850a22268719789f709eec7e47c (diff) |
From Ian Schorr:
NFSv4 usability enhancements (Info column summaries, filterable fields).
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4975
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@33442 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-rpc.c')
-rw-r--r-- | epan/dissectors/packet-rpc.c | 30 |
1 files changed, 22 insertions, 8 deletions
diff --git a/epan/dissectors/packet-rpc.c b/epan/dissectors/packet-rpc.c index 51300e0b60..9da6edb4f6 100644 --- a/epan/dissectors/packet-rpc.c +++ b/epan/dissectors/packet-rpc.c @@ -2062,15 +2062,22 @@ dissect_rpc_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, "Procedure: %s (%u)", procname, proc); } + /* Print the program version, procedure name, and message type (call or reply). */ if (check_col(pinfo->cinfo, COL_INFO)) { if (first_pdu) col_clear(pinfo->cinfo, COL_INFO); else col_append_str(pinfo->cinfo, COL_INFO, " ; "); - col_append_fstr(pinfo->cinfo, COL_INFO,"V%u %s %s", - vers, - procname, - msg_type_name); + /* Special case for NFSv4 - if the type is COMPOUND, do not print the procedure name */ + if (vers==4 && prog==NFS_PROGRAM && !strcmp(procname, "COMPOUND")) + col_append_fstr(pinfo->cinfo, COL_INFO,"V%u %s", + vers, + msg_type_name); + else + col_append_fstr(pinfo->cinfo, COL_INFO,"V%u %s %s", + vers, + procname, + msg_type_name); } /* Keep track of the address whence the call came, and the @@ -2268,15 +2275,22 @@ dissect_rpc_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, } } + /* Print the program version, procedure name, and message type (call or reply). */ if (check_col(pinfo->cinfo, COL_INFO)) { if (first_pdu) col_clear(pinfo->cinfo, COL_INFO); else col_append_str(pinfo->cinfo, COL_INFO, " ; "); - col_append_fstr(pinfo->cinfo, COL_INFO,"V%u %s %s", - vers, - procname, - msg_type_name); + /* Special case for NFSv4 - if the type is COMPOUND, do not print the procedure name */ + if (vers==4 && prog==NFS_PROGRAM && !strcmp(procname, "COMPOUND")) + col_append_fstr(pinfo->cinfo, COL_INFO,"V%u %s", + vers, + msg_type_name); + else + col_append_fstr(pinfo->cinfo, COL_INFO,"V%u %s %s", + vers, + procname, + msg_type_name); } if (rpc_tree) { |