From ed3bdd8998c5e0e18f9b8cfefa9acd00b8531585 Mon Sep 17 00:00:00 2001

From: Jakub Hrozek <jhrozek@redhat.com>

Date: Wed, 19 Apr 2017 17:44:40 +0200

Subject: [PATCH 164/165] Move sized_output_name() and sized_domain_name() into

 responder common code

MIME-Version: 1.0

Content-Type: text/plain; charset=UTF-8

Content-Transfer-Encoding: 8bit

These functions are used to format a name into a format that the user

configured for output, including case sensitiveness, replacing

whitespace and qualified format. They were used only in the NSS

responder, which typically returns strings to the NSS client library and

then the user.

But it makes sense to just reuse the same code in the IFP responder as

well, since it does essentially the same job.

The patch also renames sized_member_name to sized_domain_name.

Previously, the function was only used to format a group member, the IFP

responder would use the same function to format a group the user is a

member of.

Related to:

    https://pagure.io/SSSD/sssd/issue/3268

Reviewed-by: Pavel Březina <pbrezina@redhat.com>

(cherry picked from commit 7c074ba2f923985ab0d4f9d6a5e01ff3f2f0a7a8)

---

 src/responder/common/responder.h        | 21 ++++++++

 src/responder/common/responder_common.c | 90 +++++++++++++++++++++++++++++++++

 src/responder/nss/nsssrv_cmd.c          | 89 +-------------------------------

 3 files changed, 112 insertions(+), 88 deletions(-)

diff --git a/src/responder/common/responder.h b/src/responder/common/responder.h

index 9e3b2fdbda4e30b859df597374fc7d490b1720e5..fd6a67ba72f28f52d6cc1bbad16e1a7245462c93 100644

--- a/src/responder/common/responder.h

+++ b/src/responder/common/responder.h

@@ -358,4 +358,25 @@ char *sss_resp_create_fqname(TALLOC_CTX *mem_ctx,

                              bool name_is_upn,

                              const char *orig_name);

+/**

+ * Helper functions to format output names

+ */

+

+/* Format orig_name into a sized_string in output format as prescribed

+ * by the name_dom domain

+ */

+int sized_output_name(TALLOC_CTX *mem_ctx,

+                      struct resp_ctx *rctx,

+                      const char *orig_name,

+                      struct sss_domain_info *name_dom,

+                      struct sized_string **_name);

+

+/* Format orig_name into a sized_string in output format as prescribed

+ * by the domain read from the fully qualified name.

+ */

+int sized_domain_name(TALLOC_CTX *mem_ctx,

+                      struct resp_ctx *rctx,

+                      const char *member_name,

+                      struct sized_string **_name);

+

 #endif /* __SSS_RESPONDER_H__ */

diff --git a/src/responder/common/responder_common.c b/src/responder/common/responder_common.c

index c604c64a652221521ec7114b8588186f087eb11a..1db8a2283f3e96bccebe9a8443d5d04b9d5f4a54 100644

--- a/src/responder/common/responder_common.c

+++ b/src/responder/common/responder_common.c

@@ -1248,3 +1248,93 @@ void responder_set_fd_limit(rlim_t fd_limit)

                "Proceeding with system values\n");

     }

 }

+

+/**

+ * Helper functions to format output names

+ */

+int sized_output_name(TALLOC_CTX *mem_ctx,

+                      struct resp_ctx *rctx,

+                      const char *orig_name,

+                      struct sss_domain_info *name_dom,

+                      struct sized_string **_name)

+{

+    TALLOC_CTX *tmp_ctx = NULL;

+    errno_t ret;

+    char *username;

+    struct sized_string *name;

+

+    tmp_ctx = talloc_new(NULL);

+    if (tmp_ctx == NULL) {

+        return ENOMEM;

+    }

+

+    username = sss_output_name(tmp_ctx, orig_name, name_dom->case_preserve,

+                               rctx->override_space);

+    if (username == NULL) {

+        ret = EIO;

+        goto done;

+    }

+

+    if (name_dom->fqnames) {

+        username = sss_tc_fqname(tmp_ctx, name_dom->names, name_dom, username);

+        if (username == NULL) {

+            DEBUG(SSSDBG_CRIT_FAILURE, "sss_replace_space failed\n");

+            ret = EIO;

+            goto done;

+        }

+    }

+

+    name = talloc_zero(tmp_ctx, struct sized_string);

+    if (name == NULL) {

+        ret = ENOMEM;

+        goto done;

+    }

+

+    to_sized_string(name, username);

+    name->str = talloc_steal(name, username);

+    *_name = talloc_steal(mem_ctx, name);

+    ret = EOK;

+done:

+    talloc_zfree(tmp_ctx);

+    return ret;

+}

+

+int sized_domain_name(TALLOC_CTX *mem_ctx,

+                      struct resp_ctx *rctx,

+                      const char *member_name,

+                      struct sized_string **_name)

+{

+    TALLOC_CTX *tmp_ctx = NULL;

+    errno_t ret;

+    char *domname;

+    struct sss_domain_info *member_dom;

+

+    tmp_ctx = talloc_new(NULL);

+    if (tmp_ctx == NULL) {

+        return ENOMEM;

+    }

+

+    ret = sss_parse_internal_fqname(tmp_ctx, member_name, NULL, &domname);

+    if (ret != EOK) {

+        DEBUG(SSSDBG_CRIT_FAILURE, "sss_parse_internal_fqname failed\n");

+        goto done;

+    }

+

+    if (domname == NULL) {

+        ret = ERR_WRONG_NAME_FORMAT;

+        goto done;

+    }

+

+    member_dom = find_domain_by_name(get_domains_head(rctx->domains),

+                                     domname, true);

+    if (member_dom == NULL) {

+        ret = ERR_DOMAIN_NOT_FOUND;

+        goto done;

+    }

+

+    ret = sized_output_name(mem_ctx, rctx, member_name,

+                            member_dom, _name);

+done:

+    talloc_free(tmp_ctx);

+    return ret;

+}

diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c

index b64cea2a53ec6032904237b0afc1377022c2c804..0a9dd301266d4db1be1977cc8c337abde7cd79f5 100644

--- a/src/responder/nss/nsssrv_cmd.c

+++ b/src/responder/nss/nsssrv_cmd.c

@@ -253,93 +253,6 @@ static const char *get_shell_override(TALLOC_CTX *mem_ctx,

     return talloc_strdup(mem_ctx, NOLOGIN_SHELL);

 }

-static int sized_output_name(TALLOC_CTX *mem_ctx,

-                             struct resp_ctx *rctx,

-                             const char *orig_name,

-                             struct sss_domain_info *name_dom,

-                             struct sized_string **_name)

-{

-    TALLOC_CTX *tmp_ctx = NULL;

-    errno_t ret;

-    char *username;

-    struct sized_string *name;

-

-    tmp_ctx = talloc_new(NULL);

-    if (tmp_ctx == NULL) {

-        return ENOMEM;

-    }

-

-    username = sss_output_name(tmp_ctx, orig_name, name_dom->case_preserve,

-                               rctx->override_space);

-    if (username == NULL) {

-        ret = EIO;

-        goto done;

-    }

-

-    if (name_dom->fqnames) {

-        username = sss_tc_fqname(tmp_ctx, name_dom->names, name_dom, username);

-        if (username == NULL) {

-            DEBUG(SSSDBG_CRIT_FAILURE, "sss_replace_space failed\n");

-            ret = EIO;

-            goto done;

-        }

-    }

-

-    name = talloc_zero(tmp_ctx, struct sized_string);

-    if (name == NULL) {

-        ret = ENOMEM;

-        goto done;

-    }

-

-    to_sized_string(name, username);

-    name->str = talloc_steal(name, username);

-    *_name = talloc_steal(mem_ctx, name);

-    ret = EOK;

-done:

-    talloc_zfree(tmp_ctx);

-    return ret;

-}

-

-static int sized_member_name(TALLOC_CTX *mem_ctx,

-                             struct resp_ctx *rctx,

-                             const char *member_name,

-                             struct sized_string **_name)

-{

-    TALLOC_CTX *tmp_ctx = NULL;

-    errno_t ret;

-    char *domname;

-    struct sss_domain_info *member_dom;

-

-    tmp_ctx = talloc_new(NULL);

-    if (tmp_ctx == NULL) {

-        return ENOMEM;

-    }

-

-    ret = sss_parse_internal_fqname(tmp_ctx, member_name, NULL, &domname);

-    if (ret != EOK) {

-        DEBUG(SSSDBG_CRIT_FAILURE, "sss_parse_internal_fqname failed\n");

-        goto done;

-    }

-

-    if (domname == NULL) {

-        ret = ERR_WRONG_NAME_FORMAT;

-        goto done;

-    }

-

-    member_dom = find_domain_by_name(get_domains_head(rctx->domains),

-                                     domname, true);

-    if (member_dom == NULL) {

-        ret = ERR_DOMAIN_NOT_FOUND;

-        goto done;

-    }

-

-    ret = sized_output_name(mem_ctx, rctx, member_name,

-                            member_dom, _name);

-done:

-    talloc_free(tmp_ctx);

-    return ret;

-}

-

 static int fill_pwent(struct sss_packet *packet,

                       struct sss_domain_info *dom,

                       struct nss_ctx *nctx,

@@ -2727,7 +2640,7 @@ static int fill_members(struct sss_packet *packet,

             }

         }

-        ret = sized_member_name(tmp_ctx, rctx, fqname, &name);

+        ret = sized_domain_name(tmp_ctx, rctx, fqname, &name);

         if (ret != EOK) {

             DEBUG(SSSDBG_OP_FAILURE, "sized_member_name failed: %d\n", ret);

             goto done;

-- 

2.9.3