]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
pcre: work around harmless coverity warnings 1869/head
authorVictor Julien <victor@inliniac.net>
Tue, 23 Feb 2016 11:12:24 +0000 (12:12 +0100)
committerVictor Julien <victor@inliniac.net>
Tue, 23 Feb 2016 13:57:46 +0000 (14:57 +0100)
src/detect-pcre.c

index d69f1da6f9de23744390ee2a7fc05d5919d9264f..c14fe9f51207f100c36501089d8762b70e74d52e 100644 (file)
@@ -306,8 +306,12 @@ static DetectPcreData *DetectPcreParse (DetectEngineCtx *de_ctx, char *regexstr,
     int ret = 0, res = 0;
     int ov[MAX_SUBSTRINGS];
 
-    size_t slen = strlen(regexstr);
-    char re[slen], op_str[64] = "";
+    /* take the size of the whole input as buffer size for the regex we will
+     * extract below. Add 1 to please Coverity's alloc_strlen test. */
+    size_t slen = strlen(regexstr) + 1;
+    char re[slen];
+
+    char op_str[64] = "";
     uint16_t pos = 0;
     uint8_t negate = 0;
 
@@ -591,7 +595,10 @@ static int DetectPcreParseCapture(char *regexstr, DetectEngineCtx *de_ctx, Detec
     int ret = 0, res = 0;
     int ov[MAX_SUBSTRINGS];
     char type_str[16] = "";
-    size_t cap_buffer_len = strlen(regexstr);
+
+    /* take the size of the whole input as buffer size for the string we will
+     * extract below. Add 1 to please Coverity's alloc_strlen test. */
+    size_t cap_buffer_len = strlen(regexstr) + 1;
     char capture_str[cap_buffer_len];
     memset(capture_str, 0x00, cap_buffer_len);