diff options
author | maxim <maxim@FreeBSD.org> | 2006-05-31 15:55:52 +0000 |
---|---|---|
committer | maxim <maxim@FreeBSD.org> | 2006-05-31 15:55:52 +0000 |
commit | dd4344c347b8e5972351c21a5ef9ea9e776bd548 (patch) | |
tree | 7f4044c730431f952bf7ed9c329e8ae522ddc3a4 /sys/ufs | |
parent | dab340c4af8f8159e4efbcffffdf2597b8cadbd8 (diff) | |
download | FreeBSD-src-dd4344c347b8e5972351c21a5ef9ea9e776bd548.zip FreeBSD-src-dd4344c347b8e5972351c21a5ef9ea9e776bd548.tar.gz |
o Rearrange and remove incorrect comments.
Requested by: bde
Diffstat (limited to 'sys/ufs')
-rw-r--r-- | sys/ufs/ufs/ufs_vnops.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/sys/ufs/ufs/ufs_vnops.c b/sys/ufs/ufs/ufs_vnops.c index ffd295c..63b6271 100644 --- a/sys/ufs/ufs/ufs_vnops.c +++ b/sys/ufs/ufs/ufs_vnops.c @@ -509,8 +509,9 @@ ufs_setattr(ap) } if (vap->va_size != VNOVAL) { /* - * XXX most of this checking should be in callers instead - * of in N filesystems. The VDIR check mostly already is. + * XXX most of the following special cases should be in + * callers instead of in N filesystems. The VDIR check + * mostly already is. */ switch (vp->v_type) { case VDIR: @@ -521,12 +522,6 @@ ufs_setattr(ap) * Truncation should have an effect in these cases. * Disallow it if the filesystem is read-only or * the file is being snapshotted. - * - * XXX unfortunately the snapshot check can't be - * more global since we want to check other things - * first so as to return better error codes. But - * we miss several cases (file flags and ownership - * changes at least) by not doing a central check. */ if (vp->v_mount->mnt_flag & MNT_RDONLY) return (EROFS); |