s3-talloc Change TALLOC_ARRAY() to talloc_array()
[samba.git] / source3 / rpc_client / cli_lsarpc.c
index d49fa47f57a3cf0b8f4abee6f466134e74202e9a..66192bdf54f9ec8c8c51c121518731fb7e0aea33 100644 (file)
 */
 
 #include "includes.h"
+#include "rpc_client/rpc_client.h"
+#include "../librpc/gen_ndr/ndr_lsa_c.h"
+#include "rpc_client/cli_lsarpc.h"
+#include "rpc_client/init_lsa.h"
+#include "../libcli/security/security.h"
 
 /** @defgroup lsa LSA - Local Security Architecture
  *  @ingroup rpc_client
  * security authority", which is half of a password database.
  **/
 
-/** Open a LSA policy handle
- *
- * @param cli Handle on an initialised SMB connection */
-
-NTSTATUS rpccli_lsa_open_policy(struct rpc_pipe_client *cli,
+NTSTATUS dcerpc_lsa_open_policy(struct dcerpc_binding_handle *h,
                                TALLOC_CTX *mem_ctx,
-                               bool sec_qos, uint32 des_access,
-                               struct policy_handle *pol)
+                               bool sec_qos,
+                               uint32_t des_access,
+                               struct policy_handle *pol,
+                               NTSTATUS *result)
 {
        struct lsa_ObjectAttribute attr;
        struct lsa_QosInfo qos;
@@ -63,21 +66,47 @@ NTSTATUS rpccli_lsa_open_policy(struct rpc_pipe_client *cli,
                attr.sec_qos            = &qos;
        }
 
-       return rpccli_lsa_OpenPolicy(cli, mem_ctx,
+       return dcerpc_lsa_OpenPolicy(h,
+                                    mem_ctx,
                                     &system_name,
                                     &attr,
                                     des_access,
-                                    pol);
+                                    pol,
+                                    result);
 }
 
 /** Open a LSA policy handle
-  *
-  * @param cli Handle on an initialised SMB connection
-  */
+ *
+ * @param cli Handle on an initialised SMB connection */
 
-NTSTATUS rpccli_lsa_open_policy2(struct rpc_pipe_client *cli,
-                                TALLOC_CTX *mem_ctx, bool sec_qos,
-                                uint32 des_access, struct policy_handle *pol)
+NTSTATUS rpccli_lsa_open_policy(struct rpc_pipe_client *cli,
+                               TALLOC_CTX *mem_ctx,
+                               bool sec_qos, uint32 des_access,
+                               struct policy_handle *pol)
+{
+       NTSTATUS status;
+       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
+
+       status = dcerpc_lsa_open_policy(cli->binding_handle,
+                                       mem_ctx,
+                                       sec_qos,
+                                       des_access,
+                                       pol,
+                                       &result);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
+
+       return result;
+}
+
+NTSTATUS dcerpc_lsa_open_policy2(struct dcerpc_binding_handle *h,
+                                TALLOC_CTX *mem_ctx,
+                                const char *srv_name_slash,
+                                bool sec_qos,
+                                uint32_t des_access,
+                                struct policy_handle *pol,
+                                NTSTATUS *result)
 {
        struct lsa_ObjectAttribute attr;
        struct lsa_QosInfo qos;
@@ -95,54 +124,76 @@ NTSTATUS rpccli_lsa_open_policy2(struct rpc_pipe_client *cli,
                attr.sec_qos            = &qos;
        }
 
-       return rpccli_lsa_OpenPolicy2(cli, mem_ctx,
-                                     cli->srv_name_slash,
+       return dcerpc_lsa_OpenPolicy2(h,
+                                     mem_ctx,
+                                     srv_name_slash,
                                      &attr,
                                      des_access,
-                                     pol);
+                                     pol,
+                                     result);
+}
+
+/** Open a LSA policy handle
+  *
+  * @param cli Handle on an initialised SMB connection
+  */
+
+NTSTATUS rpccli_lsa_open_policy2(struct rpc_pipe_client *cli,
+                                TALLOC_CTX *mem_ctx, bool sec_qos,
+                                uint32 des_access, struct policy_handle *pol)
+{
+       NTSTATUS status;
+       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
+
+       status = dcerpc_lsa_open_policy2(cli->binding_handle,
+                                        mem_ctx,
+                                        cli->srv_name_slash,
+                                        sec_qos,
+                                        des_access,
+                                        pol,
+                                        &result);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
+
+       return result;
 }
 
 /* Lookup a list of sids
  *
  * internal version withOUT memory allocation of the target arrays.
- * this assumes suffciently sized arrays to store domains, names and types. */
+ * this assumes sufficiently sized arrays to store domains, names and types. */
 
-static NTSTATUS rpccli_lsa_lookup_sids_noalloc(struct rpc_pipe_client *cli,
+static NTSTATUS dcerpc_lsa_lookup_sids_noalloc(struct dcerpc_binding_handle *h,
                                               TALLOC_CTX *mem_ctx,
                                               struct policy_handle *pol,
                                               int num_sids,
-                                              const DOM_SID *sids,
+                                              const struct dom_sid *sids,
                                               char **domains,
                                               char **names,
                                               enum lsa_SidType *types,
-                                              bool use_lookupsids3)
+                                              bool use_lookupsids3,
+                                              NTSTATUS *presult)
 {
-       NTSTATUS result = NT_STATUS_OK;
-       TALLOC_CTX *tmp_ctx = NULL;
-       int i;
+       NTSTATUS status = NT_STATUS_OK;
+       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
        struct lsa_SidArray sid_array;
        struct lsa_RefDomainList *ref_domains = NULL;
        struct lsa_TransNameArray lsa_names;
+       enum lsa_LookupNamesLevel level = LSA_LOOKUP_NAMES_ALL;
        uint32_t count = 0;
-       uint16_t level = 1;
+       int i;
 
        ZERO_STRUCT(lsa_names);
 
-       tmp_ctx = talloc_new(mem_ctx);
-       if (!tmp_ctx) {
-               DEBUG(0, ("rpccli_lsa_lookup_sids_noalloc: out of memory!\n"));
-               result = NT_STATUS_UNSUCCESSFUL;
-               goto done;
-       }
-
        sid_array.num_sids = num_sids;
-       sid_array.sids = TALLOC_ARRAY(mem_ctx, struct lsa_SidPtr, num_sids);
-       if (!sid_array.sids) {
+       sid_array.sids = talloc_array(mem_ctx, struct lsa_SidPtr, num_sids);
+       if (sid_array.sids == NULL) {
                return NT_STATUS_NO_MEMORY;
        }
 
        for (i = 0; i<num_sids; i++) {
-               sid_array.sids[i].sid = sid_dup_talloc(mem_ctx, &sids[i]);
+               sid_array.sids[i].sid = dom_sid_dup(mem_ctx, &sids[i]);
                if (!sid_array.sids[i].sid) {
                        return NT_STATUS_NO_MEMORY;
                }
@@ -154,19 +205,26 @@ static NTSTATUS rpccli_lsa_lookup_sids_noalloc(struct rpc_pipe_client *cli,
 
                ZERO_STRUCT(lsa_names2);
 
-               result = rpccli_lsa_LookupSids3(cli, mem_ctx,
+               status = dcerpc_lsa_LookupSids3(h,
+                                               mem_ctx,
                                                &sid_array,
                                                &ref_domains,
                                                &lsa_names2,
                                                level,
                                                &count,
-                                               0,
-                                               0);
+                                               LSA_LOOKUP_OPTION_SEARCH_ISOLATED_NAMES,
+                                               LSA_CLIENT_REVISION_2,
+                                               &result);
+               if (!NT_STATUS_IS_OK(status)) {
+                       return status;
+               }
 
-               if (!NT_STATUS_IS_ERR(result)) {
-                       lsa_names.count = lsa_names2.count;
-                       lsa_names.names = talloc_array(mem_ctx, struct lsa_TranslatedName, lsa_names.count);
-                       if (!lsa_names.names) {
+               if(!NT_STATUS_IS_ERR(result)) {
+                       lsa_names.count = lsa_names2.count;
+                       lsa_names.names = talloc_array(mem_ctx,
+                                                      struct lsa_TranslatedName,
+                                                      lsa_names.count);
+                       if (lsa_names.names == NULL) {
                                return NT_STATUS_NO_MEMORY;
                        }
                        for (n=0; n < lsa_names.count; n++) {
@@ -177,28 +235,34 @@ static NTSTATUS rpccli_lsa_lookup_sids_noalloc(struct rpc_pipe_client *cli,
                }
 
        } else {
-               result = rpccli_lsa_LookupSids(cli, mem_ctx,
+               status = dcerpc_lsa_LookupSids(h,
+                                              mem_ctx,
                                               pol,
                                               &sid_array,
                                               &ref_domains,
                                               &lsa_names,
                                               level,
-                                              &count);
+                                              &count,
+                                              &result);
        }
 
-       DEBUG(10, ("LSA_LOOKUPSIDS returned '%s', mapped count = %d'\n",
-                  nt_errstr(result), count));
+       DEBUG(10, ("LSA_LOOKUPSIDS returned status: '%s', result: '%s', "
+                  "mapped count = %d'\n",
+                  nt_errstr(status), nt_errstr(result), count));
+
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
 
        if (!NT_STATUS_IS_OK(result) &&
            !NT_STATUS_EQUAL(result, NT_STATUS_NONE_MAPPED) &&
            !NT_STATUS_EQUAL(result, STATUS_SOME_UNMAPPED))
        {
-               /* An actual error occured */
-               goto done;
+               *presult = result;
+               return status;
        }
 
        /* Return output parameters */
-
        if (NT_STATUS_EQUAL(result, NT_STATUS_NONE_MAPPED) ||
            (count == 0))
        {
@@ -207,8 +271,8 @@ static NTSTATUS rpccli_lsa_lookup_sids_noalloc(struct rpc_pipe_client *cli,
                        (domains)[i] = NULL;
                        (types)[i] = SID_NAME_UNKNOWN;
                }
-               result = NT_STATUS_NONE_MAPPED;
-               goto done;
+               *presult = NT_STATUS_NONE_MAPPED;
+               return status;
        }
 
        for (i = 0; i < num_sids; i++) {
@@ -223,21 +287,22 @@ static NTSTATUS rpccli_lsa_lookup_sids_noalloc(struct rpc_pipe_client *cli,
                        name = lsa_names.names[i].name.string;
 
                        if (name) {
-                               (names)[i] = talloc_strdup(mem_ctx, name);
+                               (names)[i] = talloc_strdup(names, name);
                                if ((names)[i] == NULL) {
                                        DEBUG(0, ("cli_lsa_lookup_sids_noalloc(): out of memory\n"));
-                                       result = NT_STATUS_UNSUCCESSFUL;
-                                       goto done;
+                                       *presult = NT_STATUS_UNSUCCESSFUL;
+                                       return status;
                                }
                        } else {
                                (names)[i] = NULL;
                        }
-                       (domains)[i] = talloc_strdup(mem_ctx, dom_name);
+                       domains[i] = talloc_strdup(domains,
+                                                  dom_name ? dom_name : "");
                        (types)[i] = lsa_names.names[i].sid_type;
                        if (((domains)[i] == NULL)) {
                                DEBUG(0, ("cli_lsa_lookup_sids_noalloc(): out of memory\n"));
-                               result = NT_STATUS_UNSUCCESSFUL;
-                               goto done;
+                               *presult = NT_STATUS_UNSUCCESSFUL;
+                               return status;
                        }
 
                } else {
@@ -247,9 +312,8 @@ static NTSTATUS rpccli_lsa_lookup_sids_noalloc(struct rpc_pipe_client *cli,
                }
        }
 
-done:
-       TALLOC_FREE(tmp_ctx);
-       return result;
+       *presult = NT_STATUS_OK;
+       return status;
 }
 
 /* Lookup a list of sids
@@ -264,43 +328,47 @@ done:
  * at 20480 for win2k3, but we keep it at a save 1000 for now. */
 #define LOOKUP_SIDS_HUNK_SIZE 1000
 
-static NTSTATUS rpccli_lsa_lookup_sids_generic(struct rpc_pipe_client *cli,
+static NTSTATUS dcerpc_lsa_lookup_sids_generic(struct dcerpc_binding_handle *h,
                                               TALLOC_CTX *mem_ctx,
                                               struct policy_handle *pol,
                                               int num_sids,
-                                              const DOM_SID *sids,
+                                              const struct dom_sid *sids,
                                               char ***pdomains,
                                               char ***pnames,
                                               enum lsa_SidType **ptypes,
-                                              bool use_lookupsids3)
+                                              bool use_lookupsids3,
+                                              NTSTATUS *presult)
 {
+       NTSTATUS status = NT_STATUS_OK;
        NTSTATUS result = NT_STATUS_OK;
        int sids_left = 0;
        int sids_processed = 0;
-       const DOM_SID *hunk_sids = sids;
+       const struct dom_sid *hunk_sids = sids;
        char **hunk_domains;
        char **hunk_names;
        enum lsa_SidType *hunk_types;
        char **domains = NULL;
        char **names = NULL;
        enum lsa_SidType *types = NULL;
+       bool have_mapped = false;
+       bool have_unmapped = false;
 
        if (num_sids) {
-               if (!(domains = TALLOC_ARRAY(mem_ctx, char *, num_sids))) {
+               if (!(domains = talloc_array(mem_ctx, char *, num_sids))) {
                        DEBUG(0, ("rpccli_lsa_lookup_sids(): out of memory\n"));
-                       result = NT_STATUS_NO_MEMORY;
+                       status = NT_STATUS_NO_MEMORY;
                        goto fail;
                }
 
-               if (!(names = TALLOC_ARRAY(mem_ctx, char *, num_sids))) {
+               if (!(names = talloc_array(mem_ctx, char *, num_sids))) {
                        DEBUG(0, ("rpccli_lsa_lookup_sids(): out of memory\n"));
-                       result = NT_STATUS_NO_MEMORY;
+                       status = NT_STATUS_NO_MEMORY;
                        goto fail;
                }
 
-               if (!(types = TALLOC_ARRAY(mem_ctx, enum lsa_SidType, num_sids))) {
+               if (!(types = talloc_array(mem_ctx, enum lsa_SidType, num_sids))) {
                        DEBUG(0, ("rpccli_lsa_lookup_sids(): out of memory\n"));
-                       result = NT_STATUS_NO_MEMORY;
+                       status = NT_STATUS_NO_MEMORY;
                        goto fail;
                }
        }
@@ -312,7 +380,7 @@ static NTSTATUS rpccli_lsa_lookup_sids_generic(struct rpc_pipe_client *cli,
 
        while (sids_left > 0) {
                int hunk_num_sids;
-               NTSTATUS hunk_result = NT_STATUS_OK;
+               NTSTATUS hunk_result = NT_STATUS_UNSUCCESSFUL;
 
                hunk_num_sids = ((sids_left > LOOKUP_SIDS_HUNK_SIZE)
                                ? LOOKUP_SIDS_HUNK_SIZE
@@ -324,33 +392,44 @@ static NTSTATUS rpccli_lsa_lookup_sids_generic(struct rpc_pipe_client *cli,
                           sids_processed + hunk_num_sids - 1,
                           num_sids));
 
-               hunk_result = rpccli_lsa_lookup_sids_noalloc(cli,
-                                                            mem_ctx,
-                                                            pol,
-                                                            hunk_num_sids,
-                                                            hunk_sids,
-                                                            hunk_domains,
-                                                            hunk_names,
-                                                            hunk_types,
-                                                            use_lookupsids3);
+               status = dcerpc_lsa_lookup_sids_noalloc(h,
+                                                       mem_ctx,
+                                                       pol,
+                                                       hunk_num_sids,
+                                                       hunk_sids,
+                                                       hunk_domains,
+                                                       hunk_names,
+                                                       hunk_types,
+                                                       use_lookupsids3,
+                                                       &hunk_result);
+               if (!NT_STATUS_IS_OK(status)) {
+                       goto fail;
+               }
 
                if (!NT_STATUS_IS_OK(hunk_result) &&
                    !NT_STATUS_EQUAL(hunk_result, STATUS_SOME_UNMAPPED) &&
                    !NT_STATUS_EQUAL(hunk_result, NT_STATUS_NONE_MAPPED))
                {
                        /* An actual error occured */
-                       result = hunk_result;
+                       *presult = hunk_result;
                        goto fail;
                }
 
-               /* adapt overall result */
-               if (( NT_STATUS_IS_OK(result) &&
-                    !NT_STATUS_IS_OK(hunk_result))
-                   ||
-                   ( NT_STATUS_EQUAL(result, NT_STATUS_NONE_MAPPED) &&
-                    !NT_STATUS_EQUAL(hunk_result, NT_STATUS_NONE_MAPPED)))
-               {
-                       result = STATUS_SOME_UNMAPPED;
+               if (NT_STATUS_IS_OK(hunk_result)) {
+                       have_mapped = true;
+               }
+               if (NT_STATUS_EQUAL(hunk_result, NT_STATUS_NONE_MAPPED)) {
+                       have_unmapped = true;
+               }
+               if (NT_STATUS_EQUAL(hunk_result, STATUS_SOME_UNMAPPED)) {
+                       int i;
+                       for (i=0; i<hunk_num_sids; i++) {
+                               if (hunk_types[i] == SID_NAME_UNKNOWN) {
+                                       have_unmapped = true;
+                               } else {
+                                       have_mapped = true;
+                               }
+                       }
                }
 
                sids_left -= hunk_num_sids;
@@ -364,127 +443,219 @@ static NTSTATUS rpccli_lsa_lookup_sids_generic(struct rpc_pipe_client *cli,
        *pdomains = domains;
        *pnames = names;
        *ptypes = types;
-       return result;
+
+       if (!have_mapped) {
+               result = NT_STATUS_NONE_MAPPED;
+       }
+       if (have_unmapped) {
+               result = STATUS_SOME_UNMAPPED;
+       }
+       *presult = result;
+
+       return status;
 
 fail:
        TALLOC_FREE(domains);
        TALLOC_FREE(names);
        TALLOC_FREE(types);
-       return result;
+
+       return status;
+}
+
+NTSTATUS dcerpc_lsa_lookup_sids(struct dcerpc_binding_handle *h,
+                               TALLOC_CTX *mem_ctx,
+                               struct policy_handle *pol,
+                               int num_sids,
+                               const struct dom_sid *sids,
+                               char ***pdomains,
+                               char ***pnames,
+                               enum lsa_SidType **ptypes,
+                               NTSTATUS *result)
+{
+       return dcerpc_lsa_lookup_sids_generic(h,
+                                             mem_ctx,
+                                             pol,
+                                             num_sids,
+                                             sids,
+                                             pdomains,
+                                             pnames,
+                                             ptypes,
+                                             false,
+                                             result);
 }
 
 NTSTATUS rpccli_lsa_lookup_sids(struct rpc_pipe_client *cli,
                                TALLOC_CTX *mem_ctx,
                                struct policy_handle *pol,
                                int num_sids,
-                               const DOM_SID *sids,
+                               const struct dom_sid *sids,
                                char ***pdomains,
                                char ***pnames,
                                enum lsa_SidType **ptypes)
 {
-       return rpccli_lsa_lookup_sids_generic(cli, mem_ctx, pol, num_sids, sids,
-                                             pdomains, pnames, ptypes, false);
+       NTSTATUS status;
+       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
+
+       status = dcerpc_lsa_lookup_sids_generic(cli->binding_handle,
+                                               mem_ctx,
+                                               pol,
+                                               num_sids,
+                                               sids,
+                                               pdomains,
+                                               pnames,
+                                               ptypes,
+                                               false,
+                                               &result);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
+
+       return result;
+}
+
+NTSTATUS dcerpc_lsa_lookup_sids3(struct dcerpc_binding_handle *h,
+                                TALLOC_CTX *mem_ctx,
+                                struct policy_handle *pol,
+                                int num_sids,
+                                const struct dom_sid *sids,
+                                char ***pdomains,
+                                char ***pnames,
+                                enum lsa_SidType **ptypes,
+                                NTSTATUS *result)
+{
+       return dcerpc_lsa_lookup_sids_generic(h,
+                                             mem_ctx,
+                                             pol,
+                                             num_sids,
+                                             sids,
+                                             pdomains,
+                                             pnames,
+                                             ptypes,
+                                             true,
+                                             result);
 }
 
 NTSTATUS rpccli_lsa_lookup_sids3(struct rpc_pipe_client *cli,
                                 TALLOC_CTX *mem_ctx,
                                 struct policy_handle *pol,
                                 int num_sids,
-                                const DOM_SID *sids,
+                                const struct dom_sid *sids,
                                 char ***pdomains,
                                 char ***pnames,
                                 enum lsa_SidType **ptypes)
 {
-       return rpccli_lsa_lookup_sids_generic(cli, mem_ctx, pol, num_sids, sids,
-                                             pdomains, pnames, ptypes, true);
+       NTSTATUS status;
+       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
+
+       status = dcerpc_lsa_lookup_sids_generic(cli->binding_handle,
+                                               mem_ctx,
+                                               pol,
+                                               num_sids,
+                                               sids,
+                                               pdomains,
+                                               pnames,
+                                               ptypes,
+                                               true,
+                                               &result);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
+
+       return result;
 }
 
 /** Lookup a list of names */
 
-static NTSTATUS rpccli_lsa_lookup_names_generic(struct rpc_pipe_client *cli,
+static NTSTATUS dcerpc_lsa_lookup_names_generic(struct dcerpc_binding_handle *h,
                                                TALLOC_CTX *mem_ctx,
-                                               struct policy_handle *pol, int num_names,
+                                               struct policy_handle *pol,
+                                               uint32_t num_names,
                                                const char **names,
                                                const char ***dom_names,
-                                               int level,
-                                               DOM_SID **sids,
+                                               enum lsa_LookupNamesLevel level,
+                                               struct dom_sid **sids,
                                                enum lsa_SidType **types,
-                                               bool use_lookupnames4)
+                                               bool use_lookupnames4,
+                                               NTSTATUS *presult)
 {
-       NTSTATUS result;
-       int i;
+       NTSTATUS status;
        struct lsa_String *lsa_names = NULL;
        struct lsa_RefDomainList *domains = NULL;
        struct lsa_TransSidArray sid_array;
        struct lsa_TransSidArray3 sid_array3;
        uint32_t count = 0;
+       uint32_t i;
 
        ZERO_STRUCT(sid_array);
        ZERO_STRUCT(sid_array3);
 
-       lsa_names = TALLOC_ARRAY(mem_ctx, struct lsa_String, num_names);
-       if (!lsa_names) {
+       lsa_names = talloc_array(mem_ctx, struct lsa_String, num_names);
+       if (lsa_names == NULL) {
                return NT_STATUS_NO_MEMORY;
        }
 
-       for (i=0; i<num_names; i++) {
+       for (i = 0; i < num_names; i++) {
                init_lsa_String(&lsa_names[i], names[i]);
        }
 
        if (use_lookupnames4) {
-               result = rpccli_lsa_LookupNames4(cli, mem_ctx,
+               status = dcerpc_lsa_LookupNames4(h,
+                                                mem_ctx,
                                                 num_names,
                                                 lsa_names,
                                                 &domains,
                                                 &sid_array3,
                                                 level,
                                                 &count,
-                                                0,
-                                                0);
+                                                LSA_LOOKUP_OPTION_SEARCH_ISOLATED_NAMES,
+                                                LSA_CLIENT_REVISION_2,
+                                                presult);
        } else {
-               result = rpccli_lsa_LookupNames(cli, mem_ctx,
-                                               pol,
+               status = dcerpc_lsa_LookupNames(h,
+                                               mem_ctx,
+                                               pol,
                                                num_names,
                                                lsa_names,
                                                &domains,
                                                &sid_array,
                                                level,
-                                               &count);
+                                               &count,
+                                               presult);
+       }
+       if (!NT_STATUS_IS_OK(status)) {
+               goto done;
        }
 
-       if (!NT_STATUS_IS_OK(result) && NT_STATUS_V(result) !=
-           NT_STATUS_V(STATUS_SOME_UNMAPPED)) {
-
+       if (!NT_STATUS_IS_OK(*presult) &&
+           !NT_STATUS_EQUAL(*presult, STATUS_SOME_UNMAPPED)) {
                /* An actual error occured */
-
                goto done;
        }
 
        /* Return output parameters */
-
        if (count == 0) {
-               result = NT_STATUS_NONE_MAPPED;
+               *presult = NT_STATUS_NONE_MAPPED;
                goto done;
        }
 
        if (num_names) {
-               if (!((*sids = TALLOC_ARRAY(mem_ctx, DOM_SID, num_names)))) {
+               if (!((*sids = talloc_array(mem_ctx, struct dom_sid, num_names)))) {
                        DEBUG(0, ("cli_lsa_lookup_sids(): out of memory\n"));
-                       result = NT_STATUS_NO_MEMORY;
+                       *presult = NT_STATUS_NO_MEMORY;
                        goto done;
                }
 
-               if (!((*types = TALLOC_ARRAY(mem_ctx, enum lsa_SidType, num_names)))) {
+               if (!((*types = talloc_array(mem_ctx, enum lsa_SidType, num_names)))) {
                        DEBUG(0, ("cli_lsa_lookup_sids(): out of memory\n"));
-                       result = NT_STATUS_NO_MEMORY;
+                       *presult = NT_STATUS_NO_MEMORY;
                        goto done;
                }
 
                if (dom_names != NULL) {
-                       *dom_names = TALLOC_ARRAY(mem_ctx, const char *, num_names);
+                       *dom_names = talloc_array(mem_ctx, const char *, num_names);
                        if (*dom_names == NULL) {
                                DEBUG(0, ("cli_lsa_lookup_sids(): out of memory\n"));
-                               result = NT_STATUS_NO_MEMORY;
+                               *presult = NT_STATUS_NO_MEMORY;
                                goto done;
                        }
                }
@@ -498,7 +669,7 @@ static NTSTATUS rpccli_lsa_lookup_names_generic(struct rpc_pipe_client *cli,
 
        for (i = 0; i < num_names; i++) {
                uint32_t dom_idx;
-               DOM_SID *sid = &(*sids)[i];
+               struct dom_sid *sid = &(*sids)[i];
 
                if (use_lookupnames4) {
                        dom_idx         = sid_array3.sids[i].sid_index;
@@ -535,34 +706,113 @@ static NTSTATUS rpccli_lsa_lookup_names_generic(struct rpc_pipe_client *cli,
        }
 
  done:
+       return status;
+}
 
-       return result;
+NTSTATUS dcerpc_lsa_lookup_names(struct dcerpc_binding_handle *h,
+                                TALLOC_CTX *mem_ctx,
+                                struct policy_handle *pol,
+                                uint32_t num_names,
+                                const char **names,
+                                const char ***dom_names,
+                                enum lsa_LookupNamesLevel level,
+                                struct dom_sid **sids,
+                                enum lsa_SidType **types,
+                                NTSTATUS *result)
+{
+       return dcerpc_lsa_lookup_names_generic(h,
+                                              mem_ctx,
+                                              pol,
+                                              num_names,
+                                              names,
+                                              dom_names,
+                                              level,
+                                              sids,
+                                              types,
+                                              false,
+                                              result);
 }
 
 NTSTATUS rpccli_lsa_lookup_names(struct rpc_pipe_client *cli,
                                 TALLOC_CTX *mem_ctx,
-                                struct policy_handle *pol, int num_names,
+                                struct policy_handle *pol,
+                                int num_names,
                                 const char **names,
                                 const char ***dom_names,
                                 int level,
-                                DOM_SID **sids,
+                                struct dom_sid **sids,
                                 enum lsa_SidType **types)
 {
-       return rpccli_lsa_lookup_names_generic(cli, mem_ctx, pol, num_names,
-                                              names, dom_names, level, sids,
-                                              types, false);
+       NTSTATUS status;
+       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
+
+       status = dcerpc_lsa_lookup_names(cli->binding_handle,
+                                        mem_ctx,
+                                        pol,
+                                        num_names,
+                                        names,
+                                        dom_names,
+                                        level,
+                                        sids,
+                                        types,
+                                        &result);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
+
+       return result;
+}
+
+NTSTATUS dcerpc_lsa_lookup_names4(struct dcerpc_binding_handle *h,
+                                 TALLOC_CTX *mem_ctx,
+                                 struct policy_handle *pol,
+                                 uint32_t num_names,
+                                 const char **names,
+                                 const char ***dom_names,
+                                 enum lsa_LookupNamesLevel level,
+                                 struct dom_sid **sids,
+                                 enum lsa_SidType **types,
+                                 NTSTATUS *result)
+{
+       return dcerpc_lsa_lookup_names_generic(h,
+                                              mem_ctx,
+                                              pol,
+                                              num_names,
+                                              names,
+                                              dom_names,
+                                              level,
+                                              sids,
+                                              types,
+                                              true,
+                                              result);
 }
 
 NTSTATUS rpccli_lsa_lookup_names4(struct rpc_pipe_client *cli,
                                  TALLOC_CTX *mem_ctx,
-                                 struct policy_handle *pol, int num_names,
+                                 struct policy_handle *pol,
+                                 int num_names,
                                  const char **names,
                                  const char ***dom_names,
                                  int level,
-                                 DOM_SID **sids,
+                                 struct dom_sid **sids,
                                  enum lsa_SidType **types)
 {
-       return rpccli_lsa_lookup_names_generic(cli, mem_ctx, pol, num_names,
-                                              names, dom_names, level, sids,
-                                              types, true);
+       NTSTATUS status;
+       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
+
+       status = dcerpc_lsa_lookup_names4(cli->binding_handle,
+                                         mem_ctx,
+                                         pol,
+                                         num_names,
+                                         names,
+                                         dom_names,
+                                         level,
+                                         sids,
+                                         types,
+                                         &result);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
+
+       return result;
 }