diff options
-rw-r--r-- | librpc/ndr/libndr.h | 2 | ||||
-rw-r--r-- | librpc/ndr/uuid.c | 4 | ||||
-rw-r--r-- | source3/rpc_server/epmapper/srv_epmapper.c | 4 | ||||
-rw-r--r-- | source4/libnet/libnet_domain.c | 2 | ||||
-rw-r--r-- | source4/libnet/prereq_domain.c | 8 | ||||
-rw-r--r-- | source4/rpc_server/handles.c | 2 | ||||
-rw-r--r-- | source4/torture/rpc/epmapper.c | 10 | ||||
-rw-r--r-- | source4/torture/rpc/samr.c | 13 |
8 files changed, 23 insertions, 22 deletions
diff --git a/librpc/ndr/libndr.h b/librpc/ndr/libndr.h index e68f30369f..c0a82b4ad9 100644 --- a/librpc/ndr/libndr.h +++ b/librpc/ndr/libndr.h @@ -555,7 +555,7 @@ NDR_SCALAR_PROTO(double, double) enum ndr_err_code ndr_pull_policy_handle(struct ndr_pull *ndr, int ndr_flags, struct policy_handle *r); enum ndr_err_code ndr_push_policy_handle(struct ndr_push *ndr, int ndr_flags, const struct policy_handle *r); void ndr_print_policy_handle(struct ndr_print *ndr, const char *name, const struct policy_handle *r); -bool policy_handle_empty(const struct policy_handle *h); +bool ndr_policy_handle_empty(const struct policy_handle *h); bool is_valid_policy_hnd(const struct policy_handle *hnd); bool ndr_policy_handle_equal(const struct policy_handle *hnd1, const struct policy_handle *hnd2); diff --git a/librpc/ndr/uuid.c b/librpc/ndr/uuid.c index 9ebccfe29a..f440505873 100644 --- a/librpc/ndr/uuid.c +++ b/librpc/ndr/uuid.c @@ -331,14 +331,14 @@ _PUBLIC_ char *NS_GUID_string(TALLOC_CTX *mem_ctx, const struct GUID *guid) guid->node[4], guid->node[5]); } -_PUBLIC_ bool policy_handle_empty(const struct policy_handle *h) +_PUBLIC_ bool ndr_policy_handle_empty(const struct policy_handle *h) { return (h->handle_type == 0 && GUID_all_zero(&h->uuid)); } _PUBLIC_ bool is_valid_policy_hnd(const struct policy_handle *hnd) { - return !policy_handle_empty(hnd); + return !ndr_policy_handle_empty(hnd); } _PUBLIC_ bool ndr_policy_handle_equal(const struct policy_handle *hnd1, diff --git a/source3/rpc_server/epmapper/srv_epmapper.c b/source3/rpc_server/epmapper/srv_epmapper.c index cb99bde528..4fea5887a4 100644 --- a/source3/rpc_server/epmapper/srv_epmapper.c +++ b/source3/rpc_server/epmapper/srv_epmapper.c @@ -537,7 +537,7 @@ error_status_t _epm_Lookup(struct pipes_struct *p, r->in.max_ents)); if (r->in.entry_handle == NULL || - policy_handle_empty(r->in.entry_handle)) { + ndr_policy_handle_empty(r->in.entry_handle)) { struct GUID *obj; char *srv_addr = NULL; @@ -924,7 +924,7 @@ error_status_t _epm_Map(struct pipes_struct *p, } if (r->in.entry_handle == NULL || - policy_handle_empty(r->in.entry_handle)) { + ndr_policy_handle_empty(r->in.entry_handle)) { struct GUID *obj; char *srv_addr = NULL; diff --git a/source4/libnet/libnet_domain.c b/source4/libnet/libnet_domain.c index b2a88827bd..d2b96958ee 100644 --- a/source4/libnet/libnet_domain.c +++ b/source4/libnet/libnet_domain.c @@ -308,7 +308,7 @@ struct composite_context *libnet_DomainOpenSamr_send(struct libnet_context *ctx, /* libnet context's domain handle is not empty, so check out what was opened first, before doing anything */ - if (!policy_handle_empty(&ctx->samr.handle)) { + if (!ndr_policy_handle_empty(&ctx->samr.handle)) { if (strequal(ctx->samr.name, io->in.domain_name) && ctx->samr.access_mask == io->in.access_mask) { diff --git a/source4/libnet/prereq_domain.c b/source4/libnet/prereq_domain.c index e10e550585..039d99d9ef 100644 --- a/source4/libnet/prereq_domain.c +++ b/source4/libnet/prereq_domain.c @@ -45,7 +45,7 @@ bool samr_domain_opened(struct libnet_context *ctx, const char *domain_name, * if it's not been explicitly specified. */ - if (policy_handle_empty(&ctx->samr.handle)) { + if (ndr_policy_handle_empty(&ctx->samr.handle)) { domain_open->in.type = DOMAIN_SAMR; domain_open->in.domain_name = cli_credentials_get_domain(ctx->cred); domain_open->in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED; @@ -62,7 +62,7 @@ bool samr_domain_opened(struct libnet_context *ctx, const char *domain_name, * opening a new domain otherwise. */ - if (policy_handle_empty(&ctx->samr.handle) || + if (ndr_policy_handle_empty(&ctx->samr.handle) || !strequal(domain_name, ctx->samr.name)) { domain_open->in.type = DOMAIN_SAMR; domain_open->in.domain_name = domain_name; @@ -100,7 +100,7 @@ bool lsa_domain_opened(struct libnet_context *ctx, const char *domain_name, * if it's not been explicitly specified. */ - if (policy_handle_empty(&ctx->lsa.handle)) { + if (ndr_policy_handle_empty(&ctx->lsa.handle)) { domain_open->in.type = DOMAIN_LSA; domain_open->in.domain_name = cli_credentials_get_domain(ctx->cred); domain_open->in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED; @@ -119,7 +119,7 @@ bool lsa_domain_opened(struct libnet_context *ctx, const char *domain_name, * opening a new domain otherwise. */ - if (policy_handle_empty(&ctx->lsa.handle) || + if (ndr_policy_handle_empty(&ctx->lsa.handle) || !strequal(domain_name, ctx->lsa.name)) { domain_open->in.type = DOMAIN_LSA; domain_open->in.domain_name = domain_name; diff --git a/source4/rpc_server/handles.c b/source4/rpc_server/handles.c index e70510c875..be9f16ce73 100644 --- a/source4/rpc_server/handles.c +++ b/source4/rpc_server/handles.c @@ -82,7 +82,7 @@ _PUBLIC_ struct dcesrv_handle *dcesrv_handle_fetch( sid = &context->conn->auth_state.session_info->security_token->sids[PRIMARY_USER_SID_INDEX]; - if (policy_handle_empty(p)) { + if (ndr_policy_handle_empty(p)) { /* TODO: we should probably return a NULL handle here */ return dcesrv_handle_new(context, handle_type); } diff --git a/source4/torture/rpc/epmapper.c b/source4/torture/rpc/epmapper.c index c0d8c44e02..c605a05f6a 100644 --- a/source4/torture/rpc/epmapper.c +++ b/source4/torture/rpc/epmapper.c @@ -406,12 +406,12 @@ static bool test_Map_simple(struct torture_context *tctx, } while (NT_STATUS_IS_OK(status) && r.out.result == EPMAPPER_STATUS_OK && *r.out.num_ents == r.in.max_ents && - !policy_handle_empty(&entry_handle)); + !ndr_policy_handle_empty(&entry_handle)); torture_assert_ntstatus_ok(tctx, status, "epm_Map_simple failed"); torture_assert(tctx, - policy_handle_empty(&entry_handle), + ndr_policy_handle_empty(&entry_handle), "epm_Map_simple failed - The policy handle should be emtpy."); return true; @@ -427,7 +427,7 @@ static bool test_LookupHandleFree(struct torture_context *tctx, torture_skip(tctx, "Skip Insert test against Samba4"); } - if (policy_handle_empty(entry_handle)) { + if (ndr_policy_handle_empty(entry_handle)) { torture_comment(tctx, "epm_LookupHandleFree failed - empty policy_handle\n"); return false; @@ -504,13 +504,13 @@ static bool test_Lookup_simple(struct torture_context *tctx, } while (NT_STATUS_IS_OK(status) && r.out.result == EPMAPPER_STATUS_OK && *r.out.num_ents == r.in.max_ents && - !policy_handle_empty(&entry_handle)); + !ndr_policy_handle_empty(&entry_handle)); torture_assert_ntstatus_ok(tctx, status, "epm_Lookup failed"); torture_assert(tctx, r.out.result == EPMAPPER_STATUS_NO_MORE_ENTRIES, "epm_Lookup failed"); torture_assert(tctx, - policy_handle_empty(&entry_handle), + ndr_policy_handle_empty(&entry_handle), "epm_Lookup failed - The policy handle should be emtpy."); return true; diff --git a/source4/torture/rpc/samr.c b/source4/torture/rpc/samr.c index 6b90791660..bf1d713eed 100644 --- a/source4/torture/rpc/samr.c +++ b/source4/torture/rpc/samr.c @@ -4,6 +4,7 @@ Copyright (C) Andrew Tridgell 2003 Copyright (C) Andrew Bartlett <abartlet@samba.org> 2003 + Copyright (C) Jelmer Vernooij 2005-2007 Copyright (C) Guenther Deschner 2008-2010 This program is free software; you can redistribute it and/or modify @@ -5536,7 +5537,7 @@ static bool test_CreateUser2(struct dcerpc_pipe *p, struct torture_context *tctx ret = false; } - if (!policy_handle_empty(&user_handle)) { + if (!ndr_policy_handle_empty(&user_handle)) { torture_comment(tctx, "Testing DeleteUser (createuser2 test)\n"); d.in.user_handle = &user_handle; @@ -7483,7 +7484,7 @@ static bool test_ManyObjects(struct dcerpc_pipe *p, default: return false; } - if (!policy_handle_empty(&handles[i])) { + if (!ndr_policy_handle_empty(&handles[i])) { num_created++; } } @@ -7534,7 +7535,7 @@ static bool test_ManyObjects(struct dcerpc_pipe *p, for (i=0; i < num_total; i++) { - if (policy_handle_empty(&handles[i])) { + if (ndr_policy_handle_empty(&handles[i])) { continue; } @@ -7694,17 +7695,17 @@ static bool test_OpenDomain(struct dcerpc_pipe *p, struct torture_context *tctx, break; } - if (!policy_handle_empty(&user_handle) && + if (!ndr_policy_handle_empty(&user_handle) && !test_DeleteUser(b, tctx, &user_handle)) { ret = false; } - if (!policy_handle_empty(&alias_handle) && + if (!ndr_policy_handle_empty(&alias_handle) && !test_DeleteAlias(b, tctx, &alias_handle)) { ret = false; } - if (!policy_handle_empty(&group_handle) && + if (!ndr_policy_handle_empty(&group_handle) && !test_DeleteDomainGroup(b, tctx, &group_handle)) { ret = false; } |