From ecd5c7573b1adfdf48c42ce8681335ffa47f5700 Mon Sep 17 00:00:00 2001
From: Victor Julien <victor@inliniac.net>
Date: Mon, 16 Sep 2013 16:22:24 +0200
Subject: [PATCH] Coverity 1038135 fix

Small cleanup in the error handling. The extra null check confused
Coverity.
---
 src/detect-engine-siggroup.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/detect-engine-siggroup.c b/src/detect-engine-siggroup.c
index 2c21d4d00b..c300cd11c8 100644
--- a/src/detect-engine-siggroup.c
+++ b/src/detect-engine-siggroup.c
@@ -156,8 +156,7 @@ static SigGroupHead *SigGroupHeadAlloc(DetectEngineCtx *de_ctx, uint32_t size)
     return sgh;
 
 error:
-    if (sgh != NULL)
-        SigGroupHeadFree(sgh);
+    SigGroupHeadFree(sgh);
     return NULL;
 }
 
-- 
2.47.3