From: Jim Meyering <meyering@redhat.com>
Date: Mon, 18 Jan 2010 07:10:07 +0000 (+0100)
Subject: readtokens: do not leak internal token_lengths buffer
X-Git-Tag: v0.1~4740
X-Git-Url: http://erislabs.org.uk/gitweb/?a=commitdiff_plain;h=06758294115f10dd93055a531113c24f8930f992;p=gnulib.git

readtokens: do not leak internal token_lengths buffer

* lib/readtokens.c (readtokens): Free the local, lengths,
when the supplied "token_lengths" parameter is NULL.
---

diff --git a/ChangeLog b/ChangeLog
index 56b60d2a3..42257ee2f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2010-01-18  Jim Meyering  <meyering@redhat.com>
+
+	readtokens: do not leak internal token_lengths buffer
+	* lib/readtokens.c (readtokens): Free the local, lengths,
+	when the supplied "token_lengths" parameter is NULL.
+
 2010-01-17  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
 
 	Fix a couple of missing LIBTHREAD link failures on AIX.
diff --git a/lib/readtokens.c b/lib/readtokens.c
index e29f20e69..c65c1b13d 100644
--- a/lib/readtokens.c
+++ b/lib/readtokens.c
@@ -197,5 +197,7 @@ readtokens (FILE *stream,
   *tokens_out = tokens;
   if (token_lengths != NULL)
     *token_lengths = lengths;
+  else
+    free (lengths);
   return n_tokens;
 }