[28634] in CVS-changelog-for-Kerberos-V5

home help back first fref pref prev next nref lref last post

krb5 commit: Remove unused variables from kprop.c

daemon@ATHENA.MIT.EDU (Benjamin Kaduk)
Fri Oct 24 16:11:30 2014

Date: Fri, 24 Oct 2014 16:11:24 -0400
From: Benjamin Kaduk <kaduk@mit.edu>
Message-Id: <201410242011.s9OKBOYC002261@drugstore.mit.edu>
To: cvs-krb5@mit.edu
Reply-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu

https://github.com/krb5/krb5/commit/1bc97644eb689548126781645a83bb6093dfe79c
commit 1bc97644eb689548126781645a83bb6093dfe79c
Author: Ben Kaduk <kaduk@mit.edu>
Date:   Wed Oct 22 14:53:52 2014 -0400

    Remove unused variables from kprop.c
    
    Commit 29dee7d2cece615bec4616fa9b727e77210051db removed the
    need for a ccache to hold the credentials used by the process,
    but did not remove the ccname and ccache variables which became
    unused as a result.

 src/slave/kprop.c |   20 --------------------
 1 files changed, 0 insertions(+), 20 deletions(-)

diff --git a/src/slave/kprop.c b/src/slave/kprop.c
index d28a1af..696e89c 100644
--- a/src/slave/kprop.c
+++ b/src/slave/kprop.c
@@ -57,7 +57,6 @@ static char *file = KPROP_DEFAULT_FILE;
 /* The Kerberos principal we'll be sending as, initialized in get_tickets. */
 static krb5_principal my_principal;
 
-static krb5_ccache ccache;      /* Credentials cache which we'll be using */
 static krb5_creds creds;
 static krb5_address *sender_addr;
 static krb5_address *receiver_addr;
@@ -174,7 +173,6 @@ parse_args(int argc, char **argv)
 static void
 get_tickets(krb5_context context)
 {
-    char const ccname[] = "MEMORY:kpropcc";
     char *def_realm, *server;
     krb5_error_code retval;
     krb5_keytab keytab = NULL;
@@ -211,27 +209,12 @@ get_tickets(krb5_context context)
         }
     }
 
-    /* Use a memory cache to avoid possible filesystem conflicts. */
-    retval = krb5_cc_resolve(context, ccname, &ccache);
-    if (retval) {
-        com_err(progname, retval, _("while opening credential cache %s"),
-                ccname);
-        exit(1);
-    }
-
-    retval = krb5_cc_initialize(context, ccache, my_principal);
-    if (retval) {
-        com_err(progname, retval, _("when initializing cache %s"), ccname);
-        exit(1);
-    }
-
     /* Construct the principal name for the slave host. */
     memset(&creds, 0, sizeof(creds));
     retval = krb5_sname_to_principal(context, slave_host, KPROP_SERVICE_NAME,
                                      KRB5_NT_SRV_HST, &server_princ);
     if (retval) {
         com_err(progname, errno, _("while setting server principal name"));
-        krb5_cc_destroy(context, ccache);
         exit(1);
     }
     retval = krb5_unparse_name_flags(context, server_princ,
@@ -245,7 +228,6 @@ get_tickets(krb5_context context)
     retval = krb5_copy_principal(context, my_principal, &creds.client);
     if (retval) {
         com_err(progname, retval, _("while copying client principal"));
-        krb5_cc_destroy(context, ccache);
         exit(1);
     }
 
@@ -253,7 +235,6 @@ get_tickets(krb5_context context)
         retval = krb5_kt_resolve(context, srvtab, &keytab);
         if (retval) {
             com_err(progname, retval, _("while resolving keytab"));
-            krb5_cc_destroy(context, ccache);
             exit(1);
         }
     }
@@ -262,7 +243,6 @@ get_tickets(krb5_context context)
                                         0, server, NULL);
     if (retval) {
         com_err(progname, retval, _("while getting initial credentials\n"));
-        krb5_cc_destroy(context, ccache);
         exit(1);
     }
 
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5

home help back first fref pref prev next nref lref last post