New GetToken pioctl
[openafs.git] / src / aklog / aklog.c
index 43cd846..df81a75 100644 (file)
@@ -1,37 +1,37 @@
-/* 
+/*
  * $Id$
  *
  * Copyright 1990,1991 by the Massachusetts Institute of Technology
  * For distribution and copying rights, see the file "mit-copyright.h"
  */
 /*
- * Copyright (c) 2005, 2006                                     
- * The Linux Box Corporation                                    
- * ALL RIGHTS RESERVED                                          
- *                                                              
- * Permission is granted to use, copy, create derivative works  
- * and redistribute this software and such derivative works     
- * for any purpose, so long as the name of the Linux Box        
- * Corporation is not used in any advertising or publicity      
- * pertaining to the use or distribution of this software       
- * without specific, written prior authorization.  If the       
- * above copyright notice or any other identification of the    
- * Linux Box Corporation is included in any copy of any         
- * portion of this software, then the disclaimer below must     
- * also be included.                                            
- *                                                              
- * This software is provided as is, without representation      
- * from the Linux Box Corporation as to its fitness for any     
- * purpose, and without warranty by the Linux Box Corporation   
- * of any kind, either express or implied, including            
- * without limitation the implied warranties of                 
- * merchantability and fitness for a particular purpose.  The   
- * regents of the Linux Box Corporation shall not be liable     
- * for any damages, including special, indirect, incidental, or 
- * consequential damages, with respect to any claim arising     
- * out of or in connection with the use of the software, even   
- * if it has been or is hereafter advised of the possibility of 
- * such damages.                                                
+ * Copyright (c) 2005, 2006
+ * The Linux Box Corporation
+ * ALL RIGHTS RESERVED
+ *
+ * Permission is granted to use, copy, create derivative works
+ * and redistribute this software and such derivative works
+ * for any purpose, so long as the name of the Linux Box
+ * Corporation is not used in any advertising or publicity
+ * pertaining to the use or distribution of this software
+ * without specific, written prior authorization.  If the
+ * above copyright notice or any other identification of the
+ * Linux Box Corporation is included in any copy of any
+ * portion of this software, then the disclaimer below must
+ * also be included.
+ *
+ * This software is provided as is, without representation
+ * from the Linux Box Corporation as to its fitness for any
+ * purpose, and without warranty by the Linux Box Corporation
+ * of any kind, either express or implied, including
+ * without limitation the implied warranties of
+ * merchantability and fitness for a particular purpose.  The
+ * regents of the Linux Box Corporation shall not be liable
+ * for any damages, including special, indirect, incidental, or
+ * consequential damages, with respect to any claim arising
+ * out of or in connection with the use of the software, even
+ * if it has been or is hereafter advised of the possibility of
+ * such damages.
  */
 
 #include <afsconfig.h>
@@ -63,6 +63,9 @@
 #include <pwd.h>
 
 #include <afs/stds.h>
+#include <afs/ktc.h>
+#include <afs/token.h>
+
 #include <krb5.h>
 #if defined(HAVE_ET_COM_ERR_H)
 #include <et/com_err.h>
@@ -188,7 +191,7 @@ static int get_user_realm(krb5_context, char **);
 #error "Must have either krb5_princ_size or krb5_principal_get_comp_string"
 #endif
 
-#if !defined(HAVE_KRB5_ENCRYPT_TKT_PART) && defined(HAVE_ENCODE_KRB5_ENC_TKT_PART) && defined(HAVE_KRB5_C_ENCRYPT) 
+#if !defined(HAVE_KRB5_ENCRYPT_TKT_PART) && defined(HAVE_ENCODE_KRB5_ENC_TKT_PART) && defined(HAVE_KRB5_C_ENCRYPT)
 extern krb5_error_code encode_krb5_enc_tkt_part (const krb5_enc_tkt_part *rep,
                                                 krb5_data **code);
 
@@ -200,7 +203,7 @@ krb5_encrypt_tkt_part(krb5_context context,
     krb5_data *data = 0;
     int code;
     size_t enclen;
-    
+
     if ((code = encode_krb5_enc_tkt_part(ticket->enc_part2, &data)))
        goto Done;
     if ((code = krb5_c_encrypt_length(context, key->enctype,
@@ -369,7 +372,7 @@ copy_cellinfo(cellinfo_t *cellinfo)
 
     if ((new_cellinfo = (cellinfo_t *)malloc(sizeof(cellinfo_t))))
        memcpy(new_cellinfo, cellinfo, sizeof(cellinfo_t));
-    
+
     return ((char *)new_cellinfo);
 }
 
@@ -390,7 +393,7 @@ get_cellconfig(char *cell, struct afsconf_cell *cellconfig, char **local_cell)
     }
 
     if (!(configdir = afsconf_Open(AFSDIR_CLIENT_ETC_DIRPATH))) {
-       fprintf(stderr, 
+       fprintf(stderr,
                "%s: can't get afs configuration (afsconf_Open(%s))\n",
                progname, AFSDIR_CLIENT_ETC_DIRPATH);
        exit(AKLOG_AFS);
@@ -671,16 +674,18 @@ out:
  */
 static int
 rxkad_build_native_token(krb5_context context, krb5_creds *v5cred,
-                        struct ktc_token **tokenPtr, char **userPtr) {
-    char k4name[ANAME_SZ];
-    char k4inst[INST_SZ];
-    char k4realm[REALM_SZ];
+                        struct ktc_tokenUnion **tokenPtr, char **userPtr) {
     char username[BUFSIZ];
-    struct ktc_token *token;
-    int status;
+    struct ktc_tokenUnion *token;
+    struct token_rxkad *rxkadToken;
 #ifdef HAVE_NO_KRB5_524
     char *p;
     int len;
+#else
+    int status;
+    char k4name[ANAME_SZ];
+    char k4inst[INST_SZ];
+    char k4realm[REALM_SZ];
 #endif
 
     afs_dprintf("Using Kerberos V5 ticket natively\n");
@@ -720,19 +725,28 @@ rxkad_build_native_token(krb5_context context, krb5_creds *v5cred,
     }
 #endif
 
-    token = malloc(sizeof(struct ktc_token));
+    token = malloc(sizeof(struct ktc_tokenUnion));
     if (token == NULL)
        return ENOMEM;
 
-    memset(token, 0, sizeof(struct ktc_token));
+    memset(token, 0, sizeof(struct ktc_tokenUnion));
 
-    token->kvno = RXKAD_TKT_TYPE_KERBEROS_V5;
-    token->startTime = v5cred->times.starttime;;
-    token->endTime = v5cred->times.endtime;
-    memcpy(&token->sessionKey, get_cred_keydata(v5cred),
+    token->at_type = AFSTOKEN_UNION_KAD;
+    rxkadToken = &token->ktc_tokenUnion_u.at_kad;
+
+    rxkadToken->rk_kvno = RXKAD_TKT_TYPE_KERBEROS_V5;
+    rxkadToken->rk_begintime = v5cred->times.starttime;;
+    rxkadToken->rk_endtime = v5cred->times.endtime;
+    memcpy(&rxkadToken->rk_key, get_cred_keydata(v5cred),
           get_cred_keylen(v5cred));
-    token->ticketLen = v5cred->ticket.length;
-    memcpy(token->ticket, v5cred->ticket.data, token->ticketLen);
+    rxkadToken->rk_ticket.rk_ticket_len = v5cred->ticket.length;
+    rxkadToken->rk_ticket.rk_ticket_val = malloc(v5cred->ticket.length);
+    if (rxkadToken->rk_ticket.rk_ticket_val == NULL) {
+       free(token);
+       return ENOMEM;
+    }
+    memcpy(rxkadToken->rk_ticket.rk_ticket_val, v5cred->ticket.data,
+          rxkadToken->rk_ticket.rk_ticket_len);
 
     *tokenPtr = token;
     *userPtr = strdup(username);
@@ -764,7 +778,7 @@ rxkad_build_native_token(krb5_context context, krb5_creds *v5cred,
 #ifdef HAVE_NO_KRB5_524
 static int
 rxkad_get_converted_token(krb5_context context, krb5_creds *v5cred,
-                         struct ktc_token **tokenPtr, char **userPtr) {
+                         struct ktc_tokenUnion **tokenPtr, char **userPtr) {
     *tokenPtr = NULL;
     *userPtr = NULL;
 
@@ -773,10 +787,11 @@ rxkad_get_converted_token(krb5_context context, krb5_creds *v5cred,
 #else
 static int
 rxkad_get_converted_token(krb5_context context, krb5_creds *v5cred,
-                         struct ktc_token **tokenPtr, char **userPtr) {
+                         struct ktc_tokenUnion **tokenPtr, char **userPtr) {
     CREDENTIALS cred;
     char username[BUFSIZ];
-    struct ktc_token *token;
+    struct ktc_tokenUnion *token;
+    struct token_rxkad *rxkadToken;
     int status;
 
     *tokenPtr = NULL;
@@ -798,11 +813,16 @@ rxkad_get_converted_token(krb5_context context, krb5_creds *v5cred,
        strcat (username, cred.pinst);
     }
 
-    token = malloc(sizeof(struct ktc_token));
-    memset(token, 0, sizeof(struct ktc_token));
+    token = malloc(sizeof(struct ktc_tokenUnion));
+    if (token == NULL)
+       return ENOMEM;
+    memset(token, 0, sizeof(struct ktc_tokenUnion));
+
+    token->at_type = AFSTOKEN_UNION_KAD;
 
-    token->kvno = cred.kvno;
-    token->startTime = cred.issue_date;
+    rxkadToken = &token->ktc_tokenUnion_u.at_kad;
+    rxkadToken->rk_kvno = cred.kvno;
+    rxkadToken->rk_begintime = cred.issue_date;
     /*
      * It seems silly to go through a bunch of contortions to
      * extract the expiration time, when the v5 credentials already
@@ -811,10 +831,19 @@ rxkad_get_converted_token(krb5_context context, krb5_creds *v5cred,
      * Note that this isn't a security hole, as the expiration time
      * is also contained in the encrypted token
      */
-    token->endTime = v5cred->times.endtime;
-    memcpy(&token->sessionKey, cred.session, 8);
-    token->ticketLen = cred.ticket_st.length;
-    memcpy(token->ticket, cred.ticket_st.dat, token->ticketLen);
+    rxkadToken->rk_endtime = v5cred->times.endtime;
+    memcpy(&rxkadToken->rk_key, cred.session, 8);
+    rxkadToken->rk_ticket.rk_ticket_len = cred.ticket_st.length;
+    rxkadToken->rk_ticket.rk_ticket_val = malloc(cred.ticket_st.length);
+    if (rxkadToken->rk_ticket.rk_ticket_val == NULL) {
+       free(token);
+       return ENOMEM;
+    }
+    memcpy(rxkadToken->rk_ticket.rk_ticket_val, cred.ticket_st.dat,
+          rxkadToken->rk_ticket.rk_ticket_len);
+
+    *tokenPtr = token;
+    *userPtr = strdup(username);
 
     return 0;
 }
@@ -845,7 +874,7 @@ rxkad_get_converted_token(krb5_context context, krb5_creds *v5cred,
  */
 static int
 rxkad_get_token(krb5_context context, struct afsconf_cell *cell, char *realm,
-               struct ktc_token **token, char **authuser, int *foreign) {
+               struct ktc_tokenUnion **token, char **authuser, int *foreign) {
     krb5_creds *v5cred;
     char *realmUsed = NULL;
     char *username = NULL;
@@ -890,61 +919,6 @@ out:
     return status;
 }
 
-static int
-get_kernel_token(struct afsconf_cell *cell, struct ktc_token **tokenPtr) {
-    struct ktc_principal client, server;
-    struct ktc_token *token;
-    int ret;
-
-    *tokenPtr = NULL;
-
-    strncpy(server.name, AFSKEY, MAXKTCNAMELEN - 1);
-    strncpy(server.instance, AFSINST, MAXKTCNAMELEN - 1);
-    strncpy(server.cell, cell->name, MAXKTCREALMLEN - 1);
-
-    token = malloc(sizeof(struct ktc_token));
-    if (token == NULL)
-       return ENOMEM;
-
-    memset(token, 0, sizeof(struct ktc_token));
-
-    ret = ktc_GetToken(&server, token, sizeof(struct ktc_token), &client);
-    if (ret) {
-       free(token);
-       return ret;
-    }
-
-    *tokenPtr = token;
-    return 0;
-}
-
-static int
-set_kernel_token(struct afsconf_cell *cell, char *username,
-                struct ktc_token *token, int setpag)
-{
-    struct ktc_principal client, server;
-
-    strncpy(client.name, username, MAXKTCNAMELEN - 1);
-    strcpy(client.instance, "");
-    strncpy(client.cell, cell->name, MAXKTCREALMLEN - 1);
-
-    strncpy(server.name, AFSKEY, MAXKTCNAMELEN - 1);
-    strncpy(server.instance, AFSINST, MAXKTCNAMELEN - 1);
-    strncpy(server.cell, cell->name, MAXKTCREALMLEN - 1);
-
-    return ktc_SetToken(&server, token, &client, setpag);
-}
-
-static int
-tokens_equal(struct ktc_token *tokenA, struct ktc_token *tokenB) {
-    return (tokenA != NULL && tokenB != NULL &&
-           tokenA->kvno == tokenB->kvno &&
-           tokenA->ticketLen == tokenB->ticketLen &&
-           !memcmp(&tokenA->sessionKey, &tokenB->sessionKey,
-                   sizeof(tokenA->sessionKey)) &&
-           !memcmp(tokenA->ticket, tokenB->ticket, tokenA->ticketLen));
-}
-
 /* 
  * Log to a cell.  If the cell has already been logged to, return without
  * doing anything.  Otherwise, log to it and mark that it has been logged
@@ -959,19 +933,24 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
     afs_int32 viceId;          /* AFS uid of user */
 
     char *local_cell = NULL;
-    struct ktc_token *token;
-    struct ktc_token *btoken;
+    struct ktc_tokenUnion *rxkadToken = NULL;
+    struct ktc_setTokenData *token;
+    struct ktc_setTokenData *btoken = NULL;
     struct afsconf_cell cellconf;
 
     /* NULL or empty cell returns information on local cell */
     if ((status = get_cellconfig(cell, &cellconf, &local_cell)))
        return(status);
 
-    if (linkedcell != NULL && cellconf.linkedCell != NULL) {
-       *linkedcell = strdup(cellconf.linkedCell);
-       if (*linkedcell == NULL) {
-           status = ENOMEM;
-           goto out;
+    if (linkedcell != NULL) {
+       if (cellconf.linkedCell != NULL) {
+           *linkedcell = strdup(cellconf.linkedCell);
+           if (*linkedcell == NULL) {
+               status = ENOMEM;
+               goto out;
+           }
+       } else {
+           *linkedcell = NULL;
        }
     }
 
@@ -981,14 +960,14 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
        goto out;
     }
 
-    /* 
+    /*
      * Record that we have attempted to log to this cell.  We do this
      * before we try rather than after so that we will not try
      * and fail repeatedly for one cell.
      */
     ll_string(&authedcells, ll_s_add, cellconf.name);
 
-    /* 
+    /*
      * Record this cell in the list of zephyr subscriptions.  We may
      * want zephyr subscriptions even if authentication fails.
      * If this is done after we attempt to get tokens, aklog -zsubs
@@ -996,13 +975,13 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
      * are in -noauth mode.
      */
     if (ll_string(&zsublist, ll_s_add, cellconf.name) == LL_FAILURE) {
-       fprintf(stderr, 
+       fprintf(stderr,
                "%s: failure adding cell %s to zephyr subscriptions list.\n",
                progname, cellconf.name);
        exit(AKLOG_MISC);
     }
     if (ll_string(&zsublist, ll_s_add, local_cell) == LL_FAILURE) {
-       fprintf(stderr, 
+       fprintf(stderr,
                "%s: failure adding cell %s to zephyr subscriptions list.\n",
                progname, local_cell);
        exit(AKLOG_MISC);
@@ -1012,20 +991,38 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
        afs_dprintf("Authenticating to cell %s (server %s).\n", cellconf.name,
                cellconf.hostName[0]);
 
-       status = rxkad_get_token(context, &cellconf, realm, &token,
+       token = token_buildTokenJar(cellconf.name);
+       if (token == NULL) {
+           status = ENOMEM;
+           goto out;
+       }
+
+       status = rxkad_get_token(context, &cellconf, realm, &rxkadToken,
                                 &username, &isForeign);
        if (status)
-           return status;
+           goto out;
 
+       /* We need to keep the token structure around so that we can stick
+        * the viceId into it (once we know it) */
+       status = token_addToken(token, rxkadToken);
+       if (status) {
+           afs_dprintf("Add Token failed with %d", status);
+           goto out;
+       }
 
        if (!force &&
-           !get_kernel_token(&cellconf, &btoken) &&
-           tokens_equal(token, btoken)) {
+           ktc_GetTokenEx(cellconf.name, &btoken) == 0 &&
+           token_SetsEquivalent(token, btoken)) {
+
+           token_FreeSet(&btoken);
            afs_dprintf("Identical tokens already exist; skipping.\n");
            status = AKLOG_SUCCESS;
            goto out;
        }
 
+       if (btoken)
+           token_FreeSet(&btoken);
+
 #ifdef FORCE_NOPRDB
        noprdb = 1;
 #endif
@@ -1044,7 +1041,7 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
                afs_dprintf("Error %d\n", status);
            else
                afs_dprintf("Id %d\n", (int) viceId);
-           
+
 
            /*
             * This code is taken from cklog -- it lets people
@@ -1057,7 +1054,7 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
                        username, cellconf.name);
                viceId = 0;
 
-               status = set_kernel_token(&cellconf, username, token, 0);
+               status = ktc_SetTokenEx(token);
                if (status) {
                    afs_com_err(progname, status,
                                "while obtaining tokens for cell %s",
@@ -1068,7 +1065,7 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
                /*
                 * In case you're wondering, we don't need to change the
                 * filename here because we're still connecting to the
-                * same cell -- we're just using a different authentication
+                * same cell -- we're just using a different authenticat ion
                 * level
                 */
 
@@ -1090,40 +1087,26 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
            }
 #endif /* ALLOW_REGISTER */
 
-           /*
-            * This is a crock, but it is Transarc's crock, so we have to play
-            * along in order to get the functionality.  The way the afs id is
-            * stored is as a string in the username field of the token.
-            * Contrary to what you may think by looking at the code for
-            * tokens, this hack (AFS ID %d) will not work if you change %d
-            * to something else.
-            */
-
            if ((status == 0) && (viceId != ANONYMOUSID)) {
-               free(username);
-               if (afs_asprintf(&username, "AFS ID %d", (int) viceId) < 0) {
-                   status = ENOMEM;
-                   username = NULL;
-                   goto out;
-               }
+               rxkadToken->ktc_tokenUnion_u.at_kad.rk_viceid = viceId;
+               token_replaceToken(token, rxkadToken);
            }
        }
 
-       afs_dprintf("Set username to %s\n", username);
-
        afs_dprintf("Setting tokens. %s @ %s \n", username, cellconf.name);
 
 #ifndef AFS_AIX51_ENV
        /* on AIX 4.1.4 with AFS 3.4a+ if a write is not done before
-        * this routine, it will not add the token. It is not clear what 
+        * this routine, it will not add the token. It is not clear what
         * is going on here! So we will do the following operation.
         * On AIX 5, it causes the parent program to die, so we won't.
         */
        write(2,"",0); /* dummy write */
 #endif
-       status = set_kernel_token(&cellconf, username, token, afssetpag);
+       token_setPag(token, afssetpag);
+       status = ktc_SetTokenEx(token);
        if (status) {
-           afs_com_err(progname, status, "while obtaining tokens for cell %s",
+           afs_com_err(progname, status, "while setting tokens for cell %s",
                        cellconf.name);
            status = AKLOG_TOKEN;
        }
@@ -1132,6 +1115,11 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
        afs_dprintf("Noauth mode; not authenticating.\n");
 
 out:
+    if (rxkadToken) {
+       free(rxkadToken->ktc_tokenUnion_u.at_kad.rk_ticket.rk_ticket_val);
+       free(rxkadToken);
+    }
+
     if (local_cell)
        free(local_cell);
     if (username)
@@ -1162,8 +1150,8 @@ get_afs_mountpoint(char *file, char *mountpoint, int size)
     else {
        last_component = our_file;
        parent_dir = ".";
-    }    
-    
+    }
+
     memset(cellname, 0, sizeof(cellname));
 
     vio.in = last_component;
@@ -1177,7 +1165,7 @@ get_afs_mountpoint(char *file, char *mountpoint, int size)
            vio.in_size = strlen(file) + 1;
            vio.out_size = sizeof(cellname);
            vio.out = cellname;
-           
+
            if (!pioctl(file, VIOC_FILE_CELL_NAME, &vio, 1)) {
                strcat(cellname, VOLMARKERSTRING);
                strcat(cellname, mountpoint + 1);
@@ -1191,8 +1179,8 @@ get_afs_mountpoint(char *file, char *mountpoint, int size)
        return(FALSE);
 }
 
-/* 
- * This routine each time it is called returns the next directory 
+/*
+ * This routine each time it is called returns the next directory
  * down a pathname.  It resolves all symbolic links.  The first time
  * it is called, it should be called with the name of the path
  * to be descended.  After that, it should be called with the arguemnt
@@ -1212,9 +1200,9 @@ next_path(char *origpath)
     static char *elast_comp;   /* End of last component */
     char *t;
     int len;
-    
+
     static int symlinkcount = 0; /* We can't exceed MAXSYMLINKS */
-    
+
     /* If we are given something for origpath, we are initializing only. */
     if (origpath) {
        memset(path, 0, sizeof(path));
@@ -1226,7 +1214,7 @@ next_path(char *origpath)
     }
 
     /* We were not given origpath; find then next path to check */
-    
+
     /* If we've gotten all the way through already, return NULL */
     if (last_comp == NULL)
        return(NULL);
@@ -1234,11 +1222,11 @@ next_path(char *origpath)
     do {
        while (*last_comp == DIR)
            strncat(pathtocheck, last_comp++, 1);
-       len = (elast_comp = strchr(last_comp, DIR)) 
+       len = (elast_comp = strchr(last_comp, DIR))
            ? elast_comp - last_comp : strlen(last_comp);
        strncat(pathtocheck, last_comp, len);
        memset(linkbuf, 0, sizeof(linkbuf));
-       if ((link = (readlink(pathtocheck, linkbuf, 
+       if ((link = (readlink(pathtocheck, linkbuf,
                                    sizeof(linkbuf)) > 0))) {
            if (++symlinkcount > MAXSYMLINKS) {
                fprintf(stderr, "%s: %s\n", progname, strerror(ELOOP));
@@ -1248,8 +1236,8 @@ next_path(char *origpath)
            if (elast_comp)
                strcpy(tmpbuf, elast_comp);
            if (linkbuf[0] == DIR) {
-               /* 
-                * If this is a symbolic link to an absolute path, 
+               /*
+                * If this is a symbolic link to an absolute path,
                 * replace what we have by the absolute path.
                 */
                memset(path, 0, strlen(path));
@@ -1260,8 +1248,8 @@ next_path(char *origpath)
                memset(pathtocheck, 0, sizeof(pathtocheck));
            }
            else {
-               /* 
-                * If this is a symbolic link to a relative path, 
+               /*
+                * If this is a symbolic link to a relative path,
                 * replace only the last component with the link name.
                 */
                strncpy(last_comp, linkbuf, strlen(linkbuf) + 1);
@@ -1295,7 +1283,7 @@ add_hosts(char *file)
     int i;
     struct hostent *hp;
     struct in_addr in;
-    
+
     memset(outbuf, 0, sizeof(outbuf));
 
     vio.out_size = sizeof(outbuf);
@@ -1317,7 +1305,7 @@ add_hosts(char *file)
         * If it is replicated on many servers, then none are included.
         * This is not perfect, but the result is that people don't
         * get subscribed to a lot of instances of FILSRV that they
-        * probably won't need which reduces the instances of 
+        * probably won't need which reduces the instances of
         * people getting messages that don't apply to them.
         */
 #ifndef ALLHOSTS
@@ -1339,7 +1327,7 @@ add_hosts(char *file)
 }
 
 /*
- * This routine descends through a path to a directory, logging to 
+ * This routine descends through a path to a directory, logging to
  * every cell it encounters along the way.
  */
 static int
@@ -1378,9 +1366,9 @@ auth_to_path(krb5_context context, char *path)
     while ((nextpath = next_path(NULL))) {
        strcpy(pathtocheck, nextpath);
        afs_dprintf("Checking directory %s\n", pathtocheck);
-       /* 
-        * If this is an afs mountpoint, determine what cell from 
-        * the mountpoint name which is of the form 
+       /*
+        * If this is an afs mountpoint, determine what cell from
+        * the mountpoint name which is of the form
         * #cellname:volumename or %cellname:volumename.
         */
        if (get_afs_mountpoint(pathtocheck, mountpoint, sizeof(mountpoint))) {
@@ -1405,9 +1393,9 @@ auth_to_path(krb5_context context, char *path)
            if (isdir(pathtocheck, &isdirectory) < 0) {
                /*
                 * If we've logged and still can't stat, there's
-                * a problem... 
+                * a problem...
                 */
-               fprintf(stderr, "%s: stat(%s): %s\n", progname, 
+               fprintf(stderr, "%s: stat(%s): %s\n", progname,
                        pathtocheck, strerror(errno));
                return(AKLOG_BADPATH);
            }
@@ -1419,7 +1407,7 @@ auth_to_path(krb5_context context, char *path)
            }
        }
     }
-    
+
 
     return(status);
 }
@@ -1528,7 +1516,7 @@ main(int argc, char *argv[])
     for (i = 1; i < argc; i++) {
        if (strcmp(argv[i], "-d") == 0)
            dflag++;
-       else if (strcmp(argv[i], "-noauth") == 0) 
+       else if (strcmp(argv[i], "-noauth") == 0)
            noauth++;
        else if (strcmp(argv[i], "-zsubs") == 0)
            zsubs++;
@@ -1574,7 +1562,7 @@ main(int argc, char *argv[])
            }
            else
                usage();
-           
+
        else if (argv[i][0] == '-')
            usage();
        else if (!pmode && !cmode) {
@@ -1583,7 +1571,7 @@ main(int argc, char *argv[])
                pmode++;
                strcpy(path, argv[i]);
            }
-           else { 
+           else {
                cmode++;
                strcpy(cell, argv[i]);
            }
@@ -1607,7 +1595,7 @@ main(int argc, char *argv[])
                if ((new_cellinfo = copy_cellinfo(&cellinfo)))
                    ll_add_data(cur_node, new_cellinfo);
                else {
-                   fprintf(stderr, 
+                   fprintf(stderr,
                            "%s: failure copying cellinfo.\n", progname);
                    exit(AKLOG_MISC);
                }
@@ -1649,7 +1637,7 @@ main(int argc, char *argv[])
        struct passwd *pwd;
 
        status = auth_to_cell(context, NULL, NULL, &linkedcell);
-       
+
        /* If this cell is linked to a DCE cell, and user requested -linked,
         * get tokens for both. This is very useful when the AFS cell is
         * linked to a DFS cell and this system does not also have DFS.
@@ -1726,9 +1714,9 @@ main(int argc, char *argv[])
            if ((status = auth_to_path(context, cur_node->data)))
                somethingswrong++;
        }
-       
-       /* 
-        * If only one thing was logged to, we'll return the status 
+
+       /*
+        * If only one thing was logged to, we'll return the status
         * of the single call.  Otherwise, we'll return a generic
         * something failed status.
         */
@@ -1737,7 +1725,7 @@ main(int argc, char *argv[])
     }
 
     /* If we are keeping track of zephyr subscriptions, print them. */
-    if (zsubs) 
+    if (zsubs)
        for (cur_node = zsublist.first; cur_node; cur_node = cur_node->next) {
            printf("zsub: %s\n", cur_node->data);
        }
@@ -1759,12 +1747,12 @@ isdir(char *path, unsigned char *val)
     if (lstat(path, &statbuf) < 0)
        return (-1);
     else {
-       if ((statbuf.st_mode & S_IFMT) == S_IFDIR) 
+       if ((statbuf.st_mode & S_IFMT) == S_IFDIR)
            *val = TRUE;
        else
            *val = FALSE;
        return (0);
-    }  
+    }
 }
 
 static krb5_error_code
@@ -1799,8 +1787,8 @@ get_credv5_akimpersonate(krb5_context context,
     krb5_ticket ticket_reply[1];
     krb5_enc_tkt_part enc_tkt_reply[1];
     krb5_address address[30], *faddr[30];
-#endif
     krb5_data * temp;
+#endif
     int i;
     static int any_enctype[] = {0};
     *out_creds = 0;
@@ -1923,7 +1911,7 @@ get_credv5_akimpersonate(krb5_context context,
     enc_tkt_reply->times.starttime = starttime; /* krb524init needs this */
     enc_tkt_reply->times.endtime = endtime;
 #endif  /* USING_HEIMDAL */
-    /* NB:  We will discard address for now--ignoring caddr field               
+    /* NB:  We will discard address for now--ignoring caddr field
        in any case.  MIT branch does what it always did. */
 
     if (paddress && *paddress) {
@@ -2081,7 +2069,6 @@ cleanup:
         krb5_kt_close(context, kt);
     if (creds) krb5_free_creds(context, creds);
     krb5_free_keyblock_contents(context, session_key);
-out:
     return code;
 #else
     return -1;
@@ -2099,7 +2086,7 @@ get_credv5(krb5_context context, char *name, char *inst, char *realm,
 
     afs_dprintf("Getting tickets: %s%s%s@%s\n", name,
            (inst && inst[0]) ? "/" : "", inst ? inst : "", realm);
-    
+
     memset(&increds, 0, sizeof(increds));
 /* ANL - instance may be ptr to a null string. Pass null then */
     if ((r = krb5_build_principal(context, &increds.server,
@@ -2109,7 +2096,7 @@ get_credv5(krb5_context context, char *name, char *inst, char *realm,
                                  (void *) NULL))) {
         return r;
     }
-    
+
 
     if (!_krb425_ccache) {
         r = krb5_cc_default(context, &_krb425_ccache);
@@ -2125,12 +2112,12 @@ get_credv5(krb5_context context, char *name, char *inst, char *realm,
        if (r)
            return r;
     }
-    
+
     increds.client = client_principal;
     increds.times.endtime = 0;
     /* Ask for DES since that is what V4 understands */
     get_creds_enctype((&increds)) = ENCTYPE_DES_CBC_CRC;
-    
+
     if (keytab) {
        int allowed_enctypes[] = {
            ENCTYPE_DES_CBC_CRC, 0
@@ -2146,7 +2133,7 @@ get_credv5(krb5_context context, char *name, char *inst, char *realm,
                                     creds /* out */);
     } else {
        r = krb5_get_credentials(context, 0, _krb425_ccache, &increds, creds);
-    }    
+    }
     return r;
 }