[PATCHES] Rationalizing and unifing Schannel

[Volker Lendecke]

On Tue, Feb 23, 2010 at 08:33:26AM -0500, simo wrote:
> On Tue, 2010-02-23 at 09:39 +0100, Stefan (metze) Metzmacher wrote:
> > simo schrieb:
> > > I have been working on a patchset to enhance the schannel interface and
> > > also to make it possible to share this code between S3 and S4 again.
> > > 
> > > The first 6 patches shouldn't be controversial.
> > > The last 2 patches remove the use of ldb and goes back to use a tdb for
> > > samba4, the comment on patch 7 explains the rationale.
> > > 
> > > Unless there are objections I will push this patches during the week.
> > 
> > Hmm, I think we should think about the tdb_wrap_open() usage
> > a bit more. It uses talloc_reference() and the caller uses
> > talloc_free(). I think we should not explicit free it
> > we need a talloc parent that we free, as source3/lib/dbwrap_tdb.c
> > does.
> 
> What about creating a tmpctx in the caller, and then freeing only that ?

.... or rewriting tdb_wrap_open to not use talloc_reference?  :-)

Volker
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20100223/21e2061f/attachment.pgp>