svn commit: r289601 - head/usr.sbin/makefs

Garrett Cooper ngie at FreeBSD.org
Mon Oct 19 18:45:16 UTC 2015


Author: ngie
Date: Mon Oct 19 18:45:14 2015
New Revision: 289601
URL: https://svnweb.freebsd.org/changeset/base/289601

Log:
  Don't check if `val` is NULL before calling free; free(3) already
  handles this
  
  MFC after: 1 week
  PR: 203649
  Submitted by: Thomas Schmitt <scdbackup at gmx.net>
  Coverity CID: 1305659
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/usr.sbin/makefs/cd9660.c

Modified: head/usr.sbin/makefs/cd9660.c
==============================================================================
--- head/usr.sbin/makefs/cd9660.c	Mon Oct 19 18:29:32 2015	(r289600)
+++ head/usr.sbin/makefs/cd9660.c	Mon Oct 19 18:45:14 2015	(r289601)
@@ -428,8 +428,7 @@ cd9660_parse_opts(const char *option, fs
 			rv = set_option(cd9660_options, var, val);
 	}
 
-	if (var)
-		free(var);
+	free(var);
 	return (rv);
 }
 


More information about the svn-src-head mailing list