Switched to bitwise & instead of logical && in bit operations.
authorstig <stig@f5534014-38df-0310-8fa8-9805f1628bb7>
Fri, 17 Dec 2010 13:01:52 +0000 (13:01 +0000)
committerstig <stig@f5534014-38df-0310-8fa8-9805f1628bb7>
Fri, 17 Dec 2010 13:01:52 +0000 (13:01 +0000)
This bug was found by clang.

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

epan/dissectors/packet-ncp2222.inc

index e575b9ed775b203edcebd32941f750bb2e523836..da9c36f0687afdf0fc0e33ea32e632cf0a3f6092 100644 (file)
@@ -9590,7 +9590,7 @@ dissect_nds_request(tvbuff_t *tvb, packet_info *pinfo,
                 pvalues[1].bit15hfname = hf_bit15rflags;
                 pvalues[1].bit16 = "Not Defined";
                 pvalues[1].bit16hfname = hf_bit16rflags;
-                if((pvalues[1].vvalue&&0xf000) == 0xc000)
+                if((pvalues[1].vvalue & 0xf000) == 0xc000)
                 {
                     pvalues[2].vtype = VTYPE_STRING;
                     pvalues[2].vdesc = "Name Type: %s";
@@ -9662,7 +9662,7 @@ dissect_nds_request(tvbuff_t *tvb, packet_info *pinfo,
                 pvalues[1].bit15hfname = hf_bit15rflags;
                 pvalues[1].bit16 = "Not Defined";
                 pvalues[1].bit16hfname = hf_bit16rflags;
-                if((pvalues[1].vvalue&&0xf000) == 0xc000)
+                if((pvalues[1].vvalue & 0xf000) == 0xc000)
                 {
                     pvalues[2].vtype = VTYPE_STRING;
                     pvalues[2].vdesc = "Name Type: %s";