Hello, warning in XFS made me look into detail into how clearing of suid / sgid bits and security labels is done. And I've spotted a few issues: 1) file_remove_suid() is a misnomer since it also handles removing of security labels. It is even more confusing because should_remove_suid() doesn't return whether file_remove_suid() is needed or not. 2) on truncate we do clear suid bits but not security labels. I think that's a bug but please correct me if I'm wrong. 3) ocfs2 doesn't clear security labels - hard to fix, I left it alone for now. 4) XFS didn't provide proper exclusion for clearing mode bits. This series aims at fixing above issues. The second patch in the series is unrelated fix to inode_set_mask() which I spotted when playing with the code. Honza _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
