From 85e7c591e00bfdb6254396041e58e33886d37414 Mon Sep 17 00:00:00 2001 From: Tai-hwa Liang Date: Fri, 9 Feb 2007 17:22:10 +0000 Subject: [PATCH] Backing out the wrong fix which could possibly trash the memory if devfs tries to drop the reference count after our close routine returns. A more correct fix is to defer the destroy_dev() to a taskqueue(either in devfs or locally). Reminded by: jhb --- sys/netsmb/smb_dev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sys/netsmb/smb_dev.c b/sys/netsmb/smb_dev.c index deef42650715..d3ed95a8ae4a 100644 --- a/sys/netsmb/smb_dev.c +++ b/sys/netsmb/smb_dev.c @@ -175,7 +175,6 @@ nsmb_dev_close(struct cdev *dev, int flag, int fmt, struct thread *td) */ dev->si_drv1 = NULL; free(sdp, M_NSMBDEV); - dev_relthread(dev); /* XXX dealing with si_threadcount */ destroy_dev(dev); splx(s); return 0;