From 47e03d96a263a7f9a5e3061c04147e559f2fe9e0 Mon Sep 17 00:00:00 2001 From: Rafal Szczesniak Date: Sun, 10 Dec 2006 23:43:32 +0000 Subject: r20103: Change the returned type of prerequisite checking functions, as metze once suggested. rafal (This used to be commit 1f3fd5f85461d1e8c9cbdc3144d0a6533b170f76) --- source4/libnet/libnet_user.c | 38 +++++++++++++++--------------- source4/libnet/prereq_domain.c | 53 +++++++++++++++++++++++------------------- 2 files changed, 48 insertions(+), 43 deletions(-) (limited to 'source4') diff --git a/source4/libnet/libnet_user.c b/source4/libnet/libnet_user.c index f8a47c721e..53a41dc1d5 100644 --- a/source4/libnet/libnet_user.c +++ b/source4/libnet/libnet_user.c @@ -63,7 +63,7 @@ struct composite_context* libnet_CreateUser_send(struct libnet_context *ctx, struct composite_context *c; struct create_user_state *s; struct composite_context *create_req; - struct composite_context *prereq_ctx; + BOOL prereq_met = False; /* composite context allocation and setup */ c = composite_create(mem_ctx, ctx->event_ctx); @@ -80,9 +80,9 @@ struct composite_context* libnet_CreateUser_send(struct libnet_context *ctx, ZERO_STRUCT(s->r.out); /* prerequisite: make sure the domain is opened */ - prereq_ctx = samr_domain_opened(ctx, s->r.in.domain_name, c, &s->domain_open, + prereq_met = samr_domain_opened(ctx, s->r.in.domain_name, &c, &s->domain_open, continue_domain_open_create, monitor); - if (prereq_ctx) return prereq_ctx; + if (!prereq_met) return c; /* prepare arguments for useradd call */ s->user_add.in.username = r->in.user_name; @@ -232,7 +232,7 @@ struct composite_context *libnet_DeleteUser_send(struct libnet_context *ctx, struct composite_context *c; struct delete_user_state *s; struct composite_context *delete_req; - struct composite_context *prereq_ctx; + BOOL prereq_met = False; /* composite context allocation and setup */ c = composite_create(mem_ctx, ctx->event_ctx); @@ -249,9 +249,9 @@ struct composite_context *libnet_DeleteUser_send(struct libnet_context *ctx, ZERO_STRUCT(s->r.out); /* prerequisite: make sure the domain is opened before proceeding */ - prereq_ctx = samr_domain_opened(ctx, s->r.in.domain_name, c, &s->domain_open, + prereq_met = samr_domain_opened(ctx, s->r.in.domain_name, &c, &s->domain_open, continue_domain_open_delete, monitor); - if (prereq_ctx) return prereq_ctx; + if (!prereq_met) return c; /* prepare arguments for userdel call */ s->user_del.in.username = r->in.user_name; @@ -402,8 +402,8 @@ struct composite_context *libnet_ModifyUser_send(struct libnet_context *ctx, const uint16_t level = 21; struct composite_context *c; struct modify_user_state *s; - struct composite_context *prereq_ctx; struct composite_context *userinfo_req; + BOOL prereq_met = False; c = composite_create(mem_ctx, ctx->event_ctx); if (c == NULL) return NULL; @@ -416,9 +416,9 @@ struct composite_context *libnet_ModifyUser_send(struct libnet_context *ctx, s->ctx = ctx; s->r = *r; - prereq_ctx = samr_domain_opened(ctx, s->r.in.domain_name, c, &s->domain_open, + prereq_met = samr_domain_opened(ctx, s->r.in.domain_name, &c, &s->domain_open, continue_domain_open_modify, monitor); - if (prereq_ctx) return prereq_ctx; + if (!prereq_met) return c; s->user_info.in.username = r->in.user_name; s->user_info.in.domain_handle = ctx->samr.handle; @@ -626,8 +626,8 @@ struct composite_context* libnet_UserInfo_send(struct libnet_context *ctx, { struct composite_context *c; struct user_info_state *s; - struct composite_context *prereq_ctx; struct composite_context *lookup_req; + BOOL prereq_met = False; /* composite context allocation and setup */ c = composite_create(mem_ctx, ctx->event_ctx); @@ -645,9 +645,9 @@ struct composite_context* libnet_UserInfo_send(struct libnet_context *ctx, s->user_name = talloc_strdup(c, r->in.user_name); /* prerequisite: make sure the domain is opened */ - prereq_ctx = samr_domain_opened(ctx, s->domain_name, c, &s->domopen, + prereq_met = samr_domain_opened(ctx, s->domain_name, &c, &s->domopen, continue_domain_open_info, monitor); - if (prereq_ctx) return prereq_ctx; + if (!prereq_met) return c; /* prepare arguments for LookupName call */ s->lookup.in.domain_name = s->domain_name; @@ -869,8 +869,8 @@ struct composite_context* libnet_UserList_send(struct libnet_context *ctx, { struct composite_context *c; struct userlist_state *s; - struct composite_context *prereq_ctx; struct rpc_request *query_req; + BOOL prereq_met = False; /* composite context allocation and setup */ c = composite_create(mem_ctx, ctx->event_ctx); @@ -889,9 +889,9 @@ struct composite_context* libnet_UserList_send(struct libnet_context *ctx, s->monitor_fn = monitor; /* make sure we have lsa domain handle before doing anything */ - prereq_ctx = lsa_domain_opened(ctx, s->domain_name, c, &s->domain_open, + prereq_met = lsa_domain_opened(ctx, s->domain_name, &c, &s->domain_open, continue_lsa_domain_opened, monitor); - if (prereq_ctx) return prereq_ctx; + if (!prereq_met) return c; /* prepare arguments of QueryDomainInfo call */ s->query_domain.in.handle = &ctx->lsa.handle; @@ -943,8 +943,8 @@ static void continue_domain_queried(struct rpc_request *req) { struct composite_context *c; struct userlist_state *s; - struct composite_context *prereq_ctx; struct rpc_request *enum_req; + BOOL prereq_met = False; c = talloc_get_type(req->async.private, struct composite_context); s = talloc_get_type(c->private_data, struct userlist_state); @@ -957,9 +957,9 @@ static void continue_domain_queried(struct rpc_request *req) s->dominfo = s->query_domain.out.info->domain; /* make sure we have samr domain handle before continuing */ - prereq_ctx = samr_domain_opened(s->ctx, s->domain_name, c, &s->domain_open, + prereq_met = samr_domain_opened(s->ctx, s->domain_name, &c, &s->domain_open, continue_samr_domain_opened, s->monitor_fn); - if (prereq_ctx) return; + if (!prereq_met) return; /* prepare arguments od EnumDomainUsers call */ s->user_list.in.domain_handle = &s->ctx->samr.handle; @@ -1027,7 +1027,7 @@ static void continue_users_enumerated(struct rpc_request *req) /* get the actual status of the rpc call result (instead of rpc layer status) */ c->status = s->user_list.out.result; - /* we're interested in status "ok" as well as two enum-specific statuses */ + /* we're interested in status "ok" as well as two enum-specific status codes */ if (NT_STATUS_IS_OK(c->status) || NT_STATUS_EQUAL(c->status, STATUS_MORE_ENTRIES) || NT_STATUS_EQUAL(c->status, NT_STATUS_NO_MORE_ENTRIES)) { diff --git a/source4/libnet/prereq_domain.c b/source4/libnet/prereq_domain.c index e02d7d43f4..73c1ba6437 100644 --- a/source4/libnet/prereq_domain.c +++ b/source4/libnet/prereq_domain.c @@ -30,15 +30,16 @@ #include "librpc/gen_ndr/ndr_lsa.h" -struct composite_context* samr_domain_opened(struct libnet_context *ctx, - const char *domain_name, - struct composite_context *parent_ctx, - struct libnet_DomainOpen *domain_open, - void (*continue_fn)(struct composite_context*), - void (*monitor)(struct monitor_msg*)) +BOOL samr_domain_opened(struct libnet_context *ctx, const char *domain_name, + struct composite_context **parent_ctx, + struct libnet_DomainOpen *domain_open, + void (*continue_fn)(struct composite_context*), + void (*monitor)(struct monitor_msg*)) { struct composite_context *domopen_req; + if (parent_ctx == NULL || *parent_ctx == NULL) return False; + if (domain_name == NULL) { /* * Try to guess the domain name from credentials, @@ -51,8 +52,8 @@ struct composite_context* samr_domain_opened(struct libnet_context *ctx, domain_open->in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED; } else { - composite_error(parent_ctx, NT_STATUS_INVALID_PARAMETER); - return parent_ctx; + composite_error(*parent_ctx, NT_STATUS_INVALID_PARAMETER); + return True; } } else { @@ -71,27 +72,28 @@ struct composite_context* samr_domain_opened(struct libnet_context *ctx, } else { /* domain has already been opened and it's the same domain as requested */ - return NULL; + return True; } } /* send request to open the domain */ domopen_req = libnet_DomainOpen_send(ctx, domain_open, monitor); - if (composite_nomem(domopen_req, parent_ctx)) return parent_ctx; + if (composite_nomem(domopen_req, *parent_ctx)) return False; - composite_continue(parent_ctx, domopen_req, continue_fn, parent_ctx); - return parent_ctx; + composite_continue(*parent_ctx, domopen_req, continue_fn, *parent_ctx); + return False; } -struct composite_context* lsa_domain_opened(struct libnet_context *ctx, - const char *domain_name, - struct composite_context *parent_ctx, - struct libnet_DomainOpen *domain_open, - void (*continue_fn)(struct composite_context*), - void (*monitor)(struct monitor_msg*)) +BOOL lsa_domain_opened(struct libnet_context *ctx, const char *domain_name, + struct composite_context **parent_ctx, + struct libnet_DomainOpen *domain_open, + void (*continue_fn)(struct composite_context*), + void (*monitor)(struct monitor_msg*)) { struct composite_context *domopen_req; + + if (parent_ctx == NULL || *parent_ctx == NULL) return False; if (domain_name == NULL) { /* @@ -105,8 +107,10 @@ struct composite_context* lsa_domain_opened(struct libnet_context *ctx, domain_open->in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED; } else { - composite_error(parent_ctx, NT_STATUS_INVALID_PARAMETER); - return parent_ctx; + composite_error(*parent_ctx, NT_STATUS_INVALID_PARAMETER); + /* this ensures the calling function exits and composite function error + gets noticed quickly */ + return True; } } else { @@ -125,14 +129,15 @@ struct composite_context* lsa_domain_opened(struct libnet_context *ctx, } else { /* domain has already been opened and it's the same domain as requested */ - return NULL; + return True; } } /* send request to open the domain */ domopen_req = libnet_DomainOpen_send(ctx, domain_open, monitor); - if (composite_nomem(domopen_req, parent_ctx)) return parent_ctx; + /* see the comment above to find out why true is returned here */ + if (composite_nomem(domopen_req, *parent_ctx)) return True; - composite_continue(parent_ctx, domopen_req, continue_fn, parent_ctx); - return parent_ctx; + composite_continue(*parent_ctx, domopen_req, continue_fn, *parent_ctx); + return False; } -- cgit