The branch, master has been updated
       via  accbd9ee1c6 Revert "s3:libsmb: add a cache for 
cli_session_creds_prepare_krb5()"
      from  0208d5f64b2 Add docs build to CI

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit accbd9ee1c6caad729b98471ea970df358f5ac0c
Author: Andreas Schneider <[email protected]>
Date:   Wed Jun 10 09:47:51 2020 +0200

    Revert "s3:libsmb: add a cache for cli_session_creds_prepare_krb5()"
    
    This reverts commit b458f8fbb7febf3584fa648128b4e1dc764059f7.
    
    Signed-off-by: Andreas Schneider <[email protected]>
    Reviewed-by: Stefan Metzmacher <[email protected]>
    
    Autobuild-User(master): Andreas Schneider <[email protected]>
    Autobuild-Date(master): Wed Jun 10 10:10:16 UTC 2020 on sn-devel-184

-----------------------------------------------------------------------

Summary of changes:
 source3/include/client.h    |  8 --------
 source3/libsmb/cliconnect.c | 14 --------------
 2 files changed, 22 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/include/client.h b/source3/include/client.h
index c9d1f00146e..6a3b1b02ff3 100644
--- a/source3/include/client.h
+++ b/source3/include/client.h
@@ -95,14 +95,6 @@ struct cli_state {
                struct smbXcli_tcon *tcon;
                struct idr_context *open_handles;
        } smb2;
-
-       /*
-        * We don't want to kinit twice if
-        * cli_session_creds_prepare_krb5()
-        * is called more than once for a given
-        * cli_state cli_credentials combination.
-        */
-       const struct cli_credentials *last_prepared_creds;
 };
 
 struct file_info {
diff --git a/source3/libsmb/cliconnect.c b/source3/libsmb/cliconnect.c
index c1be7fc3943..1fb1f0127b9 100644
--- a/source3/libsmb/cliconnect.c
+++ b/source3/libsmb/cliconnect.c
@@ -305,18 +305,6 @@ NTSTATUS cli_session_creds_prepare_krb5(struct cli_state 
*cli,
         * only if required!
         */
        setenv(KRB5_ENV_CCNAME, "MEMORY:cliconnect", 1);
-
-       if (cli->last_prepared_creds == creds) {
-               DBG_DEBUG("Re-using previously prepared Kerberos credentials "
-                         "for %s to access %s.\n",
-                         user_principal,
-                         target_hostname);
-               TALLOC_FREE(frame);
-               return NT_STATUS_OK;
-       }
-
-       cli->last_prepared_creds = NULL;
-
        ret = kerberos_kinit_password_ext(user_principal,
                                          pass,
                                          0,
@@ -374,8 +362,6 @@ NTSTATUS cli_session_creds_prepare_krb5(struct cli_state 
*cli,
                  canon_principal,
                  target_hostname);
 
-       cli->last_prepared_creds = creds;
-
        TALLOC_FREE(frame);
        return NT_STATUS_OK;
 }


-- 
Samba Shared Repository

Reply via email to