summaryrefslogtreecommitdiffstats
path: root/sys/ufs
diff options
context:
space:
mode:
authormaxim <maxim@FreeBSD.org>2006-05-31 15:55:52 +0000
committermaxim <maxim@FreeBSD.org>2006-05-31 15:55:52 +0000
commitdd4344c347b8e5972351c21a5ef9ea9e776bd548 (patch)
tree7f4044c730431f952bf7ed9c329e8ae522ddc3a4 /sys/ufs
parentdab340c4af8f8159e4efbcffffdf2597b8cadbd8 (diff)
downloadFreeBSD-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.c11
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);
OpenPOWER on IntegriCloud