[PATCH] Remove talloc_reference of event context in samba_dsdb stack and smb_krb5_init_context

Andreas Schneider asn at samba.org
Tue Apr 29 12:52:56 MDT 2014


On Sunday 27 April 2014 14:56:57 Jeremy Allison wrote:
> On Mon, Apr 28, 2014 at 09:52:16AM +1200, Andrew Bartlett wrote:
> > On Thu, 2014-04-17 at 22:52 +1200, Andrew Bartlett wrote:
> > > Given the Team's dislike for both nested event loops and
> > > talloc_reference, it seems fitting that my 'modest proposal' for
> > > limiting one gives the opportunity to find and remove the the latter.
> > > 
> > > This patch uses the fact that the dsdb modules do not need to talk over
> > > the network, so no event context need be specified or handled over to
> > > talloc_reference, based on the messages my test of the patch in
> > > https://lists.samba.org/archive/samba-technical/2014-April/099184.html
> > > 
> > > Then I follow up by removing the parameter entirely, as it just isn't
> > > needed.
> > > 
> > > Please review/push
> > 
> > Can I please have a review/push of this?
> 
> I was planning to +1 this on Monday,
> but I might not have chance (Bradley
> arrives and I have to host).
> 
> Feel free to add my 'Reviewed-by' and
> self-push if you're happy !
> 
> Jeremy.

You have my +1 too. I think Günther and I need this so we can remove 
smb_krb5_send_and_recv_func().


I will push this now.


	-- andreas


-- 
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             asn at samba.org
www.samba.org



More information about the samba-technical mailing list