D12291: Accept file descriptor only from root owned process
Chinmoy Ranjan Pradhan
noreply at phabricator.kde.org
Sun May 27 13:13:25 UTC 2018
chinmoyr added inline comments.
INLINE COMMENTS
> ossi wrote in fdreceiver.cpp:89
> i wonder whether that shouldn't come with an unconditional acceptConnection = true; then - now the compilation succeeds, but it will fail to operate. given that the whole feature is redundant with moving the socket to a safe place (something you really should mention in the commit message), this seems like a rather pointless limitation.
I don't think making acceptConnection unconditionally true is a good idea. At least it will make this patch look horrible.
Since getsocketopt conforms to posix, how about we check for __ unix __ , and posix compliance instead of __ linux __ and upon failure inform the user to install a posix compliant libc?
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D12291
To: chinmoyr, #frameworks, dfaure, ossi
Cc: kde-frameworks-devel, ossi, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180527/e4eb2199/attachment.html>
More information about the Kde-frameworks-devel
mailing list