[Freeipa-devel] [PATCH] 1035 case sensitivity when calculating indirect members

Petr Viktorin pviktori at redhat.com
Thu Jul 19 11:45:53 UTC 2012


On 07/17/2012 08:52 PM, Rob Crittenden wrote:
> When determining whether a member is direct or indirect we were not
> doing a case-insensitive comparison which led to marking a member as
> both direct and indirect (in a test case no less).
>
> This patch fixes the comparison and the test.
>
> rob
>

When comparing DNs, you should use the DN class, not string 
manipulations: DN(x) instead of x.lower().

How urgent is this? John's DN patch solves this in a much more thorough 
way, maybe it'd be better to just wait for that.

-- 
Petr³





More information about the Freeipa-devel mailing list