<table><tr><td style="">chinmoyr added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D14467">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D14467#inline-123599">View Inline</a><span style="color: #4b4d51; font-weight: bold;">maltek</span> wrote in <span style="color: #4b4d51; font-weight: bold;">filehelper.cpp:133</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">For <tt style="background: #ebebeb; font-size: 13px;">chown</tt>, dropping privileges here means that the <tt style="background: #ebebeb; font-size: 13px;">chown</tt> later can't succeed - it's not possible to 'gift' a file to another user. I think it should be handled more like <tt style="background: #ebebeb; font-size: 13px;">DEL/RMDIR/MKDIR</tt> etc.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Ah! Since I was testing inside /opt I didn't notice. I think the order here should be: drop privilege -> change grp -> gain privilege -> change user.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D14467#inline-123600">View Inline</a><span style="color: #4b4d51; font-weight: bold;">maltek</span> wrote in <span style="color: #4b4d51; font-weight: bold;">filehelper.cpp:150</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">I just realized that this wouldn't allow changing the owner of symbolic links. The way to go here is <tt style="background: #ebebeb; font-size: 13px;">lchown</tt>.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Do you think it'll be a bad idea to skip the case for symlinks in utime, chmod, chown, for now? Right now there's no code in KIO that requires these operations to be performed on the link itself.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14467">https://phabricator.kde.org/D14467</a></div></div><br /><div><strong>To: </strong>chinmoyr, dfaure, ngraham, elvisangelaccio, Frameworks, Dolphin, maltek<br /><strong>Cc: </strong>maltek, mreeves, mgerstner, fvogt, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns<br /></div>