- (dtucker) [openbsd-compat/bsd-asprintf.c] Plug mem leak in error path.

   Patch from Jan.Pechanec at sun com.

diff --git a/ChangeLog b/ChangeLog
index b290326a1aa1766a3231ca66fb69b59f281a7e6a..6d2f482756acee55a0fea32674dfa42fd34b4b7b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+20070914
+ - (dtucker) [openbsd-compat/bsd-asprintf.c] Plug mem leak in error path.
+   Patch from Jan.Pechanec at sun com.
+
 20070910
  - (dtucker) [openbsd-compat/regress/closefromtest.c] Bug #1358: Always
    return 0 on successful test.  From David.Leonard at quest com.

diff --git a/openbsd-compat/bsd-asprintf.c b/openbsd-compat/bsd-asprintf.c
index 00fa0dfd8a7da3bad6684929d0f2550582bc3137..3368195d443ae1a8470528ecf0211ccdfba17847 100644 (file)
--- a/openbsd-compat/bsd-asprintf.c
+++ b/openbsd-compat/bsd-asprintf.c
@@ -55,6 +55,7 @@ vasprintf(char **str, const char *fmt, va_list ap)
        if (ret >= 0 && ret < INIT_SZ) { /* succeeded with initial alloc */
                *str = string;
        } else if (ret == INT_MAX || ret < 0) { /* Bad length */
+               free(string);
                goto fail;
        } else {        /* bigger than initial, realloc allowing for nul */
                len = (size_t)ret + 1;