The branch, v3-2-test has been updated via 5804d8b112e1da022988c635284eb4799974d4c7 (commit) via 87d8fc338f6e6b48691bff3eeebfc00c5d408ff7 (commit) from d9f19fc61586d606393368799dee9757c169d602 (commit)
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-2-test - Log ----------------------------------------------------------------- commit 5804d8b112e1da022988c635284eb4799974d4c7 Author: Günther Deschner <[EMAIL PROTECTED]> Date: Thu Apr 10 22:44:00 2008 +0200 Use libnetapi_open_pipe in netapi functions. Guenther commit 87d8fc338f6e6b48691bff3eeebfc00c5d408ff7 Author: Günther Deschner <[EMAIL PROTECTED]> Date: Thu Apr 10 22:42:07 2008 +0200 Add libnetapi_open_pipe, inspired by the cli_cm_ interface. Guenther ----------------------------------------------------------------------- Summary of changes: source/lib/netapi/cm.c | 110 ++++++++++++++++++++++++++++++++++++ source/lib/netapi/getdc.c | 20 ++---- source/lib/netapi/joindomain.c | 24 +++----- source/lib/netapi/netapi_private.h | 4 + source/lib/netapi/serverinfo.c | 16 ++--- source/lib/netapi/user.c | 15 ++--- 6 files changed, 141 insertions(+), 48 deletions(-) Changeset truncated at 500 lines: diff --git a/source/lib/netapi/cm.c b/source/lib/netapi/cm.c index a3b842f..9608724 100644 --- a/source/lib/netapi/cm.c +++ b/source/lib/netapi/cm.c @@ -73,3 +73,113 @@ WERROR libnetapi_shutdown_cm(struct libnetapi_ctx *ctx) return WERR_OK; } + +/******************************************************************** +********************************************************************/ + +struct client_pipe_connection { + struct client_pipe_connection *prev, *next; + struct rpc_pipe_client *pipe; +}; + +static struct client_pipe_connection *pipe_connections; + +/******************************************************************** +********************************************************************/ + +static struct rpc_pipe_client *pipe_cm_find(struct cli_state *cli, + int pipe_idx, + NTSTATUS *status) +{ + struct client_pipe_connection *p; + + for (p = pipe_connections; p; p = p->next) { + + if (!p->pipe->cli) { + *status = NT_STATUS_PIPE_EMPTY; + return NULL; + } + + if (strequal(cli->desthost, p->pipe->cli->desthost) && + pipe_idx == p->pipe->pipe_idx) { + *status = NT_STATUS_OK; + return p->pipe; + } + } + + *status = NT_STATUS_PIPE_NOT_AVAILABLE; + + return NULL; +} + +/******************************************************************** +********************************************************************/ + +static struct rpc_pipe_client *pipe_cm_connect(TALLOC_CTX *mem_ctx, + struct cli_state *cli, + int pipe_idx, + NTSTATUS *status) +{ + struct client_pipe_connection *p; + + p = TALLOC_ZERO_ARRAY(mem_ctx, struct client_pipe_connection, 1); + if (!p) { + *status = NT_STATUS_NO_MEMORY; + return NULL; + } + + p->pipe = cli_rpc_pipe_open_noauth(cli, pipe_idx, status); + if (!p->pipe) { + TALLOC_FREE(p); + return NULL; + } + + DLIST_ADD(pipe_connections, p); + + return p->pipe; +} + +/******************************************************************** +********************************************************************/ + +static struct rpc_pipe_client *pipe_cm_open(TALLOC_CTX *ctx, + struct cli_state *cli, + int pipe_idx, + NTSTATUS *status) +{ + struct rpc_pipe_client *p; + + p = pipe_cm_find(cli, pipe_idx, status); + if (!p) { + p = pipe_cm_connect(ctx, cli, pipe_idx, status); + } + + return p; +} + +/******************************************************************** +********************************************************************/ + +WERROR libnetapi_open_pipe(struct libnetapi_ctx *ctx, + struct cli_state *cli, + int pipe_idx, + struct rpc_pipe_client **pipe_cli) +{ + NTSTATUS status; + + if (!cli || !pipe_cli) { + return WERR_INVALID_PARAM; + } + + *pipe_cli = pipe_cm_open(ctx, cli, pipe_idx, &status); + if (!*pipe_cli) { + libnetapi_set_error_string(ctx, "failed to open PIPE %s: %s", + cli_get_pipe_name(pipe_idx), + get_friendly_nt_error_msg(status)); + return WERR_DEST_NOT_FOUND; + } + + return WERR_OK; +} + + diff --git a/source/lib/netapi/getdc.c b/source/lib/netapi/getdc.c index 8f88294..38aaf0e 100644 --- a/source/lib/netapi/getdc.c +++ b/source/lib/netapi/getdc.c @@ -50,10 +50,8 @@ WERROR NetGetDCName_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_NETLOGON, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_NETLOGON, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } @@ -92,12 +90,10 @@ WERROR NetGetAnyDCName_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_NETLOGON, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_NETLOGON, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; - }; + } status = rpccli_netr_GetAnyDCName(pipe_cli, ctx, r->in.server_name, @@ -152,10 +148,8 @@ WERROR DsGetDcName_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_NETLOGON, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_NETLOGON, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } diff --git a/source/lib/netapi/joindomain.c b/source/lib/netapi/joindomain.c index 056d8d2..8e0a62e 100644 --- a/source/lib/netapi/joindomain.c +++ b/source/lib/netapi/joindomain.c @@ -108,10 +108,8 @@ WERROR NetJoinDomain_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_WKSSVC, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_WKSSVC, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } @@ -239,10 +237,8 @@ WERROR NetUnjoinDomain_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_WKSSVC, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_WKSSVC, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } @@ -292,10 +288,8 @@ WERROR NetGetJoinInformation_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_WKSSVC, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_WKSSVC, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } @@ -422,10 +416,8 @@ WERROR NetGetJoinableOUs_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_WKSSVC, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_WKSSVC, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } diff --git a/source/lib/netapi/netapi_private.h b/source/lib/netapi/netapi_private.h index 2591a7b..a575f42 100644 --- a/source/lib/netapi/netapi_private.h +++ b/source/lib/netapi/netapi_private.h @@ -29,4 +29,8 @@ WERROR libnetapi_open_ipc_connection(struct libnetapi_ctx *ctx, const char *server_name, struct cli_state **cli); WERROR libnetapi_shutdown_cm(struct libnetapi_ctx *ctx); +WERROR libnetapi_open_pipe(struct libnetapi_ctx *ctx, + struct cli_state *cli, + int pipe_idx, + struct rpc_pipe_client **pipe_cli); #endif diff --git a/source/lib/netapi/serverinfo.c b/source/lib/netapi/serverinfo.c index 238b9ca..58efeb3 100644 --- a/source/lib/netapi/serverinfo.c +++ b/source/lib/netapi/serverinfo.c @@ -75,12 +75,10 @@ WERROR NetServerGetInfo_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_SRVSVC, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_SRVSVC, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; - }; + } status = rpccli_srvsvc_NetSrvGetInfo(pipe_cli, ctx, r->in.server_name, @@ -177,12 +175,10 @@ WERROR NetServerSetInfo_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_SRVSVC, - &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_SRVSVC, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; - }; + } switch (r->in.level) { case 1005: diff --git a/source/lib/netapi/user.c b/source/lib/netapi/user.c index d9505bc..31e5faa 100644 --- a/source/lib/netapi/user.c +++ b/source/lib/netapi/user.c @@ -185,9 +185,8 @@ WERROR NetUserAdd_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_SAMR, &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_SAMR, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } @@ -378,9 +377,8 @@ WERROR NetUserDel_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_SAMR, &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_SAMR, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } @@ -602,9 +600,8 @@ WERROR NetUserEnum_r(struct libnetapi_ctx *ctx, goto done; } - pipe_cli = cli_rpc_pipe_open_noauth(cli, PI_SAMR, &status); - if (!pipe_cli) { - werr = ntstatus_to_werror(status); + werr = libnetapi_open_pipe(ctx, cli, PI_SAMR, &pipe_cli); + if (!W_ERROR_IS_OK(werr)) { goto done; } -- Samba Shared Repository