diff options
| author | Sumit Bose <sbose@redhat.com> | 2012-12-19 18:37:32 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-12-20 20:06:17 +0100 |
| commit | 41147eeb43b50676ccccd1e7f5a9b20b8c19fdad (patch) | |
| tree | 9ea27b4d369f7a516c3a903bf6e8bb31a09aa5d8 /src/providers | |
| parent | adfe9a19e085f6575bb55f2a304903cf944d2202 (diff) | |
| download | sssd-41147eeb43b50676ccccd1e7f5a9b20b8c19fdad.tar.gz sssd-41147eeb43b50676ccccd1e7f5a9b20b8c19fdad.tar.xz sssd-41147eeb43b50676ccccd1e7f5a9b20b8c19fdad.zip | |
krb5 tgt renewal: fix usage of ldb_dn_get_component_val()
For some reason I was under the impression that the DN components are
counted backwards in libldb. This patch corrects this.
Diffstat (limited to 'src/providers')
| -rw-r--r-- | src/providers/krb5/krb5_renew_tgt.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/providers/krb5/krb5_renew_tgt.c b/src/providers/krb5/krb5_renew_tgt.c index ccb7e6af6..b5eee697f 100644 --- a/src/providers/krb5/krb5_renew_tgt.c +++ b/src/providers/krb5/krb5_renew_tgt.c @@ -423,11 +423,11 @@ static errno_t check_ccache_files(struct renew_tgt_ctx *renew_tgt_ctx) continue; } - /* The DNs of users in sysdb ends with ...,cn=domain.name,cn=sysdb, so - * the value of the component before the last (index 1) is the domain - * name. */ + /* The DNs of users in sysdb looks like + * name=username,cn=users,cn=domain.name,cn=sysdb + * the value of the third component (index 2) is the domain name. */ - user_dom_val = ldb_dn_get_component_val(msgs[c]->dn, 1); + user_dom_val = ldb_dn_get_component_val(msgs[c]->dn, 2); if (user_dom_val == NULL) { DEBUG(SSSDBG_OP_FAILURE, ("Invalid user DN [%s].\n", ldb_dn_get_linearized(msgs[c]->dn))); |