d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

If realloc(3) fails in copyline(), do not make matters worse by

Browse Source 
leaving without deallocating `data' thereby creating a memory leak.
This commit is contained in:
Robert Drehmel 2003-03-19 14:01:35 +00:00
parent 0f02c85ed1
commit 916560b152
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
lib/libc/gen/getgrent.c
View File

@ -293,9 +293,12 @@ _dns_grscan(rv, cb_data, ap)
}
/* only check first elem */
if (copyline(hp[0]) == 0)
return NS_UNAVAIL;
r = copyline(hp[0]);
hesiod_free_list(context, hp);
if (r == 0) {
r = NS_UNAVAIL;
break;
}
if (matchline(search, gid, name)) {
r = NS_SUCCESS;
break;
@ -360,9 +363,10 @@ _nis_grscan(rv, cb_data, ap)
return NS_UNAVAIL;
}
data[datalen] = '\0'; /* clear trailing \n */
if (copyline(data) == 0)
return NS_UNAVAIL;
r = copyline(data);
free(data);
if (r == 0)
return NS_UNAVAIL;
if (matchline(search, gid, name))
return NS_SUCCESS;
else
@ -410,9 +414,10 @@ _nis_grscan(rv, cb_data, ap)
}
}
data[datalen] = '\0'; /* clear trailing \n */
if (copyline(data) == 0)
return NS_UNAVAIL;
r = copyline(data);
free(data);
if (r == 0)
return NS_UNAVAIL;
if (matchline(search, gid, name))
return NS_SUCCESS;
}