On Wed, Nov 29, 2023 at 08:49:36PM -0800, Christoph Hellwig wrote: > On Fri, Nov 24, 2023 at 03:52:54PM -0800, Darrick J. Wong wrote: > > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > > > Building off the rmap scanner that we added in the previous patch, we > > can now find block 0 and try to use the information contained inside of > > it to guess the mode of an inode if it's totally improper. > > Maybe I'm missing something important, but I don't see why a normal > user couldn't construct a file that looks like an XFS directory, and > that's a perfectly fine thing to do? They could very well do that, and it might confuse the scanner. However, I'd like to draw your attention to xrep_dinode_mode, which will set the user/group to root with 0000 access mode. That at least will keep unprivileged users from seeing the potentially weird file until the higher level repair functions can deal with it (or the sysadmin deletes it). Hmm. That code really ought to zap the attr fork because there could be ACLs attached to the file. Let me go do that. --D
