Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Aug 2012 20:30:17 GMT
From:      dfilter@FreeBSD.ORG (dfilter service)
To:        freebsd-geom@FreeBSD.org
Subject:   Re: kern/170199: commit references a PR
Message-ID:  <201208032030.q73KUHPG056605@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help
The following reply was made to PR kern/170199; it has been noted by GNATS.

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/170199: commit references a PR
Date: Fri,  3 Aug 2012 20:24:31 +0000 (UTC)

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



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