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;
 
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to