rule-parsing: remove unnecessary code

Those lines were included former to prevent possible null pointer
dereference but that won't happen anymore with the rest of the rework
done in the code. The code even results in a control flow issue reported
by coverity scan, so just remove it.

diff --git a/src/detect-msg.c b/src/detect-msg.c
index eddf311972568c0a58cd636c73ad6bd860004c7d..5a5f4c5dfd513fbde8565f0ff6f8a066664c1204 100644 (file)
--- a/src/detect-msg.c
+++ b/src/detect-msg.c
@@ -75,10 +75,6 @@ static int DetectMsgSetup (DetectEngineCtx *de_ctx, Signature *s, char *msgstr)
         goto error;
     }
 
-    /* make sure that we don't proceed with null pointer */
-    if (unlikely(str == NULL))
-        goto error;
-
     len = strlen(str);
     if (len == 0)
         goto error;