Sorry for not replying to the previsious revisions, but I've been out
for on vacation.
I can't say I like this version. Now we've got callouts at two rather close
levels which is not very nice from the interface POV.
Maybe preference is for the first scheme where we simply move interpreation
of the ATTR_KILL_SUID/ATTR_KILL_SGID into the setattr routine and provide
a nice helper for the normal filesystem to use.
If people are really concerned about adding two lines of code to the
handfull of setattr operation there's a variant of this scheme that can
avoid it:
- notify_change is modified to not clear the ATTR_KILL_SUID/ATTR_KILL_SGID
but update ia_mode and the ia_valid flag to include ATTR_MODE.
- disk filesystems stay unchanged and never look at
ATTR_KILL_SUID/ATTR_KILL_SGID, but nfs can check for it and ignore
the ATTR_MODE flags and ia_valid in this case and do the right thing
on the server side.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
