[PATCH] ctdb-server: Introduce mem pool

Swen Schillig swen at vnet.ibm.com
Mon Mar 19 08:01:02 UTC 2018


On Mon, 2018-03-19 at 15:48 +1100, Martin Schwenke wrote:
> On Fri, 16 Mar 2018 11:55:54 +0100, Swen Schillig <swen at vnet.ibm.com>
> wrote:
> 
> > Hey Martin
> > On Fri, 2018-03-16 at 18:54 +1100, Martin Schwenke via samba-
> > technical
> > wrote:
> > > On Fri, 16 Mar 2018 08:27:14 +0100, Swen Schillig <swen at vnet.ibm.
> > > com>
> > > wrote:
> > >   
> > > > On Fri, 2018-03-16 at 16:35 +1100, Martin Schwenke via samba-
> > > > technical
> > > > wrote:  
> > 
> >  [...]  
> >  [...]  
> >  [...]  
> > > > 
> > > > you're right, I guess this got "fixed" in the last run.
> > > > 
> > > > But now, really fixed.  
> > > 
> > > EWRONGPATCH?  Or does this previous patch need fixing to get the
> > > "Introduce mem pool" one to work properly?  
> > 
> > Sorry, I'm totally out of my mind it seems, urgently need the
> > weekend.
> > 
> > Here it is.
> 
> OK, that's somewhat better.  However, it now leaks memory if rev_hdl
> ==
> NULL.  :-(
You're right and I guess it's not worth it to make it work with a
talloc_pool.

Thanks again Martin, you were a great help, as always.

Cheers Swen




More information about the samba-technical mailing list