From 68d7a5e38346393a66e1d97da0348f474dd2ae4c Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Sun, 27 Mar 2005 06:26:00 +0000 Subject: [PATCH] r6078: Correctly fix the failures for NT1 (not SPNEGO) session setups in the client. The issue was actually a cut-and-paste bug, I was filling in the .old not the .nt1 part of the union. I've also removed the 'error checks' - I'll shortly document the API for the credentials code to clarify that it will always return a pointer here, except in cases of programmer error. Tridge: I hope this is OK. Andrew Bartlett (This used to be commit 6439de9ec8c8d24197ea69dc337473e54c8b36b8) --- source4/libcli/composite/sesssetup.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/source4/libcli/composite/sesssetup.c b/source4/libcli/composite/sesssetup.c index a33ace2420b..ab46c198ef3 100644 --- a/source4/libcli/composite/sesssetup.c +++ b/source4/libcli/composite/sesssetup.c @@ -180,12 +180,8 @@ static struct smbcli_request *session_setup_nt1(struct composite_context *c, state->setup.nt1.in.os = "Unix"; state->setup.nt1.in.lanman = "Samba"; state->setup.nt1.in.user = cli_credentials_get_username(io->in.credentials); - if (state->setup.nt1.in.user == NULL) return NULL; state->setup.nt1.in.domain = cli_credentials_get_domain(io->in.credentials); - if (state->setup.nt1.in.domain == NULL) return NULL; - state->setup.old.in.domain = cli_credentials_get_domain(io->in.credentials); - state->setup.old.in.user = cli_credentials_get_username(io->in.credentials); if (!password) { state->setup.nt1.in.password1 = data_blob(NULL, 0); state->setup.nt1.in.password2 = data_blob(NULL, 0);