[SSSD] [PATCH] Read and validate configuration before daemonizing
Simo Sorce
ssorce at redhat.com
Thu Sep 10 21:31:54 UTC 2009
On Thu, 2009-09-10 at 16:58 -0400, Stephen Gallagher wrote:
> On 09/10/2009 04:42 PM, Simo Sorce wrote:
> > On Thu, 2009-09-10 at 16:18 -0400, Stephen Gallagher wrote:
> >>> mem_ctx is not used anymore in this function as far as I can see,
> >>> please remove it.
> >>
> >> It is still used for sysdb_init, so I left it there.
> >
> > That sysdb_init() is used only to make sure the db is properly
> > initialized but the results are then discarded, we should probably
> > change the code to create a tmp_ctx right before running
> sysdb_init()
> > and then talloc_zfree(tmp_ctx) instead of the db_list.
> > This would also cause the mem_ctx to not be necessary anymore.
> >
> > Simo.
> >
>
> Created a tmp_ctx around the sysdb_init and removed the mem_ctx from
> the
> function. Also noticed a place where I forgot to remove a
> talloc_free(ctx).
ack and pushed
Simo.
--
Simo Sorce * Red Hat, Inc * New York
More information about the sssd-devel
mailing list