Zbigniew Jędrzejewski-Szmek 62fe94
From 77c10205bb337585c320e91af4b416f2dcc6faba Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Tom Gundersen <teg@jklm.no>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Thu, 18 Sep 2014 13:47:00 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] shared: conf-parser - don't leak memory on error in
Zbigniew Jędrzejewski-Szmek 62fe94
 DEFINE_CONFIG_PARSE_ENUMV
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
Found by Coverity. Fixes CID #1237746.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/shared/conf-parser.h | 12 +++++++++---
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 9 insertions(+), 3 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/shared/conf-parser.h b/src/shared/conf-parser.h
Zbigniew Jędrzejewski-Szmek 62fe94
index 94185152cd..62f2a01e5e 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/shared/conf-parser.h
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/shared/conf-parser.h
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -169,7 +169,8 @@ int log_syntax_internal(const char *unit, int level,
Zbigniew Jędrzejewski-Szmek 62fe94
                      void *data,                                               \
Zbigniew Jędrzejewski-Szmek 62fe94
                      void *userdata) {                                         \
Zbigniew Jędrzejewski-Szmek 62fe94
                                                                                \
Zbigniew Jędrzejewski-Szmek 62fe94
-                type **enums = data, *xs, x, *ys;                              \
Zbigniew Jędrzejewski-Szmek 62fe94
+                type **enums = data, x, *ys;                                   \
Zbigniew Jędrzejewski-Szmek 62fe94
+                _cleanup_free_ type *xs = NULL;                                \
Zbigniew Jędrzejewski-Szmek 62fe94
                 const char *word, *state;                                      \
Zbigniew Jędrzejewski-Szmek 62fe94
                 size_t l, i = 0;                                               \
Zbigniew Jędrzejewski-Szmek 62fe94
                                                                                \
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -186,6 +187,7 @@ int log_syntax_internal(const char *unit, int level,
Zbigniew Jędrzejewski-Szmek 62fe94
                                                                                \
Zbigniew Jędrzejewski-Szmek 62fe94
                 FOREACH_WORD(word, l, rvalue, state) {                         \
Zbigniew Jędrzejewski-Szmek 62fe94
                         _cleanup_free_ char *en = NULL;                        \
Zbigniew Jędrzejewski-Szmek 62fe94
+                        type *new_xs;                                          \
Zbigniew Jędrzejewski-Szmek 62fe94
                                                                                \
Zbigniew Jędrzejewski-Szmek 62fe94
                         en = strndup(word, l);                                 \
Zbigniew Jędrzejewski-Szmek 62fe94
                         if (!en)                                               \
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -211,8 +213,10 @@ int log_syntax_internal(const char *unit, int level,
Zbigniew Jędrzejewski-Szmek 62fe94
                                 continue;                                      \
Zbigniew Jędrzejewski-Szmek 62fe94
                                                                                \
Zbigniew Jędrzejewski-Szmek 62fe94
                         *(xs + i) = x;                                         \
Zbigniew Jędrzejewski-Szmek 62fe94
-                        xs = realloc(xs, (++i + 1) * sizeof(type));            \
Zbigniew Jędrzejewski-Szmek 62fe94
-                        if (!xs)                                               \
Zbigniew Jędrzejewski-Szmek 62fe94
+                        new_xs = realloc(xs, (++i + 1) * sizeof(type));        \
Zbigniew Jędrzejewski-Szmek 62fe94
+                        if (new_xs)                                            \
Zbigniew Jędrzejewski-Szmek 62fe94
+                                xs = new_xs;                                   \
Zbigniew Jędrzejewski-Szmek 62fe94
+                        else                                                   \
Zbigniew Jędrzejewski-Szmek 62fe94
                                 return -ENOMEM;                                \
Zbigniew Jędrzejewski-Szmek 62fe94
                                                                                \
Zbigniew Jędrzejewski-Szmek 62fe94
                         *(xs + i) = invalid;                                   \
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -220,5 +224,7 @@ int log_syntax_internal(const char *unit, int level,
Zbigniew Jędrzejewski-Szmek 62fe94
                                                                                \
Zbigniew Jędrzejewski-Szmek 62fe94
                 free(*enums);                                                  \
Zbigniew Jędrzejewski-Szmek 62fe94
                 *enums = xs;                                                   \
Zbigniew Jędrzejewski-Szmek 62fe94
+                xs = NULL;                                                     \
Zbigniew Jędrzejewski-Szmek 62fe94
+                                                                               \
Zbigniew Jędrzejewski-Szmek 62fe94
                 return 0;                                                      \
Zbigniew Jędrzejewski-Szmek 62fe94
         }