[SSSD] Unexpected behavior with 'simple_allow_users ='

Stef Walter stefw at redhat.com
Tue Oct 16 13:45:59 UTC 2012


On 10/16/2012 02:04 PM, Jakub Hrozek wrote:
> I was wondering for a while whether to change the behaviour directly in
> confdb_get_string_as_list() but I think the attached patch takes a better
> approach because the other consumers of confdb_get_string_as_list() do
> not see any difference between empty and missing parameter.

Yeah figures.

> The patch works as advertized, there is just one compilation warning:
>
> src/providers/simple/simple_access.c: In function
> 'get_conf_list_or_empty':
> src/providers/simple/simple_access.c:284:9: warning: unused variable 'r'
> [-Wunused-variable]

New patch attached.

Thanks for the review.

Stef

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Recognize-empty-string-lists-in-the-simple-access-pr.patch
Type: text/x-patch
Size: 4872 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20121016/aeac5f74/attachment.bin>


More information about the sssd-devel mailing list