summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2008-08-01 15:53:01 +0200
committerStefan Metzmacher <metze@samba.org>2008-08-01 15:54:57 +0200
commitf1cdce59632b893be89f784f2fb41b28f3a87170 (patch)
treee9376cf80d03282b14ef749a01ce4b38a94225d4
parent5d68244a27c7fcdb2024d94593db30d04cc61386 (diff)
downloadsamba-f1cdce59632b893be89f784f2fb41b28f3a87170.tar.gz
samba-f1cdce59632b893be89f784f2fb41b28f3a87170.tar.bz2
samba-f1cdce59632b893be89f784f2fb41b28f3a87170.zip
auth_server: fix the logic of server_get_challenge()
metze (This used to be commit 699e3cdb52acdf2524347d8c053730306c579dd9)
-rw-r--r--source4/auth/ntlm/auth_server.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/source4/auth/ntlm/auth_server.c b/source4/auth/ntlm/auth_server.c
index 7efbdb3050..42606c14ba 100644
--- a/source4/auth/ntlm/auth_server.c
+++ b/source4/auth/ntlm/auth_server.c
@@ -79,10 +79,10 @@ static NTSTATUS server_get_challenge(struct auth_method_context *ctx, TALLOC_CTX
status = smb_composite_connect(&io, mem_ctx, lp_resolve_context(ctx->auth_ctx->lp_ctx),
ctx->auth_ctx->event_ctx);
- if (!NT_STATUS_IS_OK(status)) {
- *_blob = io.out.tree->session->transport->negotiate.secblob;
- ctx->private_data = talloc_steal(ctx, io.out.tree->session);
- }
+ NT_STATUS_NOT_OK_RETURN(status);
+
+ *_blob = io.out.tree->session->transport->negotiate.secblob;
+ ctx->private_data = talloc_steal(ctx, io.out.tree->session);
return NT_STATUS_OK;
}