From: Victor Julien <vjulien@oisf.net>
Date: Tue, 26 Apr 2022 19:33:52 +0000 (+0200)
Subject: detect: fix bad BUG_ON pattern
X-Git-Tag: suricata-6.0.6~87
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a6335a76b487070fd63fadd45a23fed35d70f0c7;p=thirdparty%2Fsuricata.git

detect: fix bad BUG_ON pattern

cppcheck:

src/detect-engine-uint.c:73:13: warning: Conversion of string literal "unknown mode" to bool always evaluates to true. [incorrectStringBooleanError]
            BUG_ON("unknown mode");
            ^
src/detect-engine-uint.c:328:13: warning: Conversion of string literal "unknown mode" to bool always evaluates to true. [incorrectStringBooleanError]
            BUG_ON("unknown mode");
            ^
src/detect-pcre.c:291:25: warning: Conversion of string literal "Impossible captype" to bool always evaluates to true. [incorrectStringBooleanError]
                        BUG_ON("Impossible captype");
                        ^

Bug: #5291.
(cherry picked from commit 7e2ed11a11cae89bc07e9a25425192581687993d)
---

diff --git a/src/detect-engine-uint.c b/src/detect-engine-uint.c
index 51513d877d..2979cf29eb 100644
--- a/src/detect-engine-uint.c
+++ b/src/detect-engine-uint.c
@@ -60,7 +60,7 @@ int DetectU32Match(const uint32_t parg, const DetectU32Data *du32)
             }
             return 0;
         default:
-            BUG_ON("unknown mode");
+            BUG_ON(1); // unknown mode
     }
     return 0;
 }
@@ -255,7 +255,7 @@ int DetectU8Match(const uint8_t parg, const DetectU8Data *du8)
             }
             return 0;
         default:
-            BUG_ON("unknown mode");
+            BUG_ON(1); // unknown mode
     }
     return 0;
 }