Fix memory leak when batch-reading large NIS password maps.

Not the whole was traversed in the function to free all buffers.
BZ #10203

diff --git a/ChangeLog b/ChangeLog
index 402b5cb..2bfa774 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2009-06-15  Ulrich Drepper  <drepper@redhat.com>
 
+       [BZ #10203]
+       * nis/nss_nis/nis-pwd.c (internal_nis_endpwent): Free all buffers,
+       not just the currently used one and those which follow.
+       Patch by Joe Landers <jlanders@vmware.com>.
+
        [BZ #10196]
        * libio/tst-fgetwc.c (do_test): Use de_DE.UTF-8 as locale name.
        Patch by Bruce Dubbs <bdubbs@linuxfromscratch.org>.

diff --git a/nis/nss_nis/nis-pwd.c b/nis/nss_nis/nis-pwd.c
index 1b5206a..fdc7dc9 100644 (file)
--- a/nis/nss_nis/nis-pwd.c
+++ b/nis/nss_nis/nis-pwd.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1996-1998,2001,2002,2003,2006 Free Software Foundation, Inc.
+/* Copyright (C) 1996-1998,2001-2003,2006,2009 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Thorsten Kukuk <kukuk@vt.uni-paderborn.de>, 1996.
 
@@ -116,7 +116,7 @@ internal_nis_endpwent (void)
       oldkeylen = 0;
     }
 
-  struct response_t *curr = intern.next;
+  struct response_t *curr = intern.start;
 
   while (curr != NULL)
     {