Date: Fri, 6 Jul 2007 16:41:51 GMT From: Marko Zec <zec@FreeBSD.org> To: Perforce Change Reviews <perforce@FreeBSD.org> Subject: PERFORCE change 123023 for review Message-ID: <200707061641.l66GfpWE028959@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=123023 Change 123023 by zec@zec_tca51 on 2007/07/06 16:41:46 linkNum in ng_bridge can be equal to NG_BRIDGE_MAX_LINKS since the introduction of a special-purpose "anchor" hook, which is always has the maximum possible linkNum. Hence, update the assert accordingly not to panic on "anchor" hook deletion. NB. the "anchor" hack might become replaced by a more generic mechanism in the future for pinning up persisten ng nodes. Affected files ... .. //depot/projects/vimage/src/sys/netgraph/ng_bridge.c#3 edit Differences ... ==== //depot/projects/vimage/src/sys/netgraph/ng_bridge.c#3 (text+ko) ==== @@ -791,7 +791,7 @@ /* Get link number */ linkNum = (intptr_t)NG_HOOK_PRIVATE(hook); - KASSERT(linkNum >= 0 && linkNum < NG_BRIDGE_MAX_LINKS, + KASSERT(linkNum >= 0 && linkNum <= NG_BRIDGE_MAX_LINKS, ("%s: linkNum=%u", __func__, linkNum)); /* Remove all hosts associated with this link */
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200707061641.l66GfpWE028959>