]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
fuzz: limit memcap for datasets when fuzzing
authorPhilippe Antoine <contact@catenacyber.fr>
Tue, 3 Nov 2020 10:21:06 +0000 (11:21 +0100)
committerVictor Julien <victor@inliniac.net>
Tue, 1 Dec 2020 04:33:07 +0000 (05:33 +0100)
src/util-thash.c

index 2753dae17d8901f82923ed3fc63a0eb0b9f4528d..a90558066b1ea4d311b2eb8e8712559dad1be49c 100644 (file)
@@ -308,16 +308,16 @@ THashTableContext *THashInit(const char *cnf_prefix, size_t data_size,
     ctx->config.hash_size = hashsize > 0 ? hashsize : THASH_DEFAULT_HASHSIZE;
     /* Reset memcap in case of loading from file to the highest possible value
      unless defined by the rule keyword */
+#ifdef FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION
+    // limit memcap size to default when fuzzing
+    ctx->config.memcap = THASH_DEFAULT_MEMCAP;
+#else
     if (memcap > 0) {
         ctx->config.memcap = memcap;
     } else {
-#ifdef FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION
-        // limit hash size to default when fuzzing
-        ctx->config.memcap = THASH_DEFAULT_MEMCAP;
-#else
         ctx->config.memcap = reset_memcap ? UINT64_MAX : THASH_DEFAULT_MEMCAP;
-#endif
     }
+#endif
     ctx->config.prealloc = THASH_DEFAULT_PREALLOC;
 
     SC_ATOMIC_INIT(ctx->counter);