[SSSD] [PATCH] added a ASQ search API for sysdb

Simo Sorce ssorce at redhat.com
Tue Oct 27 14:27:40 UTC 2009


On Mon, 2009-10-26 at 17:43 +0100, Sumit Bose wrote:
> Hi,
> 
> this patch adds a sysdb interface for ASQ (attribute scoped query)
> searches. These are useful to limit searches to objects listed in
> member/memberof attributes (this is not limited to member/memberof you
> can take any attribute you want).

The sysdb part looks good, so I'd say: ACK

The tests should work fine although I have a couple of remarks on the
style, and what is tested.

The tests use a synchronous style, so in this case the _done() function
should just get you out of the loop.
The _recv() function should be called after the test_loop() returns.
We are being synchronous here so no need to suffer the pain of jumping
through functions to see the program flow.

Also the test itself is not a request, so the request structure in this
case should be named "req" and not "subreq".

You are testing the return of only one result, it may be worth testing
with something that will return at least 2 results to exercise the
realloc machinery and the checks with more than one result.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York




More information about the sssd-devel mailing list