[PATCH] winbindd: control number of winbindd's client connections
jra at samba.org
Fri Jun 5 13:23:16 MDT 2015
On Thu, Jun 04, 2015 at 10:54:44PM +0300, Uri Simchoni wrote:
> Here's V2 with followiing changes:
> v2 1/6 - int->unsigned
> v2 6/6 (new) - periodically (every 5 seconds) check for idle clients
> and close them.
Sorry, reviewing again I spotted a change I'd like
to make. There's no need to expose struct winbindd_listen_state
outside of source3/winbindd/winbindd.c, as all you really
need for winbindd_is_accepting()/winbindd_pause_accepting()
and winbindd_resume_accepting() are the struct tevent_fd
pointers for the privileged and non-privileged listeners.
Given that we can make these functions and behaviors
static inside source3/winbindd/winbindd.c. This shouldn't
be a behavior change, but a matter of taste (I don't
like exposing across modules if I can help it, even
though it looks like winbindd_is_accepting etc. belong
in winbindd_util, as the deal with data only inside
winbindd.c I don't think they do).
Here is a modified patchset that does this. Can you
confirm it's still good for you ?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 16402 bytes
Desc: not available
More information about the samba-technical