Removed unused parameter to parseServiceNodeId.
authorstig <stig@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 23 Mar 2011 08:03:02 +0000 (08:03 +0000)
committerstig <stig@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 23 Mar 2011 08:03:02 +0000 (08:03 +0000)
Coverity 596.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@36283 f5534014-38df-0310-8fa8-9805f1628bb7

plugins/opcua/opcua_application_layer.c
plugins/opcua/opcua_application_layer.h
plugins/opcua/opcua_transport_layer.c

index 1d701274bb5b6f1c23fad82c99865c9c5893397e..5d08b633d073ce00119c8423e1ef84ddc4f3f3e8 100644 (file)
@@ -72,14 +72,12 @@ void registerApplicationLayerTypes(int proto)
 /** Parses an OpcUa Service NodeId and returns the service type.
  * In this cases the NodeId is always from type numeric and NSId = 0.
  */
-int parseServiceNodeId(proto_tree *tree, tvbuff_t *tvb, gint *pOffset, char *szFieldName)
+int parseServiceNodeId(proto_tree *tree, tvbuff_t *tvb, gint *pOffset)
 {
     gint    iOffset = *pOffset;
     guint8  EncodingMask;
     guint32 Numeric = 0;
 
-       szFieldName = 0; /* avoid warning */
-
     EncodingMask = tvb_get_guint8(tvb, iOffset);
     proto_tree_add_item(tree, hf_opcua_nodeid_encodingmask, tvb, iOffset, 1, TRUE);
     iOffset++;
index 13a52e07b118176fbf63ee1a441040f98aa597a9..066a353f975b65a26924f8e3c7c93a328a6ba18c 100644 (file)
@@ -24,4 +24,4 @@
 void registerApplicationLayerTypes(int proto);
 
 /* Ua type parsers */
-int parseServiceNodeId(proto_tree *tree, tvbuff_t *tvb, gint *pOffset, char *szFieldName);
+int parseServiceNodeId(proto_tree *tree, tvbuff_t *tvb, gint *pOffset);
index 426f6835fcc1a975265d762e947c254ebad18c0a..2340db6e287921295f8c8d441cc9ebb9d20aeed2 100644 (file)
@@ -188,7 +188,7 @@ int parseMessage(proto_tree *tree, tvbuff_t *tvb, gint *pOffset)
     /* add nodeid subtree */
     ti = proto_tree_add_text(encobj_tree, tvb, 0, -1, "TypeId : ExpandedNodeId");
     nodeid_tree = proto_item_add_subtree(ti, ett_opcua_nodeid);
-    ServiceId = parseServiceNodeId(nodeid_tree, tvb, pOffset, "NodeId");
+    ServiceId = parseServiceNodeId(nodeid_tree, tvb, pOffset);
 
     dispatchService(encobj_tree, tvb, pOffset, ServiceId);
     return ServiceId;
@@ -218,7 +218,7 @@ int parseOpenSecureChannel(proto_tree *tree, tvbuff_t *tvb, gint *pOffset)
     /* add nodeid subtree */
     ti = proto_tree_add_text(encobj_tree, tvb, 0, -1, "TypeId : ExpandedNodeId");
     nodeid_tree = proto_item_add_subtree(ti, ett_opcua_nodeid);
-    ServiceId = parseServiceNodeId(nodeid_tree, tvb, pOffset, "NodeId");
+    ServiceId = parseServiceNodeId(nodeid_tree, tvb, pOffset);
 
     dispatchService(encobj_tree, tvb, pOffset, ServiceId);
     return -1;