Blame wireshark-nfsv4-opts.patch

Steve Dickson 01a52d
diff -up wireshark-1.0.2/epan/dissectors/packet-nfs.c.orig wireshark-1.0.2/epan/dissectors/packet-nfs.c
Steve Dickson 01a52d
--- wireshark-1.0.2/epan/dissectors/packet-nfs.c.orig	2008-07-10 13:39:23.000000000 -0400
Steve Dickson 01a52d
+++ wireshark-1.0.2/epan/dissectors/packet-nfs.c	2008-07-25 07:59:45.000000000 -0400
Steve Dickson 01a52d
@@ -7962,6 +7962,8 @@ dissect_nfs_argop4(tvbuff_t *tvb, int of
Steve Dickson 01a52d
 		ftree = proto_item_add_subtree(fitem, ett_nfs_argop4);
Steve Dickson 01a52d
 	}
Radek Vokál 0fc81e
 
Radek Vokál 0fc81e
+	proto_item_append_text(tree, ", Ops(%d):", ops);
Radek Vokál 0fc81e
+
Radek Vokál 0fc81e
 	for (ops_counter=0; ops_counter
Radek Vokál 0fc81e
 	{
Radek Vokál 0fc81e
 		opcode = tvb_get_ntohl(tvb, offset);
Steve Dickson 01a52d
@@ -7991,6 +7993,15 @@ dissect_nfs_argop4(tvbuff_t *tvb, int of
Steve Dickson 01a52d
 			break;
Steve Dickson 01a52d
 		}
Radek Vokál 0fc81e
 
Radek Vokál 0fc81e
+		if (opcode != NFS4_OP_PUTFH) {
Radek Vokál 0fc81e
+			if (check_col(pinfo->cinfo, COL_INFO)) {
Radek Vokál 0fc81e
+				col_append_fstr(pinfo->cinfo, COL_INFO, " %s", 
Radek Vokál 0fc81e
+					names_nfsv4_operation[opcode - 3].strptr);
Radek Vokál 0fc81e
+			}
Radek Vokál 0fc81e
+		}
Radek Vokál 0fc81e
+		proto_item_append_text(tree, " %s", 
Radek Vokál 0fc81e
+			names_nfsv4_operation[opcode - 3].strptr);
Radek Vokál 0fc81e
+
Radek Vokál 0fc81e
 		switch(opcode)
Radek Vokál 0fc81e
 		{
Radek Vokál 0fc81e
 		case NFS4_OP_ACCESS:
Steve Dickson 01a52d
@@ -8420,6 +8431,8 @@ dissect_nfs_resop4(tvbuff_t *tvb, int of
Steve Dickson 01a52d
 		ftree = proto_item_add_subtree(fitem, ett_nfs_resop4);
Steve Dickson 01a52d
 	}
Radek Vokál 0fc81e
 
Radek Vokál 0fc81e
+	proto_item_append_text(tree, ", Ops(%d):", ops);
Radek Vokál 0fc81e
+
Radek Vokál 0fc81e
 	for (ops_counter = 0; ops_counter < ops; ops_counter++)
Radek Vokál 0fc81e
 	{
Radek Vokál 0fc81e
 		opcode = tvb_get_ntohl(tvb, offset);
Steve Dickson 01a52d
@@ -8451,6 +8464,17 @@ dissect_nfs_resop4(tvbuff_t *tvb, int of
Radek Vokál 0fc81e
 
Radek Vokál 0fc81e
 		offset = dissect_nfs_nfsstat4(tvb, offset, newftree, &status);
Radek Vokál 0fc81e
 
Radek Vokál 0fc81e
+		if (check_col(pinfo->cinfo, COL_INFO)) {
Radek Vokál 0fc81e
+			if (status != NFS4_OK) {
Radek Vokál 0fc81e
+				col_append_fstr(pinfo->cinfo, COL_INFO, " %s(%d)", 
Radek Vokál 0fc81e
+					names_nfsv4_operation[opcode - 3].strptr, status);
Radek Vokál 0fc81e
+			} else if (opcode != NFS4_OP_PUTFH) {
Radek Vokál 0fc81e
+				col_append_fstr(pinfo->cinfo, COL_INFO, " %s", 
Radek Vokál 0fc81e
+					names_nfsv4_operation[opcode - 3].strptr);
Radek Vokál 0fc81e
+			}
Radek Vokál 0fc81e
+		}
Radek Vokál 0fc81e
+		proto_item_append_text(tree, " %s(%d)", 
Radek Vokál 0fc81e
+			names_nfsv4_operation[opcode - 3].strptr, status);
Radek Vokál 0fc81e
 		/*
Radek Vokál 0fc81e
 		 * With the exception of NFS4_OP_LOCK, NFS4_OP_LOCKT, and
Radek Vokál 0fc81e
 		 * NFS4_OP_SETATTR, all other ops do *not* return data with the
Steve Dickson 01a52d
@@ -8770,7 +8794,7 @@ static const value_string nfsv3_proc_val
Radek Vokál 0fc81e
 static const vsff nfs4_proc[] = {
Radek Vokál 0fc81e
 	{ 0, "NULL",
Radek Vokál 0fc81e
 	dissect_nfs3_null_call,		dissect_nfs3_null_reply },
Radek Vokál 0fc81e
-	{ 1, "COMPOUND",
Radek Vokál 0fc81e
+	{ 1, "COMP",
Radek Vokál 0fc81e
 	dissect_nfs4_compound_call, dissect_nfs4_compound_reply },
Radek Vokál 0fc81e
 	{ 0, NULL, NULL, NULL }
Radek Vokál 0fc81e
 };