kern/116896: Typo in a kassert in GEOM

VANHULLEBUS Yvan vanhu at netasq.com
Thu Oct 4 02:20:04 PDT 2007


>Number:         116896
>Category:       kern
>Synopsis:       Typo in a kassert in GEOM
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Oct 04 09:20:02 GMT 2007
>Closed-Date:
>Last-Modified:
>Originator:     VANHULLEBUS Yvan
>Release:        FreeBSD 6.2-STABLE i386
>Organization:
NETASQ
>Environment:
System: FreeBSD darkstar.netasq.com 6.2-STABLE FreeBSD 6.2-STABLE #0: Mon Jan 22 14:18:45 CET 2007 root at darkstar.netasq.com:/usr/obj/usr/src/sys/GENERIC i386


	
>Description:

I'm not sure if this could be a real problem, but there is a small typo
in a KASSERT: 
in geom/geom_subr.c, function g_destroy_provider(), pp->acw is checked twice
but pp->ace is not checked.

>How-To-Repeat:


>Fix:
Index: geom/geom_subr.c
===================================================================
RCS file: /home/ncvs/src/sys/geom/geom_subr.c,v
retrieving revision 1.91
diff -b -u -p -r1.91 geom_subr.c
--- geom/geom_subr.c	5 May 2007 16:33:44 -0000	1.91
+++ geom/geom_subr.c	4 Oct 2007 09:14:13 -0000
@@ -554,7 +554,7 @@ g_destroy_provider(struct g_provider *pp
 	    ("g_destroy_provider but attached"));
 	KASSERT (pp->acr == 0, ("g_destroy_provider with acr"));
 	KASSERT (pp->acw == 0, ("g_destroy_provider with acw"));
-	KASSERT (pp->acw == 0, ("g_destroy_provider with ace"));
+	KASSERT (pp->ace == 0, ("g_destroy_provider with ace"));
 	g_cancel_event(pp);
 	LIST_REMOVE(pp, provider);
 	gp = pp->geom;

>Release-Note:
>Audit-Trail:
>Unformatted:


More information about the freebsd-bugs mailing list