* io/fts.c (fts_build): Call fts_lfree in the two error cases

	after the loop [Coverity CID 187].

	* nis/nis_getservlist.c (nis_getservlist): Also free lookup result
	in error case [Coverity CID 189].

	* nis/nis_getservlist.c (nis_getservlist): One more free call
	needed [Coverity CID 190].

diff --git a/ChangeLog b/ChangeLog
index 0c72a94..ed170f4 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2006-04-08  Ulrich Drepper  <drepper@redhat.com>
+
+       * io/fts.c (fts_build): Call fts_lfree in the two error cases
+       after the loop [Coverity CID 187].
+
+       * nis/nis_getservlist.c (nis_getservlist): Also free lookup result
+       in error case [Coverity CID 189].
+
+       * nis/nis_getservlist.c (nis_getservlist): One more free call
+       needed [Coverity CID 190].
+
 2006-04-07  Jakub Jelinek  <jakub@redhat.com>
 
        * sysdeps/generic/unwind-dw2-fde.c (end_fde_sort): Remove

diff --git a/io/fts.c b/io/fts.c
index 71400d1..6383fe8 100644 (file)
--- a/io/fts.c
+++ b/io/fts.c
@@ -828,6 +828,7 @@ mem1:                               saved_errno = errno;
             fts_safe_changedir(sp, cur->fts_parent, -1, ".."))) {
                cur->fts_info = FTS_ERR;
                SET(FTS_STOP);
+               fts_lfree(head);
                return (NULL);
        }
 
@@ -835,6 +836,7 @@ mem1:                               saved_errno = errno;
        if (!nitems) {
                if (type == BREAD)
                        cur->fts_info = FTS_DP;
+               fts_lfree(head);
                return (NULL);
        }
 

diff --git a/nis/nis_getservlist.c b/nis/nis_getservlist.c
index bea9d4d..e57649d 100644 (file)
--- a/nis/nis_getservlist.c
+++ b/nis/nis_getservlist.c
@@ -72,6 +72,10 @@ nis_getservlist (const_nis_name dir)
                  free (serv[i]);
                }
 
+             free (serv);
+
+             nis_freeresult (res);
+
              return NULL;
            }
 
@@ -141,8 +145,7 @@ nis_getservlist (const_nis_name dir)
        serv[0] = NULL;
     }
 
-  if (res != NULL)
-    nis_freeresult (res);
+  nis_freeresult (res);
 
   return serv;
 }