summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2008-02-11 18:36:06 +0100
committerVolker Lendecke <vl@samba.org>2008-02-11 18:37:58 +0100
commitb4dd60efa9d59dc70f0f0e58be6fa769c543d059 (patch)
tree21ab2d53b13f61c00a5758ac27333219d62229f1
parentde48cd925a11414063cae7ead89fa060c280352a (diff)
downloadsamba-b4dd60efa9d59dc70f0f0e58be6fa769c543d059.tar.gz
samba-b4dd60efa9d59dc70f0f0e58be6fa769c543d059.tar.bz2
samba-b4dd60efa9d59dc70f0f0e58be6fa769c543d059.zip
Add a missing return
If I'm not completely blind, we should return here. Not doing it here seems not to be a major flaw, as far as I can see we're only missing the error code. This might account for some of the very unhelpful NT_STATUS_UNSUCCESSFUL error messages people see during joins. All with stake in Samba client, please check! (This used to be commit eadd15c9363a57c214ede3c489057646baca48f8)
-rw-r--r--source3/libsmb/cliconnect.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/libsmb/cliconnect.c b/source3/libsmb/cliconnect.c
index f3926b777b..e97be98fc1 100644
--- a/source3/libsmb/cliconnect.c
+++ b/source3/libsmb/cliconnect.c
@@ -627,7 +627,7 @@ static ADS_STATUS cli_session_setup_kerberos(struct cli_state *cli, const char *
if (!cli_session_setup_blob(cli, negTokenTarg, session_key_krb5)) {
data_blob_free(&negTokenTarg);
data_blob_free(&session_key_krb5);
- ADS_ERROR_NT(cli_nt_error(cli));
+ return ADS_ERROR_NT(cli_nt_error(cli));
}
cli_set_session_key(cli, session_key_krb5);