From 9d121ad8eb563ab0d754fdb24251c88fa6b34707 Mon Sep 17 00:00:00 2001 From: truckman Date: Wed, 15 Jun 2016 06:42:30 +0000 Subject: [PATCH] MFC r301596 Don't leak olinep if malloc() fails. If malloc() fails to allocate linep, then free olinep (if it exists) before returning to avoid a memory leak. Reported by: Coverity CID: 1016716 Reviewed by: kib Differential Revision: https://reviews.freebsd.org/D6755 git-svn-id: svn://svn.freebsd.org/base/stable/10@301921 ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f --- lib/libc/gen/getnetgrent.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/libc/gen/getnetgrent.c b/lib/libc/gen/getnetgrent.c index 4bf0a1468..8aa75e2e9 100644 --- a/lib/libc/gen/getnetgrent.c +++ b/lib/libc/gen/getnetgrent.c @@ -615,6 +615,8 @@ read_for_group(const char *group) if (linep == NULL) { free(lp->l_groupname); free(lp); + if (olen > 0) + free(olinep); return (NULL); } if (olen > 0) { -- 2.45.0