svn commit: r260883 - head/sys/geom/multipath

Alexander Motin mav at FreeBSD.org
Sun Jan 19 16:37:58 UTC 2014


Author: mav
Date: Sun Jan 19 16:37:57 2014
New Revision: 260883
URL: http://svnweb.freebsd.org/changeset/base/260883

Log:
  Removed unneeded and dangerous assignment.  It would probably cause NULL
  refererence panic if compiler not optimize it out.
  
  Found with:	Clang static analyzer
  MFC after:	2 weeks

Modified:
  head/sys/geom/multipath/g_multipath.c

Modified: head/sys/geom/multipath/g_multipath.c
==============================================================================
--- head/sys/geom/multipath/g_multipath.c	Sun Jan 19 16:07:27 2014	(r260882)
+++ head/sys/geom/multipath/g_multipath.c	Sun Jan 19 16:37:57 2014	(r260883)
@@ -1089,7 +1089,6 @@ g_multipath_ctl_create(struct gctl_req *
 		gctl_error(req, "Device %s already exist", mpname);
 		return;
 	}
-	sc = gp->softc;
 
 	memset(&md, 0, sizeof(md));
 	strlcpy(md.md_magic, G_MULTIPATH_MAGIC, sizeof(md.md_magic));


More information about the svn-src-head mailing list