svn commit: r239021 - head/sys/geom/virstor

Jim Harris jimharris at FreeBSD.org
Fri Aug 3 20:24:17 UTC 2012


Author: jimharris
Date: Fri Aug  3 20:24:16 2012
New Revision: 239021
URL: http://svn.freebsd.org/changeset/base/239021

Log:
  In virstor_ctl_stop(), check for a valid softc before trying to update
   metadata.
  
  Sponsored by:		Intel
  Reported and tested by:	Marcelo Gondim <gondim at bsdinfo dot com dot br>
  PR:			kern/170199
  MFC after:		3 days

Modified:
  head/sys/geom/virstor/g_virstor.c

Modified: head/sys/geom/virstor/g_virstor.c
==============================================================================
--- head/sys/geom/virstor/g_virstor.c	Fri Aug  3 18:40:44 2012	(r239020)
+++ head/sys/geom/virstor/g_virstor.c	Fri Aug  3 20:24:16 2012	(r239021)
@@ -235,6 +235,12 @@ virstor_ctl_stop(struct gctl_req *req, s
 			return;
 		}
 		sc = virstor_find_geom(cp, name);
+		if (sc == NULL) {
+			gctl_error(req, "Don't know anything about '%s'", name);
+			g_topology_unlock();
+			return;
+		}
+
 		LOG_MSG(LVL_INFO, "Stopping %s by the userland command",
 		    sc->geom->name);
 		update_metadata(sc);


More information about the svn-src-all mailing list