diff options
author | Volker Lendecke <vl@samba.org> | 2008-02-11 18:36:06 +0100 |
---|---|---|
committer | Volker Lendecke <vl@samba.org> | 2008-02-11 18:37:58 +0100 |
commit | b4dd60efa9d59dc70f0f0e58be6fa769c543d059 (patch) | |
tree | 21ab2d53b13f61c00a5758ac27333219d62229f1 | |
parent | de48cd925a11414063cae7ead89fa060c280352a (diff) | |
download | samba-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.c | 2 |
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); |