<html><body><div style="color:#000; background-color:#fff; font-family:times new roman, new york, times, serif;font-size:12pt"><div><span>Thanks, we will test the 1.5 patches and report back if we run into any issues.<br></span></div><div><br></div>  <div style="font-family: times new roman, new york, times, serif; font-size: 12pt;"> <div style="font-family: times new roman, new york, times, serif; font-size: 12pt;"> <div dir="ltr"> <font face="Arial" size="2"> <hr size="1">  <b><span style="font-weight:bold;">From:</span></b> Stephen Gallagher &lt;sgallagh@redhat.com&gt;<br> <b><span style="font-weight: bold;">To:</span></b> Development of the System Security Services Daemon &lt;sssd-devel@lists.fedorahosted.org&gt; <br><b><span style="font-weight: bold;">Cc:</span></b> Shantanu Goel &lt;sgoel01@yahoo.com&gt; <br> <b><span style="font-weight: bold;">Sent:</span></b> Monday, June 18, 2012 2:43 PM<br> <b><span style="font-weight:
 bold;">Subject:</span></b> Re: [SSSD] [PATCH] Add support for terminating idle connections in sssd_nss<br> </font> </div> <br>
On Mon, 2012-06-18 at 14:11 -0400, Simo Sorce wrote:<br>&gt; On Mon, 2012-06-18 at 13:49 -0400, Stephen Gallagher wrote:<br>&gt; &gt; On Mon, 2012-06-18 at 13:32 -0400, Simo Sorce wrote:<br>&gt; &gt; &gt; On Mon, 2012-06-18 at 11:33 -0400, Stephen Gallagher wrote:<br>&gt; &gt; &gt; &gt; On Mon, 2012-06-18 at 09:33 -0400, Stephen Gallagher wrote:<br>&gt; &gt; &gt; &gt; &gt; On Mon, 2012-06-18 at 06:30 -0700, Shantanu Goel wrote:<br>&gt; &gt; &gt; &gt; &gt; &gt; Hi Stephen,<br>&gt; &gt; &gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; &gt; &gt; Please feel free to modify the patch in any way or shape you deem<br>&gt; &gt; &gt; &gt; &gt; &gt; necessary for inclusion.&nbsp; We are just glad that you agree there is a<br>&gt; &gt; &gt; &gt; &gt; &gt; real problem which needs fixing.&nbsp; One thing I ask is if you expect to<br>&gt; &gt; &gt; &gt; &gt; &gt; have rhel 5 or 6 test RPMs that we could test with the ultimate fix<br>&gt;
 &gt; &gt; &gt; &gt; &gt; any time soon, please drop me a note and we will gladly install them<br>&gt; &gt; &gt; &gt; &gt; &gt; on some of our problematic machines here to see if they address the<br>&gt; &gt; &gt; &gt; &gt; &gt; problems we have seen.<br>&gt; &gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; &gt; Sure, once this is done I'm going to be committing it upstream for the<br>&gt; &gt; &gt; &gt; &gt; master branch (future 1.9), the sssd-1-8 branch (our current LTM<br>&gt; &gt; &gt; &gt; &gt; release) and the sssd-1-5 branch (our previous LTM release).<br>&gt; &gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; &gt; You should be able to pull the patches from the sssd-1-5 branch and<br>&gt; &gt; &gt; &gt; &gt; build them for your systems once they're ready.<br>&gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; Ok, new patches attached. Shantanu, these are currently designed for the<br>&gt; &gt; &gt; &gt; master branch. We'll get them committed there first and tested
 out for a<br>&gt; &gt; &gt; &gt; little while, then we'll backport them.<br>&gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; Patch 0001: Return the correct errno value. Previously it could have<br>&gt; &gt; &gt; &gt; been reset by closing the socket.<br>&gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; Patch 0002: Add some additional debugging to the client_destructor()<br>&gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; Patch 0003: On systems that support MSG_NOSIGNAL, we should use it. This<br>&gt; &gt; &gt; &gt; way, if a client app isn't configured to listen for SIGPIPE, it will not<br>&gt; &gt; &gt; &gt; crash.<br>&gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; Patch 0004: Add a timer to each client context. If sixty seconds pass<br>&gt; &gt; &gt; &gt; (configurable in the patch 0005) without either read or write activity,<br>&gt; &gt; &gt; &gt; we will free the client context and close the socket. The client code is<br>&gt; &gt; &gt; &gt; already written to be
 tolerant of this and will reconnect on the next<br>&gt; &gt; &gt; &gt; request. This will help us avoid resource exhaustion if we have clients<br>&gt; &gt; &gt; &gt; that hang on to NSS and PAM file descriptors indefinitely (like 'su' and<br>&gt; &gt; &gt; &gt; 'login' do for PAM).<br>&gt; &gt; &gt; &gt; <br>&gt; &gt; &gt; &gt; Patch 0005: Make the client idle timeout value configurable and add it<br>&gt; &gt; &gt; &gt; to the manpages and config API.<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; 0001 ack<br>&gt; &gt; &gt; 0002 ack<br>&gt; &gt; &gt; 0003 Please always use send with a default set of flags, make the ifdef<br>&gt; &gt; &gt; set the default set of flags (0 vs MSG_NOSIGNAL)<br>&gt; &gt; &gt; 0004 ack (not like much the TODO but I was told 4/5 got split for<br>&gt; &gt; &gt; reviewability, so ok)<br>&gt; &gt; &gt; 0005 ack<br>&gt; &gt; <br>&gt; &gt; <br>&gt; &gt; <br>&gt; &gt; Thanks for the review. New patches attached.<br>&gt; <br>&gt; ack to
 all.<br><br><br>Pushed to master. Also backported to sssd-1-8 and sssd-1-5 and pushed<br>there as well. See attachments for the exact backported patches.<br><br><br> </div> </div>  </div></body></html>