[PATCH 3/3] grep: plug leak of pcre chartables in PCRE2

From: Carlo Marcelo Arenas Belón <carenas@gmail.com>
Date: Jul 27, 2019, 20:27 UTC
Message-ID: <20190727202759.22310-4-carenas@gmail.com>
In-Reply-To: <20190727202759.22310-1-carenas@gmail.com>

Just as it is done with PCRE1, make sure that the allocated chartables get free at cleanup time.

This assumes no global context is used (NULL passed when created the tables), but will likely be updated in tandem if that ever changes.

Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com>
---
 grep.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/grep.c b/grep.c
index d04635fad4..d9768c5f05 100644
--- a/grep.c
+++ b/grep.c
@@ -604,6 +604,7 @@ static void free_pcre2_pattern(struct grep_pat *p)
 	pcre2_match_data_free(p->pcre2_match_data);
 	pcre2_jit_stack_free(p->pcre2_jit_stack);
 	pcre2_match_context_free(p->pcre2_match_context);
+	free((void *)p->pcre_tables);
 }
 #else /* !USE_LIBPCRE2 */
 static void compile_pcre2_pattern(struct grep_pat *p, const struct grep_opt *opt)

-- 
2.22.0