On Fri, Dec 13, 2013 at 08:57:48AM +0100, Sumit Bose wrote:
On Thu, Dec 12, 2013 at 09:57:38PM +0100, Jakub Hrozek wrote:
> On Wed, Dec 11, 2013 at 03:12:13PM +0100, Sumit Bose wrote:
> > Hi,
> >
> > this patch should fix the issues related to group-memberships in the AD
> > provider. The aim was to fix
https://fedorahosted.org/sssd/ticket/2161
> > but since now the group lookups are done against LDAP
> >
https://fedorahosted.org/sssd/ticket/2148 should be fixed as well.
> >
> > bye,
> > Sumit
>
> Hi,
>
> thank you, these patches work nicely. I think I can ACK them now, I'll
> just run them through Coverity to make sure there are no new warnings.
+1 for Coverity
No new warnings, pushed to master and sssd-1-11
>
> I only wonder if there is any looming code duplication between this
> module and the helpers in ad_common.c? It's not urgent to solve now,
> though.
ad_subdom_ad_ctx_new() is "inspired" by ipa_ad_ctx_new(). But since this
patch is aiming for a minor release I didn't want to break
ipa_ad_ctx_new() by refactoring it. But yes, there is a duplication.
That's fine, I think it would make sense to refactor both post-1.12 with
the rest of the LDAP provider.