From Martin Lutz: Patch tvb_memeql return check;
authorwmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7>
Thu, 17 Dec 2009 16:22:50 +0000 (16:22 +0000)
committerwmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7>
Thu, 17 Dec 2009 16:22:50 +0000 (16:22 +0000)
  ... tvb_memeql returns 0 for truth.
See: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3986#c7

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

plugins/opcua/opcua.c

index 0b1a89f421ae182e83bc465bd490d8957ad1a21f..a7feaebfb67373250bed5d79978aa53bde490bcf 100644 (file)
@@ -167,32 +167,32 @@ static void dissect_opcua_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree
     col_set_str(pinfo->cinfo, COL_PROTOCOL, "OpcUa");
 
     /* parse message type */
-    if (tvb_memeql(tvb, 0, "HEL", 3))
+    if (tvb_memeql(tvb, 0, "HEL", 3) == 0)
     {
         msgtype = MSG_HELLO;
         pfctParse = parseHello;
     }
-    else if (tvb_memeql(tvb, 0, "ACK", 3))
+    else if (tvb_memeql(tvb, 0, "ACK", 3) == 0)
     {
         msgtype = MSG_ACKNOWLEDGE;
         pfctParse = parseAcknowledge;
     }
-    else if (tvb_memeql(tvb, 0, "ERR", 3))
+    else if (tvb_memeql(tvb, 0, "ERR", 3) == 0)
     {
         msgtype = MSG_ERROR;
         pfctParse = parseError;
     }
-    else if (tvb_memeql(tvb, 0, "MSG", 3))
+    else if (tvb_memeql(tvb, 0, "MSG", 3) == 0)
     {
         msgtype = MSG_MESSAGE;
         pfctParse = parseMessage;
     }
-    else if (tvb_memeql(tvb, 0, "OPN", 3))
+    else if (tvb_memeql(tvb, 0, "OPN", 3) == 0)
     {
         msgtype = MSG_OPENSECURECHANNEL;
         pfctParse = parseOpenSecureChannel;
     }
-    else if (tvb_memeql(tvb, 0, "CLO", 3))
+    else if (tvb_memeql(tvb, 0, "CLO", 3) == 0)
     {
         msgtype = MSG_CLOSESECURECHANNEL;
         pfctParse = parseCloseSecureChannel;