diff options
| author | David Stacey <drstacey@tiscali.co.uk> | 2015-05-27 13:08:29 +0300 |
|---|---|---|
| committer | Corinna Vinschen <corinna@vinschen.de> | 2015-05-27 14:30:20 +0300 |
| commit | ca632c90157aaa0f0e00843bf8501c6831abe84b (patch) | |
| tree | d4c0ae81cce00274144098f89a974177b421fe52 | |
| parent | 4f3cc70cb6c9b7760d8d13e7661c4d0105006f91 (diff) | |
Fix potential memory leak in argz_replace
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
| -rw-r--r-- | newlib/ChangeLog | 4 | ||||
| -rw-r--r-- | newlib/libc/argz/argz_replace.c | 5 |
2 files changed, 8 insertions, 1 deletions
diff --git a/newlib/ChangeLog b/newlib/ChangeLog index a70fefea9..d1b0afec8 100644 --- a/newlib/ChangeLog +++ b/newlib/ChangeLog @@ -1,3 +1,7 @@ +2015-05-27 David Stacey <drstacey@tiscali.co.uk> + + * libc/argz/argz_replace.c: Fix potential memory leak. + 2015-05-26 DJ Delorie <dj@redhat.com> * libm/math/e_sqrt.c (__ieee754_sqrt): Don't truncate constant to diff --git a/newlib/libc/argz/argz_replace.c b/newlib/libc/argz/argz_replace.c index cb01eaef5..6bfd04b8a 100644 --- a/newlib/libc/argz/argz_replace.c +++ b/newlib/libc/argz/argz_replace.c @@ -71,7 +71,10 @@ _DEFUN (argz_replace, (argz, argz_len, str, with, replace_count), /* reallocate argz, and copy over the new value. */ if(!(*argz = (char *)realloc(*argz, new_argz_len))) - return ENOMEM; + { + free(new_argz); + return ENOMEM; + } memcpy(*argz, new_argz, new_argz_len); *argz_len = new_argz_len; |