tevent_abort_nesting crash in idmap_ad

[Jeremy Allison]

On Sat, Jun 25, 2016 at 06:07:34PM +0200, Ralph Boehme wrote:
> On Sat, Jun 25, 2016 at 05:42:04PM +0200, Volker Lendecke wrote:
> > On Sat, Jun 25, 2016 at 08:16:57AM -0700, Jeremy Allison wrote:
> > > Oh I agree. I don't think offering such an async api
> > > was a great idea when under the covers it can never
> > > be so. But it's easily avoided here, so let's just
> > > do that. No large code reverts needed.
> > 
> > Can you please explain what exactly is avoided here? If we still offer
> > _send and _recv calls for bind in tldap, that's a completely pointless
> > thing that needs to be removed.
> 
> I think the point was to also remove the async bind interface from
> tldap.

Yes that's right. As the _send/_recv calls aren't used
externally we just make them static so they're called
only from the sync tldap_gensec_bind code.