ngraham added a comment. Restricted Application added a subscriber: kde-frameworks-devel.
You're right, the destination file was indeed silently corrupted in the "before" case. Yikes, that's bad. Really, both options are bad. I can see now that your patch doesn't actually regress anything--it just exposes a dormant problem that was previously hidden but becomes very visible with the patch. That's a good sign that we ought to resolve that issue first. Perhaps for overwrites, if there's enough space on the destination filesystem, we should copy to a temp file, then if that completes, swap the target file and the temp file, and then finally delete the old target file. What do you think? REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D10663 To: dmitrio, #frameworks, dfaure, ngraham Cc: kde-frameworks-devel, elvisangelaccio, ngraham, anthonyfieroni, meven, #frameworks, michaelh, bruns
