On Mon, Feb 20, 2012 at 05:25:06PM +0100, Jan Zelený wrote:
> > On Fri, Feb 10, 2012 at 02:21:01PM +0100, Jan Zelený wrote:
> > > > This will make confdb coherent with changes made in sysdb. Not
> > > > using context hierarchy will help detecting memory leaks if there
> > > > are some.
> > >
> > > Self nack on the first version, I skipped one call of
> > > confdb_get_int() in ssh module. New patch attached.
> > >
> > > Jan
> >
> > nack, you still need to free tmp_ctx even if you free its contents.
>
> Thanks,
> Fixed patch in attachment.
>
> Jan
Sorry, but still a nack. You're only freeing tmp_ctx in the failed:
label, you need to free it in both success and failure cases.
This is what I get when I send patches in haste, sorry for the inconvenience.
Hopefully this will be the last patch in the row :)
Jan