From 190afa261ad4c77130b136c6f8b72227a9463543 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=A9mie=20Galarneau?= Date: Sat, 6 May 2017 15:20:29 -0400 Subject: [PATCH] Fix: return of free'd string on error in strutils_split() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Reported-by: Coverity Scan CID 1374808 (#1 of 1): Use after free (USE_AFTER_FREE) use_after_free: Using freed pointer substrings. Signed-off-by: Jérémie Galarneau --- src/common/string-utils/string-utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/common/string-utils/string-utils.c b/src/common/string-utils/string-utils.c index c91d920e5..8e1f516b6 100644 --- a/src/common/string-utils/string-utils.c +++ b/src/common/string-utils/string-utils.c @@ -355,7 +355,7 @@ char **strutils_split(const char *input, char delim, bool escape_delim) error: strutils_free_null_terminated_array_of_strings(substrings); - + substrings = NULL; end: return substrings; } -- 2.34.1