Review Request 129259: Fix the buffersize in certain situations.

[taro yamada]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129259/
-----------------------------------------------------------

(Updated Oct. 29, 2016, 7:14 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Changes
-------

Submitted with commit 44512b6acbe6bb1c595e330bef702a0100d7bd4b by Andreas Hartmetz on behalf of Taro Yamada to branch master.


Bugs: 369275
    https://bugs.kde.org/show_bug.cgi?id=369275


Repository: kio


Description
-------

Currently, KIO uses lstat to get the buffersize for readlink.
But in certain situations, it returns inappropriate value.

For example, "/proc/self" or "/sys/bus/cpu/devices/*" returns its size is 0 , and then readlink fails with EINVAL.(so link won't be shown in kde application.)
TMSU seems it returns its target's actual filesize insted of the link's filesize itself.

This patch changes the buffersize to 1024 bytes if it is 0.
And later truncate it to actual size.


Diffs
-----

  src/ioslaves/file/file.cpp 8b17d31 

Diff: https://git.reviewboard.kde.org/r/129259/diff/


Testing
-------


Thanks,

taro yamada

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161029/105a746a/attachment.html>