[SSSD] [PATCH] Converge accept_fd_handler and accept_priv_fd_handler

Stephen Gallagher sgallagh at redhat.com
Fri Jul 29 18:06:17 UTC 2011


On Fri, 2011-07-29 at 19:43 +0200, Jakub Hrozek wrote:
> On Fri, Jul 29, 2011 at 09:21:30AM -0400, Stephen Gallagher wrote:
> > On Fri, 2011-07-29 at 09:16 +0200, Jakub Hrozek wrote:
> > > On 07/28/2011 09:35 PM, Stephen Gallagher wrote:
> > > > These two functions were almost identical. Better to maintain them
> > > > as a single function.
> > > > 
> > > 
> > > In the old implementation, only accept_priv_fd_handler() used to set
> > > cctx->priv = 1, now the converged implementation sets it unconditionally
> > > for both cases, is that intentional?
> > 
> > Nope, that would have been a bug. New patch attached.
> > 
> 
> Ack

Pushed to master.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110729/efa7d492/attachment.sig>


More information about the sssd-devel mailing list