<html>

 <body>

  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">

   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">

    <tr>

     <td>

      This is an automatically generated e-mail. To reply, visit:

      <a href="https://git.reviewboard.kde.org/r/119011/">https://git.reviewboard.kde.org/r/119011/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit ff5ea1ab8568893c7d7b3a4518997080d3533308 by Dan Vrátil to branch master.</pre>

 <br />

<p>- Commit Hook</p>

<br />

<p>On June 29th, 2014, 10:50 a.m. UTC, Dan Vrátil wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">

 <tr>

  <td>

<div>Review request for KDE Frameworks.</div>

<div>By Dan Vrátil.</div>

<p style="color: grey;"><i>Updated June 29, 2014, 10:50 a.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

kinit

</div>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>

 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">

 <tr>

  <td>

   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">While packaging kinit, we got a warning from rpmlint that start_kdeinit calls setgid() without calling setgroups() first. From rpmlint:

   This executable is calling setuid and setgid without setgroups or initgroups.

   There is a high probability this mean it didn't relinquish all groups, and

   this would be a potential security issue to be fixed. Seek POS36-C on the web

   for details about the problem.

The reasoning is that when you drop privileges from root to regular user, there might be some extra groups left that, if not cleared, might grant the process privileges to do superuser things.

The code does not check for return value, as the call will fail if we are not a superuser.

This oneliner makes rpmlint happy and maybe prevents a security issue.</pre>

  </td>

 </tr>

</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>

<ul style="margin-left: 3em; padding-left: 0;">

 <li>src/start_kdeinit/start_kdeinit.c <span style="color: grey">(07a28d3)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/119011/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>