detect: Correct Coverity REVERSE_INULL issue

This commit corrects the "Null pointer dereferences" issue (CID
1465221).

diff --git a/src/detect-engine.c b/src/detect-engine.c
index b984e587ca4bd0f9e3269cf55b85c4fa4a1d381c..a5f0d9225b2b7177bfa677ddcc59cce2a1bd4fa8 100644 (file)
--- a/src/detect-engine.c
+++ b/src/detect-engine.c
@@ -767,6 +767,10 @@ static void DetectBufferTypeFreeFunc(void *data)
 {
     DetectBufferType *map = (DetectBufferType *)data;
 
+    if (map == NULL) {
+        return;
+    }
+
     /* Release transformation option memory, if any */
     for (int i = 0; i < map->transforms.cnt; i++) {
         if (map->transforms.transforms[i].options == NULL)
@@ -779,9 +783,8 @@ static void DetectBufferTypeFreeFunc(void *data)
         }
         sigmatch_table[map->transforms.transforms[i].transform].Free(NULL, map->transforms.transforms[i].options);
     }
-    if (map != NULL) {
-        SCFree(map);
-    }
+
+    SCFree(map);
 }
 
 static int DetectBufferTypeInit(void)