On Thu, 13 May 2010 10:56:35 -0400
Stephen Gallagher <sgallagh(a)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