From: Victor Julien Date: Fri, 13 May 2022 08:01:51 +0000 (+0200) Subject: detect/iponly: remove unused code X-Git-Tag: suricata-6.0.6~45 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F7494%2Fhead;p=thirdparty%2Fsuricata.git detect/iponly: remove unused code (cherry picked from commit 51ef6f4e3a124816ba8cb17f63060c03120d96ae) --- diff --git a/src/detect-engine-iponly.c b/src/detect-engine-iponly.c index 390e3495e7..d9030b0373 100644 --- a/src/detect-engine-iponly.c +++ b/src/detect-engine-iponly.c @@ -889,15 +889,6 @@ error: */ void IPOnlyInit(DetectEngineCtx *de_ctx, DetectEngineIPOnlyCtx *io_ctx) { - io_ctx->sig_init_size = DetectEngineGetMaxSigId(de_ctx) / 8 + 1; - - if ( (io_ctx->sig_init_array = SCMalloc(io_ctx->sig_init_size)) == NULL) { - FatalError(SC_ERR_FATAL, - "Fatal error encountered in IPOnlyInit. Exiting..."); - } - - memset(io_ctx->sig_init_array, 0, io_ctx->sig_init_size); - io_ctx->tree_ipv4src = SCRadixCreateRadixTree(SigNumArrayFree, SigNumArrayPrint); io_ctx->tree_ipv4dst = SCRadixCreateRadixTree(SigNumArrayFree, @@ -965,10 +956,6 @@ void IPOnlyDeinit(DetectEngineCtx *de_ctx, DetectEngineIPOnlyCtx *io_ctx) if (io_ctx->tree_ipv6dst != NULL) SCRadixReleaseRadixTree(io_ctx->tree_ipv6dst); io_ctx->tree_ipv6dst = NULL; - - if (io_ctx->sig_init_array) - SCFree(io_ctx->sig_init_array); - io_ctx->sig_init_array = NULL; } /** @@ -1589,9 +1576,6 @@ void IPOnlyAddSignature(DetectEngineCtx *de_ctx, DetectEngineIPOnlyCtx *io_ctx, if (s->num > io_ctx->max_idx) io_ctx->max_idx = s->num; - /* enable the sig in the bitarray */ - io_ctx->sig_init_array[(s->num/8)] |= 1 << (s->num % 8); - /** no longer ref to this, it's in the table now */ s->CidrSrc = NULL; s->CidrDst = NULL; diff --git a/src/detect.h b/src/detect.h index 8f1904f74a..24b72b0ef7 100644 --- a/src/detect.h +++ b/src/detect.h @@ -680,31 +680,13 @@ typedef struct DetectEngineIPOnlyThreadCtx_ { /** \brief IP only rules matching ctx. */ typedef struct DetectEngineIPOnlyCtx_ { - /* lookup hashes */ - HashListTable *ht16_src, *ht16_dst; - HashListTable *ht24_src, *ht24_dst; - /* Lookup trees */ SCRadixTree *tree_ipv4src, *tree_ipv4dst; SCRadixTree *tree_ipv6src, *tree_ipv6dst; /* Used to build the radix trees */ IPOnlyCIDRItem *ip_src, *ip_dst; - - /* counters */ - uint32_t a_src_uniq16, a_src_total16; - uint32_t a_dst_uniq16, a_dst_total16; - uint32_t a_src_uniq24, a_src_total24; - uint32_t a_dst_uniq24, a_dst_total24; - uint32_t max_idx; - - uint8_t *sig_init_array; /* bit array of sig nums */ - uint32_t sig_init_size; /* size in bytes of the array */ - - /* number of sigs in this head */ - uint32_t sig_cnt; - uint32_t *match_array; } DetectEngineIPOnlyCtx; typedef struct DetectEngineLookupFlow_ {