svn commit: r240881 - head/lib/libstand

Kevin Lo kevlo at FreeBSD.org
Mon Sep 24 05:24:11 UTC 2012


Author: kevlo
Date: Mon Sep 24 05:24:10 2012
New Revision: 240881
URL: http://svn.freebsd.org/changeset/base/240881

Log:
  Revert r240850 and remove redundant NULL check before free(3).
  free(3) handles NULL parameter fine.
  
  Reviewed by:	kib, Garrett Cooper

Modified:
  head/lib/libstand/nfs.c

Modified: head/lib/libstand/nfs.c
==============================================================================
--- head/lib/libstand/nfs.c	Mon Sep 24 03:14:17 2012	(r240880)
+++ head/lib/libstand/nfs.c	Mon Sep 24 05:24:10 2012	(r240881)
@@ -606,10 +606,8 @@ nfs_open(const char *upath, struct open_
 	error = 0;
 
 out:
-	if (newfd)
-		free(newfd);
-	if (path)
-		free(path);
+	free(newfd);
+	free(path);
 #else
         currfd->iodesc = desc;
 
@@ -1256,10 +1254,8 @@ nfs_open(const char *upath, struct open_
 	error = 0;
 
 out:
-	if (newfd)
-		free(newfd);
-	if (path)
-		free(path);
+	free(newfd);
+	free(path);
 #else
 	currfd->iodesc = desc;
 


More information about the svn-src-all mailing list