Use define instead of magic number for pmq's per detect thread

diff --git a/src/detect-engine.c b/src/detect-engine.c
index da13f39778e3fbf536445b32b83f748e1ff5a77a..bf45329d8b7ca22b1d8ae8bed744c81d0e53add6 100644 (file)
--- a/src/detect-engine.c
+++ b/src/detect-engine.c
@@ -1069,7 +1069,7 @@ static TmEcode ThreadCtxDoInit (DetectEngineCtx *de_ctx, DetectEngineThreadCtx *
     PatternMatchThreadPrepare(&det_ctx->mtcu, de_ctx->mpm_matcher, DetectUricontentMaxId(de_ctx));
 
     PmqSetup(&det_ctx->pmq, 0, de_ctx->max_fp_id);
-    for (i = 0; i < 256; i++) {
+    for (i = 0; i < DETECT_SMSG_PMQ_NUM; i++) {
         PmqSetup(&det_ctx->smsg_pmq[i], 0, de_ctx->max_fp_id);
     }
 
@@ -1196,7 +1196,7 @@ TmEcode DetectEngineThreadCtxDeinit(ThreadVars *tv, void *data) {
 
     PmqFree(&det_ctx->pmq);
     int i;
-    for (i = 0; i < 256; i++) {
+    for (i = 0; i < DETECT_SMSG_PMQ_NUM; i++) {
         PmqFree(&det_ctx->smsg_pmq[i]);
     }
 

diff --git a/src/detect.h b/src/detect.h
index 480ce61e2fee9f4369180e338de7dca64d6ec4a8..dbbf90d06addff821a9894d5b6daf43c964a27c4 100644 (file)
--- a/src/detect.h
+++ b/src/detect.h
@@ -734,6 +734,8 @@ typedef struct HttpReassembledBody_ {
 } HttpReassembledBody;
 
 #define DETECT_FILESTORE_MAX 15
+/** \todo review how many we actually need here */
+#define DETECT_SMSG_PMQ_NUM 256
 
 /**
   * Detection engine thread data.
@@ -805,7 +807,7 @@ typedef struct DetectionEngineThreadCtx_ {
     MpmThreadCtx mtcu;  /**< thread ctx for uricontent mpm */
     MpmThreadCtx mtcs;  /**< thread ctx for stream mpm */
     PatternMatcherQueue pmq;
-    PatternMatcherQueue smsg_pmq[256];
+    PatternMatcherQueue smsg_pmq[DETECT_SMSG_PMQ_NUM];
 
     /** ip only rules ctx */
     DetectEngineIPOnlyThreadCtx io_ctx;