diff options
author | Pavel Reichl <preichl@redhat.com> | 2014-01-28 17:19:07 +0000 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-06-03 12:57:11 +0200 |
commit | 4deef23b3a0e7da0c1ef4f45f792736c056c3123 (patch) | |
tree | 4e9bb3be36511270fcccf840b99d18fc80bd0c6a | |
parent | 99736adbc8da137e3ea061436e4d8baf3a015877 (diff) | |
download | sssd-4deef23b3a0e7da0c1ef4f45f792736c056c3123.tar.gz sssd-4deef23b3a0e7da0c1ef4f45f792736c056c3123.tar.xz sssd-4deef23b3a0e7da0c1ef4f45f792736c056c3123.zip |
refactor calls of sss_parse_name
sss_parse_name now supports NULL as output parameters so existing calls passing
arguments which were never read were substituted by NULL.
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
-rw-r--r-- | src/providers/krb5/krb5_common.c | 3 | ||||
-rw-r--r-- | src/providers/ldap/sdap_access.c | 3 | ||||
-rw-r--r-- | src/responder/nss/nsssrv_cmd.c | 3 | ||||
-rw-r--r-- | src/util/domain_info_utils.c | 3 |
4 files changed, 4 insertions, 8 deletions
diff --git a/src/providers/krb5/krb5_common.c b/src/providers/krb5/krb5_common.c index 48cb69080..4eafe366a 100644 --- a/src/providers/krb5/krb5_common.c +++ b/src/providers/krb5/krb5_common.c @@ -1005,7 +1005,6 @@ errno_t krb5_get_simple_upn(TALLOC_CTX *mem_ctx, struct krb5_ctx *krb5_ctx, char *uc_dom = NULL; char *upn; char *name; - char *domname; TALLOC_CTX *tmp_ctx = NULL; errno_t ret; @@ -1035,7 +1034,7 @@ errno_t krb5_get_simple_upn(TALLOC_CTX *mem_ctx, struct krb5_ctx *krb5_ctx, /* Subdomains already have a fully qualified name, which contains * the domain name. We need to replace it with the realm name */ - ret = sss_parse_name(tmp_ctx, dom->names, username, &domname, &name); + ret = sss_parse_name(tmp_ctx, dom->names, username, NULL, &name); if (ret != EOK) { DEBUG(SSSDBG_OP_FAILURE, "Could not parse [%s] into name and " diff --git a/src/providers/ldap/sdap_access.c b/src/providers/ldap/sdap_access.c index db0e878d8..4e1709028 100644 --- a/src/providers/ldap/sdap_access.c +++ b/src/providers/ldap/sdap_access.c @@ -666,7 +666,6 @@ static struct tevent_req *sdap_access_filter_send(TALLOC_CTX *mem_ctx, char *clean_username; errno_t ret = ERR_INTERNAL; char *name; - char *domname; req = tevent_req_create(mem_ctx, &state, struct sdap_access_filter_req_ctx); if (req == NULL) { @@ -720,7 +719,7 @@ static struct tevent_req *sdap_access_filter_send(TALLOC_CTX *mem_ctx, /* Construct the filter */ /* Subdomain users are identified by FQDN. We need to use just the username */ - ret = sss_parse_name(state, domain->names, username, &domname, &name); + ret = sss_parse_name(state, domain->names, username, NULL, &name); if (ret != EOK) { DEBUG(SSSDBG_OP_FAILURE, "Could not parse [%s] into name and " diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c index d273d8040..59f2c5779 100644 --- a/src/responder/nss/nsssrv_cmd.c +++ b/src/responder/nss/nsssrv_cmd.c @@ -177,13 +177,12 @@ static const char *get_homedir_override(TALLOC_CTX *mem_ctx, { const char *homedir; char *name; - char *domname; errno_t ret; homedir = ldb_msg_find_attr_as_string(msg, SYSDB_HOMEDIR, NULL); /* Subdomain users store FQDN in their name attribute */ - ret = sss_parse_name(mem_ctx, dom->names, orig_name, &domname, &name); + ret = sss_parse_name(mem_ctx, dom->names, orig_name, NULL, &name); if (ret != EOK) { DEBUG(SSSDBG_MINOR_FAILURE, "Could not parse [%s] into " "name-value components.\n", orig_name); diff --git a/src/util/domain_info_utils.c b/src/util/domain_info_utils.c index 472011c73..bd45a11cd 100644 --- a/src/util/domain_info_utils.c +++ b/src/util/domain_info_utils.c @@ -156,7 +156,6 @@ find_subdomain_by_object_name(struct sss_domain_info *domain, TALLOC_CTX *tmp_ctx; struct sss_domain_info *dom = NULL; char *domainname = NULL; - char *name = NULL; errno_t ret; tmp_ctx = talloc_new(NULL); @@ -166,7 +165,7 @@ find_subdomain_by_object_name(struct sss_domain_info *domain, } ret = sss_parse_name(tmp_ctx, domain->names, object_name, - &domainname, &name); + &domainname, NULL); if (ret != EOK) { DEBUG(SSSDBG_CRIT_FAILURE, "Unable to parse name '%s' [%d]: %s\n", object_name, ret, sss_strerror(ret)); |