summaryrefslogtreecommitdiff
path: root/source4/librpc
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2006-03-10 20:08:15 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 13:56:57 -0500
commit2b3767b1fe7744f0412531e1522e1d5af5dc39c6 (patch)
tree8f737c61ea156fa020b31c3c719ea332e3b65758 /source4/librpc
parent3452e89999296e4d9c724b96f712cc4777e95c22 (diff)
downloadsamba-2b3767b1fe7744f0412531e1522e1d5af5dc39c6.tar.gz
samba-2b3767b1fe7744f0412531e1522e1d5af5dc39c6.tar.bz2
samba-2b3767b1fe7744f0412531e1522e1d5af5dc39c6.zip
r14172: composite_is_ok(c) destroys c, when the status isn't ok.
so we need to return directly. metze (This used to be commit fba4229e56dce867a7c9daf9a3523a4dc3e454ba)
Diffstat (limited to 'source4/librpc')
-rw-r--r--source4/librpc/rpc/dcerpc_connect.c54
1 files changed, 8 insertions, 46 deletions
diff --git a/source4/librpc/rpc/dcerpc_connect.c b/source4/librpc/rpc/dcerpc_connect.c
index 417f3d3e49..973ddb9787 100644
--- a/source4/librpc/rpc/dcerpc_connect.c
+++ b/source4/librpc/rpc/dcerpc_connect.c
@@ -47,15 +47,10 @@ static void continue_pipe_open_smb(struct composite_context *ctx)
{
struct composite_context *c = talloc_get_type(ctx->async.private_data,
struct composite_context);
- struct pipe_np_smb_state *s = talloc_get_type(c->private_data,
- struct pipe_np_smb_state);
/* receive result of named pipe open request on smb */
c->status = dcerpc_pipe_open_smb_recv(ctx);
- if (!composite_is_ok(c)) {
- DEBUG(0,("Failed to open pipe %s - %s\n", s->io.pipe_name, nt_errstr(c->status)));
- return;
- }
+ if (!composite_is_ok(c)) return;
composite_done(c);
}
@@ -73,10 +68,7 @@ static void continue_smb_connect(struct composite_context *ctx)
/* receive result of smb connect request */
c->status = smb_composite_connect_recv(ctx, c);
- if (!composite_is_ok(c)) {
- DEBUG(0,("Failed to connect to %s - %s\n", s->io.binding->host, nt_errstr(c->status)));
- return;
- }
+ if (!composite_is_ok(c)) return;
/* prepare named pipe open parameters */
s->tree = s->conn.out.tree;
@@ -187,15 +179,10 @@ static void continue_pipe_open_smb2(struct composite_context *ctx)
{
struct composite_context *c = talloc_get_type(ctx->async.private_data,
struct composite_context);
- struct pipe_np_smb2_state *s = talloc_get_type(c->private_data,
- struct pipe_np_smb2_state);
/* receive result of named pipe open request on smb2 */
c->status = dcerpc_pipe_open_smb2_recv(ctx);
- if (!composite_is_ok(c)) {
- DEBUG(0,("Failed to open pipe %s - %s\n", s->io.pipe_name, nt_errstr(c->status)));
- return;
- }
+ if (!composite_is_ok(c)) return;
composite_done(c);
}
@@ -214,10 +201,7 @@ static void continue_smb2_connect(struct composite_context *ctx)
/* receive result of smb2 connect request */
c->status = smb2_connect_recv(ctx, c, &s->tree);
- if (!composite_is_ok(c)) {
- DEBUG(0,("Failed to connect to %s - %s\n", s->io.binding->host, nt_errstr(c->status)));
- return;
- }
+ if (!composite_is_ok(c)) return;
/* prepare named pipe open parameters */
s->io.pipe_name = s->io.binding->endpoint;
@@ -315,16 +299,10 @@ static void continue_pipe_open_ncacn_ip_tcp(struct composite_context *ctx)
{
struct composite_context *c = talloc_get_type(ctx->async.private_data,
struct composite_context);
- struct pipe_ip_tcp_state *s = talloc_get_type(c->private_data,
- struct pipe_ip_tcp_state);
/* receive result of named pipe open request on tcp/ip */
c->status = dcerpc_pipe_open_tcp_recv(ctx);
- if (!composite_is_ok(c)) {
- DEBUG(0,("Failed to connect to %s:%d - %s\n", s->host, s->port,
- nt_errstr(c->status)));
- return;
- }
+ if (!composite_is_ok(c)) return;
composite_done(c);
}
@@ -403,16 +381,10 @@ static void continue_pipe_open_ncacn_unix_stream(struct composite_context *ctx)
{
struct composite_context *c = talloc_get_type(ctx->async.private_data,
struct composite_context);
- struct pipe_unix_state *s = talloc_get_type(c->private_data,
- struct pipe_unix_state);
/* receive result of pipe open request on unix socket */
c->status = dcerpc_pipe_open_unix_stream_recv(ctx);
- if (!composite_is_ok(c)) {
- DEBUG(0,("Failed to open unix socket %s - %s\n",
- s->io.binding->endpoint, nt_errstr(c->status)));
- return;
- }
+ if (!composite_is_ok(c)) return;
composite_done(c);
}
@@ -497,16 +469,10 @@ static void continue_pipe_open_ncalrpc(struct composite_context *ctx)
{
struct composite_context *c = talloc_get_type(ctx->async.private_data,
struct composite_context);
- struct pipe_ncalrpc_state *s = talloc_get_type(c->private_data,
- struct pipe_ncalrpc_state);
/* receive result of pipe open request on ncalrpc */
c->status = dcerpc_pipe_connect_ncalrpc_recv(ctx);
- if (!composite_is_ok(c)) {
- DEBUG(0,("Failed to open ncalrpc pipe '%s' - %s\n", s->io.binding->endpoint,
- nt_errstr(c->status)));
- return;
- }
+ if (!composite_is_ok(c)) return;
composite_done(c);
}
@@ -599,11 +565,7 @@ static void continue_map_binding(struct composite_context *ctx)
struct pipe_connect_state);
c->status = dcerpc_epm_map_binding_recv(ctx);
- if (!composite_is_ok(c)) {
- DEBUG(0,("Failed to map DCERPC endpoint for '%s' - %s\n",
- GUID_string(c, &s->table->uuid), nt_errstr(c->status)));
- return;
- }
+ if (!composite_is_ok(c)) return;
DEBUG(2,("Mapped to DCERPC endpoint %s\n", s->binding->endpoint));