Re: [PATCH 1/1] xfs: online repair of symbolic links

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 29, 2024 at 09:16:32AM -0800, Darrick J. Wong wrote:
> We don't want to salvage in that case.  I forgot to finish that last
> paragraph:
> 
> "If salvage recovers more bytes than i_disk_size then we have no idea if
> di_size was broken or not because the target isn't null-terminated.  In
> theory the kernel will never do this (because it zeroes the xfs_buf
> contents in xfs_trans_buf_get) but fuzzers could do that.  Set the
> target to DUMMY_TARGET in this case."
> 
> and maybe add:
> 
> "The symlink target will be preserved if scrub does not find any errors
> in the symlink file, the number of bytes recovered matches i_disk_size,
> and there are no nulls in the recovered target.  In all other cases it
> is set to DUMMY_TARGET."

Sounds good.




[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux