On Mon, Jan 30, 2012 at 09:31:14PM -0500, Stephen Gallagher wrote:
On Wed, 2012-01-25 at 18:12 +0100, Sumit Bose wrote:
> Hi,
>
> a few days ago I send a draft patch where the sysdb context is stored
> in the domain info struct. I created a patch which is a bot more
> conservative than the last one and included the comments by Simo (added
> a destructor and don't unconditionally add the context all the time).
>
> This patch will make the handling of sub-domains much easier, because
> now only the domain info struct for the sub-domains needs to be
> up-to-date.
Nack.
To save ourselves some cycles, if we go through the search in
sysdb_get_ctx_from_list(), it would be best to assign the sysdb context
to the domain once it's discovered. That said, I'm confused why we would
ever hit that code. I'm somewhat inclined to remove it and just always
return the domain->sysdb value and see if anything blows up and fix that
instead.
ok, this was my first idea as well, but then I thought it might be
better to be conservative here and look for the debug message from some
time to see if this path is ever taken.
Would you prefer to keep sysdb_get_ctx_from_list() and let it just
return domain->sysdb or shall I remove it completely and let the caller
just take domain->sysdb and check for NULL?
bye,
Sumit
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel