D22105: WIP : Fix SFTP Plugin of KIO for Windows
David Faure
noreply at phabricator.kde.org
Sat Jul 20 12:24:10 BST 2019
dfaure requested changes to this revision.
dfaure added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> kio_sftp.cpp:1946
> // check if destination is ok ...
> QT_STATBUF buff;
> + QFileInfo copyFile(sCopyFile);
Please remove this variable, since you're not calling QT_STAT anymore.
You'll notice that you're still using buff.st_size, which should become QFileInfo's size() method instead...
> kio_sftp.cpp:1973
> }
> bResume = canResume( buff.st_size );
> }
Red alert! Red alert! Uninitialized data being used!
> kio_sftp.cpp:2067
> // check if source is ok ...
> QT_STATBUF buff;
> + QFileInfo copyFile(sCopyFile);
Same here
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D22105
To: brute4s99, albertvaka, vonreth, sredman, sitter, dfaure
Cc: andriusr, kde-frameworks-devel, kfm-devel, aprcela, fprice, LeGast00n, sbergeron, fbampaloukas, alexde, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190720/32cb13e8/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list