Ok to push this?  Sorry, I haven't yet explored how valgrind tests are 
supposed to be added to gnulib.

>From 165ffc9aaa9bfe41a9977a80edf80e830d765ab9 Mon Sep 17 00:00:00 2001
From: Joel E. Denny <jde...@clemson.edu>
Date: Tue, 15 Dec 2009 02:43:07 -0500
Subject: [PATCH] fprintf-posix: fix memory leak

* lib/fprintf.c (fprintf): Free memory allocated by vasnprintf.
---
 ChangeLog     |    5 +++++
 lib/fprintf.c |    3 +++
 2 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 2756f67..05d75db 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2009-12-15  Joel E. Denny  <jde...@clemson.edu>
+
+       fprintf-posix: fix memory leak
+       * lib/fprintf.c (fprintf): Free memory allocated by vasnprintf.
+
 2009-12-14  Eric Blake  <e...@byu.net>
 
        accept4: adjust module dependencies
diff --git a/lib/fprintf.c b/lib/fprintf.c
index eb1cf8e..00cbb60 100644
--- a/lib/fprintf.c
+++ b/lib/fprintf.c
@@ -63,6 +63,9 @@ fprintf (FILE *fp, const char *format, ...)
       return -1;
     }
 
+  if (output != buf)
+    free (output);
+
   if (len > INT_MAX)
     {
       errno = EOVERFLOW;
-- 
1.5.4.3



Reply via email to