summaryrefslogtreecommitdiffstats
path: root/src/providers/ldap/sdap_sudo_cache.c
diff options
context:
space:
mode:
authorJan Engelhardt <jengelh@inai.de>2013-02-21 13:12:25 +0100
committerJakub Hrozek <jhrozek@redhat.com>2013-02-26 17:18:04 +0100
commit6072f51a6c91f580c6582c527a08acbe51824d6a (patch)
treecc9fc0f47853cec6450717b8fba66017c95cb189 /src/providers/ldap/sdap_sudo_cache.c
parent8bbdf179644bd92ec044b597d62edd9c77453f81 (diff)
downloadsssd-6072f51a6c91f580c6582c527a08acbe51824d6a.tar.gz
sssd-6072f51a6c91f580c6582c527a08acbe51824d6a.tar.xz
sssd-6072f51a6c91f580c6582c527a08acbe51824d6a.zip
sysdb: try dealing with binary-content attributes
https://fedorahosted.org/sssd/ticket/1818 I have here a LDAP user entry which has this attribute loginAllowedTimeMap:: AAAAAAAAAP///38AAP///38AAP///38AAP///38AAP///38AAAAAAAAA In the function sysdb_attrs_add_string(), called from sdap_attrs_add_ldap_attr(), strlen() is called on this blob, which is the wrong thing to do. The result of strlen is then used to populate the .v_length member of a struct ldb_val - and this will set it to zero in this case. (There is also the problem that there may not be a '\0' at all in the blob.) Subsequently, .v_length being 0 makes ldb_modify(), called from sysdb_set_entry_attr(), return LDB_ERR_INVALID_ATTRIBUTE_SYNTAX. End result is that users do not get stored in the sysdb, and programs like `id` or `getent ...` show incomplete information. The bug was encountered with sssd-1.8.5. sssd-1.5.11 seemed to behave fine, but that may not mean that is the absolute lower boundary of introduction of the problem.
Diffstat (limited to 'src/providers/ldap/sdap_sudo_cache.c')
0 files changed, 0 insertions, 0 deletions