diff options
author | alc <alc@FreeBSD.org> | 2003-04-06 19:26:30 +0000 |
---|---|---|
committer | alc <alc@FreeBSD.org> | 2003-04-06 19:26:30 +0000 |
commit | ac3de07d6fbff07ee8e8aa0bb1f3d45bb86aeca7 (patch) | |
tree | 10a02943ba51af175d86174396407b2d38cfd024 /sys/ufs | |
parent | 8f3a17f085651acc9f2ef8a9a145fbab19443928 (diff) | |
download | FreeBSD-src-ac3de07d6fbff07ee8e8aa0bb1f3d45bb86aeca7.zip FreeBSD-src-ac3de07d6fbff07ee8e8aa0bb1f3d45bb86aeca7.tar.gz |
Sufficient access checks are performed by vmapbuf() that calling useracc()
is pointless. Remove the call to useracc().
Don't reinitialize fields that are already initialized by getpbuf().
Reviewed by: tegge
Diffstat (limited to 'sys/ufs')
-rw-r--r-- | sys/ufs/ffs/ffs_rawread.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/sys/ufs/ffs/ffs_rawread.c b/sys/ufs/ffs/ffs_rawread.c index ccf80ae..3b89129 100644 --- a/sys/ufs/ffs/ffs_rawread.c +++ b/sys/ufs/ffs/ffs_rawread.c @@ -210,9 +210,6 @@ ffs_rawread_readahead(struct vnode *vp, } bp->b_lblkno = bp->b_blkno = blockno; - if (!useracc(bp->b_data, bp->b_bcount, VM_PROT_WRITE)) { - return EFAULT; - } error = VOP_BMAP(vp, bp->b_lblkno, &dp, &bp->b_blkno, &bforwards, NULL); @@ -298,7 +295,6 @@ ffs_rawread_main(struct vnode *vp, bp = getpbuf(&ffsrawbufcnt); sa = bp->b_data; bp->b_vp = vp; - bp->b_error = 0; error = ffs_rawread_readahead(vp, udata, offset, resid, td, bp, sa); if (error != 0) @@ -313,7 +309,6 @@ ffs_rawread_main(struct vnode *vp, if (nbp != NULL) { nsa = nbp->b_data; nbp->b_vp = vp; - nbp->b_error = 0; nerror = ffs_rawread_readahead(vp, udata + |