[SSSD] [PATCH] Add ldap_access_filter option

Simo Sorce ssorce at redhat.com
Thu May 13 16:07:18 UTC 2010


On Thu, 13 May 2010 10:56:35 -0400
Stephen Gallagher <sgallagh at redhat.com> wrote:

> +    ret = sysdb_attrs_add_uint32(attrs, SYSDB_LDAP_ACCESS,
> +                                 state->pam_status == PAM_SUCCESS ?
> +                                                      true :
> +                                                      false);

Just a nitpick, but normally in LDAP true false values are stored as
the strings "TRUE" or "FALSE" (upper case). It makes it clearer this is
a boolean attribute than storing 0 vs a random value (usually 1).

Simo.

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



More information about the sssd-devel mailing list