diff options
| author | Sami Kerola | 2012-06-09 23:05:23 +0200 |
|---|---|---|
| committer | Sami Kerola | 2012-06-11 19:50:35 +0200 |
| commit | 41420120a865186a1cbd97f01aea00c809d6b6ba (patch) | |
| tree | 5c30c778acf3d8271755fa3773879401796a0e0c /misc-utils/blkid.c | |
| parent | setarch: do not use -1 as array index [cppcheck] (diff) | |
| download | kernel-qcow2-util-linux-41420120a865186a1cbd97f01aea00c809d6b6ba.tar.gz kernel-qcow2-util-linux-41420120a865186a1cbd97f01aea00c809d6b6ba.tar.xz kernel-qcow2-util-linux-41420120a865186a1cbd97f01aea00c809d6b6ba.zip | |
blkid: fix realloc memory leak [cppcheck]
[misc-utils/blkid.c:367]: (error) Common realloc mistake: 'str' nulled but not freed upon failure
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Diffstat (limited to 'misc-utils/blkid.c')
| -rw-r--r-- | misc-utils/blkid.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/misc-utils/blkid.c b/misc-utils/blkid.c index 25e6c00e8..840cfe4f6 100644 --- a/misc-utils/blkid.c +++ b/misc-utils/blkid.c @@ -365,8 +365,10 @@ static int append_str(char **res, size_t *sz, const char *a, const char *b) return -1; str = realloc(str, len + 1); - if (!str) + if (!str) { + free(*res); return -1; + } *res = str; str += *sz; |