summaryrefslogtreecommitdiffstats
path: root/sys/kern/kern_intr.c
diff options
context:
space:
mode:
authorjhb <jhb@FreeBSD.org>2001-02-22 00:23:56 +0000
committerjhb <jhb@FreeBSD.org>2001-02-22 00:23:56 +0000
commit65856928608430b7102a8749b4af122de79587c3 (patch)
tree6612481e4193a995617e15e4bed238827d5afe09 /sys/kern/kern_intr.c
parent5ec338f4cce3f7a8557b9b82aeb924bb32680d4e (diff)
downloadFreeBSD-src-65856928608430b7102a8749b4af122de79587c3.zip
FreeBSD-src-65856928608430b7102a8749b4af122de79587c3.tar.gz
Fix a bug where the 'ithread' variable was being set in a KASSERT()
condition and thus was not initialized properly in the !INVARIANTS case. Noticed by: bde Pointy hat to: me
Diffstat (limited to 'sys/kern/kern_intr.c')
-rw-r--r--sys/kern/kern_intr.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/kern/kern_intr.c b/sys/kern/kern_intr.c
index 4463ac8..1be0564 100644
--- a/sys/kern/kern_intr.c
+++ b/sys/kern/kern_intr.c
@@ -279,8 +279,8 @@ ithread_remove_handler(void *cookie)
if (handler == NULL)
return (EINVAL);
-
- KASSERT((ithread = handler->ih_ithread) != NULL,
+ ithread = handler->ih_ithread;
+ KASSERT(ithread != NULL,
("interrupt handler \"%s\" has a NULL interrupt thread",
handler->ih_name));
mtx_lock_spin(&ithread_list_lock);
OpenPOWER on IntegriCloud