]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
ndpi: fix memory in keyword setup
authorJason Ish <jason.ish@oisf.net>
Fri, 24 Jan 2025 17:18:54 +0000 (11:18 -0600)
committerVictor Julien <victor@inliniac.net>
Sat, 29 Mar 2025 21:14:00 +0000 (22:14 +0100)
- remove duplicate calls to ndpi_init_detection_module
- cleanup ndpi_init_detection_module when no longer needed

plugins/ndpi/ndpi.c

index ec0342c78ae8e2f189f56122f5f40edbae7b8dc5..60accf457680e7d4bd3bf330652bc7d575d42870 100644 (file)
@@ -226,7 +226,6 @@ static DetectnDPIProtocolData *DetectnDPIProtocolParse(const char *arg, bool neg
     if (unlikely(ndpi_struct == NULL))
         return NULL;
 
-    ndpi_struct = ndpi_init_detection_module(NULL);
     NDPI_BITMASK_SET_ALL(all);
     ndpi_set_protocol_detection_bitmask2(ndpi_struct, &all);
     ndpi_finalize_initialization(ndpi_struct);
@@ -346,10 +345,10 @@ static DetectnDPIRiskData *DetectnDPIRiskParse(const char *arg, bool negate)
     if (unlikely(ndpi_struct == NULL))
         return NULL;
 
-    ndpi_struct = ndpi_init_detection_module(NULL);
     NDPI_BITMASK_SET_ALL(all);
     ndpi_set_protocol_detection_bitmask2(ndpi_struct, &all);
     ndpi_finalize_initialization(ndpi_struct);
+    ndpi_exit_detection_module(ndpi_struct);
 
     if (isdigit(arg[0]))
         risk_mask = atoll(arg);