[fedfs-utils] [PATCH 6/9] nsdb: don't leak a read buffer upon certfile open failure

Jim Meyering jim at meyering.net
Thu Dec 1 07:58:23 PST 2011


From: Jim Meyering <meyering at redhat.com>

* src/libnsdb/nsdb.c (nsdb_read_certfile): Free "buf" also
on the failed-open path.  Spotted by coverity.

Signed-off-by: Jim Meyering <meyering at redhat.com>
---
 src/libnsdb/nsdb.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/libnsdb/nsdb.c b/src/libnsdb/nsdb.c
index c81f228..02c0588 100644
--- a/src/libnsdb/nsdb.c
+++ b/src/libnsdb/nsdb.c
@@ -524,6 +524,7 @@ nsdb_read_certfile(const char *pathname, char **certdata,
 	if (fd == -1) {
 		xlog(D_GENERAL, "%s: Failed to open %s: %m",
 			__func__, pathname);
+		free(buf);
 		goto out;
 	}

-- 
1.7.8.rc4




More information about the fedfs-utils-devel mailing list