D27872: sftp: fix partial transfer resuming when copying to local

Harald Sitter noreply at phabricator.kde.org
Mon Mar 9 11:10:15 GMT 2020


sitter added inline comments.

INLINE COMMENTS

> bruns wrote in kio_sftp.cpp:1935
> Dependent on the file system, this will no longer catch `partFile.isDir()`. stat.st_size is only defined for regular files and symlinks, for all other types it is implementation defined. E.g. XFS and Btrfs return 0 for empty directories.

Your comment is dazzling. What you meant to say is that the size check ought to be after the dir check?

> bruns wrote in kio_sftp.cpp:1936
> should be `if (!partFile.isFile())` - we cant resume a pipe or socket ...

Please make a diff.

REPOSITORY
  R320 KIO Extras

REVISION DETAIL
  https://phabricator.kde.org/D27872

To: sitter, ngraham, feverfew, bruns
Cc: bruns, kde-frameworks-devel, kfm-devel, pberestov, iasensio, fprice, LeGast00n, cblack, MrPepe, fbampaloukas, alexde, GB_2, Codezela, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200309/d6e025b5/attachment.html>


More information about the Kde-frameworks-devel mailing list