* 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].
This commit is contained in:
Ulrich Drepper 2006-04-08 19:35:24 +00:00
parent 8b7600e8cf
commit db67c2c98b
3 changed files with 18 additions and 2 deletions

View File

@ -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

View File

@ -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);
}

View File

@ -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;
}