From: Philippe Antoine <pantoine@oisf.net>
Date: Wed, 29 Oct 2025 21:29:11 +0000 (+0100)
Subject: util/var: add NULL check in VarNameStoreRegister
X-Git-Tag: suricata-8.0.2~17
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F14213%2Fhead;p=thirdparty%2Fsuricata.git

util/var: add NULL check in VarNameStoreRegister

And check return value in entropy keyword setup

(cherry picked from commit 854201703e51afd0945c5a9ec4d5b62303f9ca9d)
---

diff --git a/src/detect-entropy.c b/src/detect-entropy.c
index 4ef97b7e94..fee8307a0e 100644
--- a/src/detect-entropy.c
+++ b/src/detect-entropy.c
@@ -56,6 +56,9 @@ static int DetectEntropySetup(DetectEngineCtx *de_ctx, Signature *s, const char
     } else {
         ded->fv_idx = VarNameStoreRegister("content", VAR_TYPE_FLOW_FLOAT);
     }
+    if (ded->fv_idx == 0) {
+        goto error;
+    }
 
     if (SCSigMatchAppendSMToList(de_ctx, s, DETECT_ENTROPY, (SigMatchCtx *)ded, sm_list) != NULL) {
         SCReturnInt(0);
diff --git a/src/util-var-name.c b/src/util-var-name.c
index b5b518d6b0..a81920f351 100644
--- a/src/util-var-name.c
+++ b/src/util-var-name.c
@@ -154,6 +154,9 @@ void VarNameStoreDestroy(void)
  */
 uint32_t VarNameStoreRegister(const char *name, const enum VarTypes type)
 {
+    if (name == NULL) {
+        return 0;
+    }
     SCMutexLock(&base_lock);
     uint32_t id = 0;