[PATCH] Re: Issues to fix before 4.6.1?

Oswald Buddenhagen ossi at kde.org
Wed Mar 30 11:16:54 UTC 2005


On Wed, Mar 30, 2005 at 10:11:19AM +0200, Jindrich Novy wrote:
>     if (!vfs_file_is_local (filename) ||
>         (fd = mc_open (filename, O_WRONLY | O_BINARY)) == -1)
> 
> what actually truncates the filename to zero size.
>
it does not, unless mc_open has totally misleading semantics - there is
no O_TRUNCATE. the truncation comes during the actual write ... which
leaves us with a truncated/empty file, if we run out-of-space in the
middle.
the "solution" would be not to use O_TRUNCATE at all. instead, first
write the part that extends beyond the end of the old file. if that
fails to complete, truncate to the old length. if we crash now, the old
file will have garbage appended, but at least it's not clobbered
alltogether. note that we cannot just seek to the new end and write one
byte because of sparse files. after the "extension" was written,
overwrite the start of the file with the new content and possibly
truncate the file to the new length. this is all sooo ugly ...

-- 
Hi! I'm a .signature virus! Copy me into your ~/.signature, please!
--
Chaos, panic, and disorder - my work here is done.



More information about the mc-devel mailing list