svn commit: r258124 - head/usr.sbin/portsnap/portsnap

Colin Percival cperciva at freebsd.org
Thu Nov 14 09:51:23 UTC 2013


On 11/14/13 01:47, Robert Watson wrote:
> On Thu, 14 Nov 2013, Colin Percival wrote:
>>  When verifying that files we need are present, also check that they don't
>>  have zero length.  Filesystem corruption will tend to truncate files, and
>>  since these are short that's likely to result in them becoming empty.
> 
> I think I'd probably express this slightly differently: the filesystem isn't
> corrupted, it just doesn't contain the data you expect.  As filesystems don't
> guarantee data is there until fsync() has returned, it is actually meeting all
> its promises -- e.g., vs corrupted inode contents due to a filesystem bug.

True, I should have written "crashes resulting in data loss".

-- 
Colin Percival
Security Officer Emeritus, FreeBSD | The power to serve
Founder, Tarsnap | www.tarsnap.com | Online backups for the truly paranoid



More information about the svn-src-head mailing list