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!

diff --git a/source/libsmb/cliconnect.c b/source/libsmb/cliconnect.c
index f3926b777b92e9381e416200fe7fbb80427eb3f1..e97be98fc19dc1de3c41e3478f3ff4eb571e1c1f 100644 (file)
--- a/source/libsmb/cliconnect.c
+++ b/source/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);