Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 19 Oct 2015 18:45:14 +0000 (UTC)
From:      Garrett Cooper <ngie@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r289601 - head/usr.sbin/makefs
Message-ID:  <201510191845.t9JIjE1O098034@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
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@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);
 }
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201510191845.t9JIjE1O098034>