summaryrefslogtreecommitdiff
path: root/src/shared
diff options
context:
space:
mode:
authorTom Gundersen <teg@jklm.no>2014-09-18 13:47:00 +0200
committerTom Gundersen <teg@jklm.no>2014-09-18 13:48:44 +0200
commit77c10205bb337585c320e91af4b416f2dcc6faba (patch)
tree26f3f7bb97fd746810f5d5babb1d57b6630bf6ad /src/shared
parent9dedfe7f667a8cb22ba85d0223556c69c4fd0e9a (diff)
shared: conf-parser - don't leak memory on error in DEFINE_CONFIG_PARSE_ENUMV
Found by Coverity. Fixes CID #1237746.
Diffstat (limited to 'src/shared')
-rw-r--r--src/shared/conf-parser.h12
1 files changed, 9 insertions, 3 deletions
diff --git a/src/shared/conf-parser.h b/src/shared/conf-parser.h
index 94185152cd..62f2a01e5e 100644
--- a/src/shared/conf-parser.h
+++ b/src/shared/conf-parser.h
@@ -169,7 +169,8 @@ int log_syntax_internal(const char *unit, int level,
void *data, \
void *userdata) { \
\
- type **enums = data, *xs, x, *ys; \
+ type **enums = data, x, *ys; \
+ _cleanup_free_ type *xs = NULL; \
const char *word, *state; \
size_t l, i = 0; \
\
@@ -186,6 +187,7 @@ int log_syntax_internal(const char *unit, int level,
\
FOREACH_WORD(word, l, rvalue, state) { \
_cleanup_free_ char *en = NULL; \
+ type *new_xs; \
\
en = strndup(word, l); \
if (!en) \
@@ -211,8 +213,10 @@ int log_syntax_internal(const char *unit, int level,
continue; \
\
*(xs + i) = x; \
- xs = realloc(xs, (++i + 1) * sizeof(type)); \
- if (!xs) \
+ new_xs = realloc(xs, (++i + 1) * sizeof(type)); \
+ if (new_xs) \
+ xs = new_xs; \
+ else \
return -ENOMEM; \
\
*(xs + i) = invalid; \
@@ -220,5 +224,7 @@ int log_syntax_internal(const char *unit, int level,
\
free(*enums); \
*enums = xs; \
+ xs = NULL; \
+ \
return 0; \
}