On 23 Jul 2013, at 17:52, Phil Mayers <[email protected]> wrote:

> On 23/07/13 17:19, Franks Andy (RLZ) IT Systems Engineer wrote:
>> This will probably be obvious, but I can’t see it!
> 
> Looks like a bug - the code here:
> 
> https://github.com/FreeRADIUS/freeradius-server/blob/master/src/modules/rlm_ldap/groups.c#L495
> 
> ...passes NULL for the "result" argument to rlm_ldap_search, which means this 
> code:
> 
> https://github.com/FreeRADIUS/freeradius-server/blob/master/src/modules/rlm_ldap/ldap.c#L679
> 
> ...doesn't get run, so 0 results is "ok".

Nice catch. Fixed.

Arran Cudbard-Bell <[email protected]>
FreeRADIUS Development Team

-
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to