Aren
/
NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

in spec_ioctl(), don't dereference v_specinfo if it's NULL. 

this is needed due to rev. 1.231 of kern/vfs_subr.c, which now sets
v_specinfo to NULL before changing the vnode's ops vector.
This commit is contained in:
chs 2005-09-11 14:18:54 +00:00
parent 355ecfe43b
commit 0840b7949f
1 changed files with 21 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
sys/miscfs/specfs/spec_vnops.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: spec_vnops.c,v 1.82 2005/08/30 20:08:01 xtraeme Exp $ */
/* $NetBSD: spec_vnops.c,v 1.83 2005/09/11 14:18:54 chs Exp $ */
/*
* Copyright (c) 1989, 1993
@ -32,7 +32,7 @@
*/
#include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: spec_vnops.c,v 1.82 2005/08/30 20:08:01 xtraeme Exp $");
__KERNEL_RCSID(0, "$NetBSD: spec_vnops.c,v 1.83 2005/09/11 14:18:54 chs Exp $");
#include <sys/param.h>
#include <sys/proc.h>
@ -461,9 +461,26 @@ spec_ioctl(v)
} */ *ap = v;
const struct bdevsw *bdev;
const struct cdevsw *cdev;
dev_t dev = ap->a_vp->v_rdev;
struct vnode *vp;
dev_t dev;
switch (ap->a_vp->v_type) {
/*
* Extract all the info we need from the vnode, taking care to
* avoid a race with VOP_REVOKE().
*/
vp = ap->a_vp;
dev = NODEV;
simple_lock(&vp->v_interlock);
if ((vp->v_flag & VXLOCK) == 0 && vp->v_specinfo) {
dev = vp->v_rdev;
}
simple_unlock(&vp->v_interlock);
if (dev == NODEV) {
return ENXIO;
}
switch (vp->v_type) {
case VCHR:
cdev = cdevsw_lookup(dev);