From d20a56f2f05a011e62ba921e70124583e3c5b652 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Wed, 22 Jun 2016 07:22:59 +0200 Subject: RESPONDERS: Return the sysdb name from cache_req name.name is the input name. Since cache_req is an internal interface, we need to return the sysdb name instead. Reviewed-by: Sumit Bose --- src/responder/common/responder_cache_req.c | 2 +- src/tests/cmocka/test_responder_cache_req.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/responder/common/responder_cache_req.c b/src/responder/common/responder_cache_req.c index dd4d2870f..507d1c243 100644 --- a/src/responder/common/responder_cache_req.c +++ b/src/responder/common/responder_cache_req.c @@ -1478,7 +1478,7 @@ errno_t cache_req_recv(TALLOC_CTX *mem_ctx, if (state->cr->data->name.lookup == NULL) { *_name = NULL; } else { - name = talloc_strdup(mem_ctx, state->cr->data->name.name); + name = talloc_strdup(mem_ctx, state->cr->data->name.lookup); if (name == NULL) { return ENOMEM; } diff --git a/src/tests/cmocka/test_responder_cache_req.c b/src/tests/cmocka/test_responder_cache_req.c index bb1e382a3..eba7406ec 100644 --- a/src/tests/cmocka/test_responder_cache_req.c +++ b/src/tests/cmocka/test_responder_cache_req.c @@ -632,7 +632,7 @@ void test_user_by_name_multiple_domains_parse(void **state) check_user(test_ctx, &users[0], domain); assert_non_null(test_ctx->name); - assert_string_equal(users[0].short_name, test_ctx->name); + assert_string_equal(input_fqn, test_ctx->name); talloc_free(input_fqn); } @@ -1145,7 +1145,7 @@ void test_group_by_name_multiple_domains_parse(void **state) check_group(test_ctx, &groups[0], domain); assert_non_null(test_ctx->name); - assert_string_equal(groups[0].short_name, test_ctx->name); + assert_string_equal(input_fqn, test_ctx->name); talloc_free(input_fqn); } -- cgit