diff options
author | avatar <avatar@FreeBSD.org> | 2007-02-09 17:22:10 +0000 |
---|---|---|
committer | avatar <avatar@FreeBSD.org> | 2007-02-09 17:22:10 +0000 |
commit | c9f2d4f91b291c9495fba17e9955d40de997be3f (patch) | |
tree | dcd73d11d4e3ebf9c57ecf4821cd998daf371daa /sys/netsmb/smb_dev.c | |
parent | f64d74256b448bdacf27f60516d00ede1b9b6426 (diff) | |
download | FreeBSD-src-c9f2d4f91b291c9495fba17e9955d40de997be3f.zip FreeBSD-src-c9f2d4f91b291c9495fba17e9955d40de997be3f.tar.gz |
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
Diffstat (limited to 'sys/netsmb/smb_dev.c')
-rw-r--r-- | sys/netsmb/smb_dev.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sys/netsmb/smb_dev.c b/sys/netsmb/smb_dev.c index deef426..d3ed95a 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; |