Module Name: src Committed By: dholland Date: Tue Sep 1 06:08:37 UTC 2015
Modified Files: src/sbin/fsck_lfs: dir.c inode.c lfs.c pass6.c segwrite.c setup.c src/sys/ufs/lfs: lfs_accessors.h lfs_alloc.c lfs_balloc.c lfs_inode.c lfs_inode.h lfs_itimes.c lfs_rfw.c lfs_segment.c lfs_syscalls.c lfs_vfsops.c lfs_vnops.c ulfs_bmap.c ulfs_inode.h ulfs_vnops.c Log Message: Use the lfs dinode accessors in place of the ufs-derived ones. (Mostly.) The ufs-derived ones are fake structure member macros, which are gross and not very safe. Also, it seems that a lot of places in the lfs code were using the ffsv1 branch of them unconditionally, and this way it's guaranteed all those places have been updated. Found while doing this: for non-devices, have getattr produce NODEV in the rdev field instead of leaking the address of the first direct block. To generate a diff of this commit: cvs rdiff -u -r1.36 -r1.37 src/sbin/fsck_lfs/dir.c cvs rdiff -u -r1.60 -r1.61 src/sbin/fsck_lfs/inode.c cvs rdiff -u -r1.57 -r1.58 src/sbin/fsck_lfs/lfs.c cvs rdiff -u -r1.43 -r1.44 src/sbin/fsck_lfs/pass6.c cvs rdiff -u -r1.42 -r1.43 src/sbin/fsck_lfs/segwrite.c cvs rdiff -u -r1.55 -r1.56 src/sbin/fsck_lfs/setup.c cvs rdiff -u -r1.15 -r1.16 src/sys/ufs/lfs/lfs_accessors.h \ src/sys/ufs/lfs/ulfs_inode.h cvs rdiff -u -r1.128 -r1.129 src/sys/ufs/lfs/lfs_alloc.c cvs rdiff -u -r1.86 -r1.87 src/sys/ufs/lfs/lfs_balloc.c cvs rdiff -u -r1.145 -r1.146 src/sys/ufs/lfs/lfs_inode.c cvs rdiff -u -r1.10 -r1.11 src/sys/ufs/lfs/lfs_inode.h cvs rdiff -u -r1.18 -r1.19 src/sys/ufs/lfs/lfs_itimes.c cvs rdiff -u -r1.30 -r1.31 src/sys/ufs/lfs/lfs_rfw.c cvs rdiff -u -r1.258 -r1.259 src/sys/ufs/lfs/lfs_segment.c cvs rdiff -u -r1.169 -r1.170 src/sys/ufs/lfs/lfs_syscalls.c cvs rdiff -u -r1.341 -r1.342 src/sys/ufs/lfs/lfs_vfsops.c cvs rdiff -u -r1.287 -r1.288 src/sys/ufs/lfs/lfs_vnops.c cvs rdiff -u -r1.6 -r1.7 src/sys/ufs/lfs/ulfs_bmap.c cvs rdiff -u -r1.26 -r1.27 src/sys/ufs/lfs/ulfs_vnops.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.