D17632: Move/copy job: skip stat'ing sources if the destination dir isn't writable

Chinmoy Ranjan Pradhan noreply at phabricator.kde.org
Mon Dec 17 15:38:45 GMT 2018


chinmoyr added inline comments.

INLINE COMMENTS

> copyjob.cpp:419
> +            // TODO: this is done only for local files, but we could use the UDSEntry to do this portably
> +            // ... assuming all kioslaves set permissions correctly
> +            if (fileInfo.exists() && !fileInfo.isWritable()) {

For file permissions, 'file:' and 'trash:' use the result of QT_LSTAT, 'archive:' makes use of stat, 'sftp:' uses sftp_lstat, and smb sets UDS_ACCESS to 0500. These are the kioslaves(I found) which KDE ships and set UDS_ACCESS entry. I think using UDSEntry can be considered here.

REPOSITORY
  R241 KIO

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

To: dfaure, shubham, broulik, bruns
Cc: chinmoyr, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181217/5950cf63/attachment.html>


More information about the Kde-frameworks-devel mailing list