system/freebsd-src
system/freebsd-src
1
0
Fork 0
mirror of https://github.com/freebsd/freebsd-src synced 2024-09-22 17:54:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixed device number checking in bdevvp():

Browse source 
- dev != NODEV was checked for, but 0 was returned on failure.  This was
  fixed in Lite2 (except the return code was still slightly wrong (ENODEV
  instead of ENXIO)) but the changes were not merged.  This case probably
  doesn't actually occur under FreeBSD.
- major(dev) was not checked to have a valid non-NULL bdevsw entry.  This
  caused panics when the driver for the root device didn't exist.

Fixed minor misformattings in bdevvp().  Rev.1.14 consisted mainly of
gratuitous reformattings that seem to have caused many Lite2 merge
errors.

PR:			8417
This commit is contained in:
Bruce Evans 1998-10-25 16:11:49 +00:00
parent e5eb83d8c3
commit 37906c686d
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=40647
2 changed files with 20 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
sys/kern/vfs_export.c
View file

@ -36,7 +36,7 @@
* SUCH DAMAGE.
*
* @(#)vfs_subr.c 8.31 (Berkeley) 5/26/95
* $Id: vfs_subr.c,v 1.165 1998/10/13 08:24:41 dg Exp $
* $Id: vfs_subr.c,v 1.166 1998/10/14 15:05:52 dt Exp $
*/
/*
@ -46,6 +46,7 @@
#include <sys/param.h>
#include <sys/systm.h>
#include <sys/conf.h>
#include <sys/kernel.h>
#include <sys/proc.h>
#include <sys/malloc.h>
@ -1126,16 +1127,19 @@ bdevvp(dev, vpp)
struct vnode *nvp;
int error;
if (dev == NODEV)
return (0);
error = getnewvnode(VT_NON, (struct mount *) 0, spec_vnodeop_p, &nvp);
if (dev == NODEV || major(dev) >= nblkdev ||
bdevsw[major(dev)] == NULL) {
*vpp = NULLVP;
return (ENXIO);
}
error = getnewvnode(VT_NON, (struct mount *)0, spec_vnodeop_p, &nvp);
if (error) {
*vpp = 0;
*vpp = NULLVP;
return (error);
}
vp = nvp;
vp->v_type = VBLK;
if ((nvp = checkalias(vp, dev, (struct mount *) 0))) {
if ((nvp = checkalias(vp, dev, (struct mount *)0)) != NULL) {
vput(vp);
vp = nvp;
}

16
sys/kern/vfs_subr.c
View file

@ -36,7 +36,7 @@
* SUCH DAMAGE.
*
* @(#)vfs_subr.c 8.31 (Berkeley) 5/26/95
* $Id: vfs_subr.c,v 1.165 1998/10/13 08:24:41 dg Exp $
* $Id: vfs_subr.c,v 1.166 1998/10/14 15:05:52 dt Exp $
*/
/*
@ -46,6 +46,7 @@
#include <sys/param.h>
#include <sys/systm.h>
#include <sys/conf.h>
#include <sys/kernel.h>
#include <sys/proc.h>
#include <sys/malloc.h>
@ -1126,16 +1127,19 @@ bdevvp(dev, vpp)
struct vnode *nvp;
int error;
if (dev == NODEV)
return (0);
error = getnewvnode(VT_NON, (struct mount *) 0, spec_vnodeop_p, &nvp);
if (dev == NODEV || major(dev) >= nblkdev ||
bdevsw[major(dev)] == NULL) {
*vpp = NULLVP;
return (ENXIO);
}
error = getnewvnode(VT_NON, (struct mount *)0, spec_vnodeop_p, &nvp);
if (error) {
*vpp = 0;
*vpp = NULLVP;
return (error);
}
vp = nvp;
vp->v_type = VBLK;
if ((nvp = checkalias(vp, dev, (struct mount *) 0))) {
if ((nvp = checkalias(vp, dev, (struct mount *)0)) != NULL) {
vput(vp);
vp = nvp;
}