[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