[PATCH] Do not leak memory in is_mountpoint()

[PATCH] Do not leak memory in is_mountpoint()

[Olaf Kirch]

Not sure if this was discussed before. Anyway, I
stumbled across this today.

Cheers,
Olaf
------------------------------------------------------------------
From: Olaf Kirch <okir@suse.de>
Subject: Do not leak memory in is_mountpoint()

Kind of obvious when you look at the code :-)

Signed-off-by: Olaf Kirch <okir@suse.de>

 support/misc/mountpoint.c |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

Index: nfs-utils-1.0.8/support/misc/mountpoint.c
===================================================================
--- nfs-utils-1.0.8.orig/support/misc/mountpoint.c
+++ nfs-utils-1.0.8/support/misc/mountpoint.c
@@ -20,17 +20,20 @@ is_mountpoint(char *path)
 	 */
 	char *dotdot;
 	struct stat stb, pstb;
+	int rv = 0;
 
 	dotdot = malloc(strlen(path)+4);
 	strcat(strcpy(dotdot, path), "/..");
 	if (lstat(path, &stb) != 0 ||
 	    lstat(dotdot, &pstb) != 0)
-		return 0;
+		goto done;
 
 	if (stb.st_dev != pstb.st_dev
 	    || stb.st_ino == pstb.st_ino)
-		return 1;
-	return 0;
+		rv = 1;
+
+done:	free(dotdot);
+	return rv;
 }
 
 int
-- 
Walks like a duck. Quacks like a duck. Must be a chicken.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs

Re: [PATCH] Do not leak memory in is_mountpoint()

[Olaf Kirch]

On Wed, Nov 29, 2006 at 12:51:03PM +0100, Olaf Kirch wrote:
> Not sure if this was discussed before. Anyway, I
> stumbled across this today.

Okay, sorry for the noise. I was looking at old source.

Olaf
-- 
Walks like a duck. Quacks like a duck. Must be a chicken.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs