aklog: Fix improper use of readlink
[openafs.git] / src / aklog / aklog.c
index ecb1cb7..22df70d 100644 (file)
@@ -1,68 +1,50 @@
-/* 
+/*
  * $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>
+#include <afs/param.h>
+#include <afs/stds.h>
 
-#include <stdio.h>
-#include <string.h>
-#include <ctype.h>
-#include <sys/types.h>
-#ifdef HAVE_UNISTD_H
-#include <unistd.h>
-#endif
-#ifdef HAVE_STDLIB_H
-#include <stdlib.h>
-#endif
-#ifdef HAVE_MEMORY_H
-#include <memory.h>
-#endif
-#include <errno.h>
+#include <roken.h>
 
-#include <sys/stat.h>
-#include <fcntl.h>
+#include <ctype.h>
 
-#include <sys/param.h>
-#include <sys/errno.h>
-#include <netdb.h>
-#include <sys/socket.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
-#include <pwd.h>
+#include <afs/ktc.h>
+#include <afs/token.h>
 
-#include <afs/stds.h>
 #include <krb5.h>
 #if defined(HAVE_ET_COM_ERR_H)
 #include <et/com_err.h>
 #include <afs/com_err.h>
 #endif
 
-#include <afs/param.h>
 #ifdef AFS_SUN5_ENV
 #include <sys/ioccom.h>
 #endif
 
-/* Prevent inclusion of des.h to avoid conflicts with des types */
-#define NO_DES_H_INCLUDE
-
 #include <afs/auth.h>
 #include <afs/cellconfig.h>
 #include <afs/vice.h>
 #include <afs/venus.h>
 #include <afs/ptserver.h>
 #include <afs/ptuser.h>
+#include <afs/pterror.h>
 #include <afs/dirpath.h>
+#include <afs/afsutil.h>
 
 #include "aklog.h"
 #include "linked_list.h"
 #define AKLOG_BADPATH 6
 #define AKLOG_MISC 7
 
-#ifndef NULL
-#define NULL 0
-#endif
-
 #ifndef TRUE
 #define TRUE 1
 #endif
@@ -186,7 +162,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);
 
@@ -198,7 +174,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,
@@ -240,10 +216,11 @@ Done:
 #error "Must have either keyblock or session member of krb5_creds"
 #endif
 
-#if !defined(HAVE_KRB5_524_CONVERT_CREDS) && defined(HAVE_KRB524_CONVERT_CREDS_KDC)
-#define krb5_524_convert_creds krb524_convert_creds_kdc
-#elif !defined(HAVE_KRB5_524_CONVERT_CREDS) && !defined(HAVE_KRB524_CONVERT_CREDS_KDC)
+/* MITKerberosShim logs but returns success */
+#if !defined(HAVE_KRB5_524_CONV_PRINCIPAL) || defined(AFS_DARWIN110_ENV) || (!defined(HAVE_KRB5_524_CONVERT_CREDS) && !defined(HAVE_KRB524_CONVERT_CREDS_KDC))
 #define HAVE_NO_KRB5_524
+#elif !defined(HAVE_KRB5_524_CONVERT_CREDS) && defined(HAVE_KRB524_CONVERT_CREDS_KDC)
+#define krb5_524_convert_creds krb524_convert_creds_kdc
 #endif
 
 #if USING_HEIMDAL
@@ -331,10 +308,22 @@ redirect_errors(const char *who, afs_int32 code, const char *fmt, va_list ap)
     if (code) {
        const char *str = afs_error_message(code);
        if (strncmp(str, "unknown", strlen("unknown")) == 0) {
-           str = error_message(code);
+#ifdef HAVE_KRB5_SVC_GET_MSG
+           krb5_svc_get_msg(code,&str);
+#elif defined(HAVE_KRB5_GET_ERROR_MESSAGE)
+           krb5_context context;
+           krb5_init_context(&context);
+           str = krb5_get_error_message(context, code);
+           krb5_free_context(context);
+#else
+           ; /* IRIX apparently has neither: use the string we have */
+#endif
        }
        fputs(str, stderr);
        fputs(" ", stderr);
+#ifdef HAVE_KRB5_SVC_GET_MSG
+       krb5_free_string(str);
+#endif
     }
     if (fmt) {
        vfprintf(stderr, fmt, ap);
@@ -343,20 +332,31 @@ redirect_errors(const char *who, afs_int32 code, const char *fmt, va_list ap)
     fflush(stderr);
 }
 
+static void
+afs_dprintf(char *fmt, ...) {
+    va_list ap;
+
+    va_start(ap, fmt);
+    if (dflag)
+       vprintf(fmt, ap);
+    va_end(ap);
+}
+
 static char *
 copy_cellinfo(cellinfo_t *cellinfo)
 {
     cellinfo_t *new_cellinfo;
 
-    if ((new_cellinfo = (cellinfo_t *)malloc(sizeof(cellinfo_t))))
+    if ((new_cellinfo = malloc(sizeof(cellinfo_t))))
        memcpy(new_cellinfo, cellinfo, sizeof(cellinfo_t));
-    
+
     return ((char *)new_cellinfo);
 }
 
 
 static int
-get_cellconfig(char *cell, struct afsconf_cell *cellconfig, char **local_cell)
+get_cellconfig(const char *config, char *cell,
+              struct afsconf_cell *cellconfig, char **local_cell)
 {
     int status = AKLOG_SUCCESS;
     struct afsconf_dir *configdir;
@@ -370,10 +370,10 @@ get_cellconfig(char *cell, struct afsconf_cell *cellconfig, char **local_cell)
        exit(AKLOG_AFS);
     }
 
-    if (!(configdir = afsconf_Open(AFSDIR_CLIENT_ETC_DIRPATH))) {
-       fprintf(stderr, 
+    if (!(configdir = afsconf_Open(config))) {
+       fprintf(stderr,
                "%s: can't get afs configuration (afsconf_Open(%s))\n",
-               progname, AFSDIR_CLIENT_ETC_DIRPATH);
+               progname, config);
        exit(AKLOG_AFS);
     }
 
@@ -441,59 +441,495 @@ get_realm_from_cred(krb5_context context, krb5_creds *v5cred, char **realm) {
 #endif
 }
 
+/*!
+ * Get a Kerberos service ticket to use as the base of an rxkad token for
+ * a given AFS cell.
+ *
+ * @param[in] context
+ *     An initialized Kerberos v5 context
+ * @param[in] realm
+ *     The realm to look in for the service principal. If NULL, then the
+ *     realm is determined from the cell name or the user's credentials
+ *     (see below for the heuristics used)
+ * @param[in] cell
+ *     The cell information for the cell to obtain a ticket for
+ * @param[out] v5cred
+ *     A Kerberos credentials structure containing the ticket acquired
+ *     for the cell. This is a dynamically allocated structure, which
+ *     should be freed by using the appropriate Kerberos API function.
+ * @param[out] realmUsed
+ *     The realm in which the cell's service principal was located. If
+ *     unset, then the principal was located in the same realm as the
+ *     current user. This is a malloc'd string which should be freed
+ *     by the caller.
+ *
+ * @returns
+ *     0 on success, an error value upon failure
+ *
+ * @notes
+ *     This code tries principals in the following, much debated,
+ *     order:
+ *
+ *     If the realm is specified on the command line we do
+ *        - afs/cell@COMMAND-LINE-REALM
+ *        - afs@COMMAND-LINE-REALM
+ *
+ *     Otherwise, we do
+ *        - afs/cell@REALM-FROM-USERS-PRINCIPAL
+ *        - afs/cell@krb5_get_host_realm(db-server)
+ *       Then, if krb5_get_host_realm(db-server) is non-empty
+ *          - afs@ krb5_get_host_realm(db-server)
+ *       Otherwise
+ *          - afs/cell@ upper-case-domain-of-db-server
+ *          - afs@ upper-case-domain-of-db-server
+ *
+ *     In all cases, the 'afs@' variant is only tried where the
+ *     cell and the realm match case-insensitively.
+ */
+
+static int
+rxkad_get_ticket(krb5_context context, char *realm,
+                struct afsconf_cell *cell,
+                krb5_creds **v5cred, char **realmUsed) {
+    char *realm_of_cell = NULL;
+    char *realm_of_user = NULL;
+    char *realm_from_princ = NULL;
+    int status;
+    int retry;
+
+    *realmUsed = NULL;
+
+    if ((status = get_user_realm(context, &realm_of_user))) {
+       fprintf(stderr, "%s: Couldn't determine realm of user:", progname);
+       afs_com_err(progname, status, " while getting realm");
+       status = AKLOG_KERBEROS;
+       goto out;
+    }
+
+    retry = 1;
+
+    while(retry) {
+       /* Cell on command line - use that one */
+       if (realm && realm[0]) {
+           realm_of_cell = realm;
+           status = AKLOG_TRYAGAIN;
+           afs_dprintf("We were told to authenticate to realm %s.\n", realm);
+       } else {
+           /* Initially, try using afs/cell@USERREALM */
+           afs_dprintf("Trying to authenticate to user's realm %s.\n",
+                   realm_of_user);
+           realm_of_cell = realm_of_user;
+           status = get_credv5(context, AFSKEY, cell->name, realm_of_cell,
+                               v5cred);
+
+           /* If that failed, try to determine the realm from the name of
+            * one of the DB servers */
+           if (TRYAGAIN(status)) {
+               realm_of_cell = afs_realm_of_cell(context, cell, FALSE);
+               if (!realm_of_cell) {
+                   fprintf(stderr, "%s: Couldn't figure out realm for cell "
+                           "%s.\n", progname, cell->name);
+                   exit(AKLOG_MISC);
+               }
+
+               if (realm_of_cell[0])
+                   afs_dprintf("We've deduced that we need to authenticate"
+                           " to realm %s.\n", realm_of_cell);
+                   else
+                   afs_dprintf("We've deduced that we need to authenticate "
+                           "using referrals.\n");
+           }
+       }
+
+       if (TRYAGAIN(status)) {
+           /* If we've got the full-princ-first option, or we're in a
+            * different realm from the cell - use the cell name as the
+            * instance */
+           if (AFS_TRY_FULL_PRINC ||
+               strcasecmp(cell->name, realm_of_cell)!=0) {
+               status = get_credv5(context, AFSKEY, cell->name,
+                                   realm_of_cell, v5cred);
+
+               /* If we failed & we've got an empty realm, then try
+                * calling afs_realm_for_cell again. */
+               if (TRYAGAIN(status) && !realm_of_cell[0]) {
+                   /* This time, get the realm by taking the domain
+                    * component of the db server and make it upper case */
+                   realm_of_cell = afs_realm_of_cell(context, cell, TRUE);
+                   if (!realm_of_cell) {
+                       fprintf(stderr,
+                               "%s: Couldn't figure out realm for cell %s.\n",
+                               progname, cell->name);
+                       exit(AKLOG_MISC);
+                   }
+                   afs_dprintf("We've deduced that we need to authenticate"
+                           " to realm %s.\n", realm_of_cell);
+               }
+               status = get_credv5(context, AFSKEY, cell->name,
+                                   realm_of_cell, v5cred);
+           }
+
+           /* If the realm and cell name match, then try without an
+            * instance, but only if realm is non-empty */
+
+           if (TRYAGAIN(status) &&
+               strcasecmp(cell->name, realm_of_cell) == 0) {
+               status = get_credv5(context, AFSKEY, NULL, realm_of_cell,
+                                   v5cred);
+               if (!AFS_TRY_FULL_PRINC && TRYAGAIN(status)) {
+                   status = get_credv5(context, AFSKEY, cell->name,
+                                       realm_of_cell, v5cred);
+               }
+           }
+       }
+
+       /* Try to find a service principal for this cell.
+        * Some broken MIT libraries return KRB5KRB_AP_ERR_MSG_TYPE upon
+        * the first attempt, so we try twice to be sure */
+
+       if (status == KRB5KRB_AP_ERR_MSG_TYPE && retry == 1)
+           retry++;
+       else
+           retry = 0;
+    }
+
+    if (status != 0) {
+       afs_dprintf("Kerberos error code returned by get_cred : %d\n", status);
+       fprintf(stderr, "%s: Couldn't get %s AFS tickets:\n",
+               progname, cell->name);
+       afs_com_err(progname, status, "while getting AFS tickets");
+#ifdef KRB5_CC_NOT_KTYPE
+       if (status == KRB5_CC_NOT_KTYPE) {
+           fprintf(stderr, "allow_weak_crypto may be required in the Kerberos configuration\n");
+       }
+#endif
+       status = AKLOG_KERBEROS;
+       goto out;
+    }
+
+    /* If we've got a valid ticket, and we still don't know the realm name
+     * try to figure it out from the contents of the ticket
+     */
+    if (strcmp(realm_of_cell, "") == 0) {
+       status = get_realm_from_cred(context, *v5cred, &realm_from_princ);
+       if (status) {
+           fprintf(stderr,
+                   "%s: Couldn't decode ticket to determine realm for "
+                   "cell %s.\n",
+                   progname, cell->name);
+       } else {
+           if (realm_from_princ)
+               realm_of_cell = realm_from_princ;
+       }
+    }
+
+    /* If the realm of the user and cell differ, then we need to use the
+     * realm when we later construct the user's principal */
+    if (realm_of_cell != NULL && strcmp(realm_of_user, realm_of_cell) != 0)
+       *realmUsed = realm_of_user;
+
+out:
+    if (realm_from_princ)
+       free(realm_from_princ);
+    if (realm_of_user && *realmUsed == NULL)
+       free(realm_of_user);
+
+    return status;
+}
+
+/*!
+ * Build an rxkad token from a Kerberos ticket, using only local tools (that
+ * is, without using a 524 conversion service)
+ *
+ * @param[in] context
+ *     An initialised Kerberos 5 context
+ * @param[in] v5cred
+ *     A Kerberos credentials structure containing a suitable service ticket
+ * @param[out] tokenPtr
+ *     An AFS token structure containing an rxkad token. This is a malloc'd
+ *     structure which should be freed by the caller.
+ * @param[out[ userPtr
+ *     A string containing the principal of the user to whom the token was
+ *     issued. This is a malloc'd block which should be freed by the caller.
+ *
+ * @returns
+ *     0 on success, an error value upon failure
+ */
+static int
+rxkad_build_native_token(krb5_context context, krb5_creds *v5cred,
+                        struct ktc_tokenUnion **tokenPtr, char **userPtr) {
+    char username[BUFSIZ];
+    struct ktc_token token;
+    int status;
+#ifdef HAVE_NO_KRB5_524
+    char *p;
+    int len;
+#else
+    char k4name[ANAME_SZ];
+    char k4inst[INST_SZ];
+    char k4realm[REALM_SZ];
+#endif
+
+    afs_dprintf("Using Kerberos V5 ticket natively\n");
+
+    *tokenPtr = NULL;
+    *userPtr = NULL;
+
+#ifndef HAVE_NO_KRB5_524
+    status = krb5_524_conv_principal (context, v5cred->client,
+                                     (char *) &k4name,
+                                     (char *) &k4inst,
+                                     (char *) &k4realm);
+    if (status) {
+       afs_com_err(progname, status, "while converting principal "
+                   "to Kerberos V4 format");
+       return AKLOG_KERBEROS;
+    }
+    strcpy (username, k4name);
+    if (k4inst[0]) {
+       strcat (username, ".");
+       strcat (username, k4inst);
+    }
+#else
+    len = min(get_princ_len(context, v5cred->client, 0),
+             second_comp(context, v5cred->client) ?
+             MAXKTCNAMELEN - 2 : MAXKTCNAMELEN - 1);
+    strncpy(username, get_princ_str(context, v5cred->client, 0), len);
+    username[len] = '\0';
+
+    if (second_comp(context, v5cred->client)) {
+       strcat(username, ".");
+       p = username + strlen(username);
+       len = min(get_princ_len(context, v5cred->client, 1),
+                 MAXKTCNAMELEN - strlen(username) - 1);
+       strncpy(p, get_princ_str(context, v5cred->client, 1), len);
+       p[len] = '\0';
+    }
+#endif
+
+    memset(&token, 0, sizeof(struct ktc_token));
+
+    token.kvno = RXKAD_TKT_TYPE_KERBEROS_V5;
+    token.startTime = v5cred->times.starttime;;
+    token.endTime = v5cred->times.endtime;
+    memcpy(&token.sessionKey, get_cred_keydata(v5cred),
+          get_cred_keylen(v5cred));
+    token.ticketLen = v5cred->ticket.length;
+    memcpy(token.ticket, v5cred->ticket.data, token.ticketLen);
+
+    status = token_importRxkadViceId(tokenPtr, &token, 0);
+    if (status) {
+       return status;
+    }
+
+    *userPtr = strdup(username);
+
+    return 0;
+}
+
+/*!
+ * Convert a Keberos ticket to an rxkad token, using information obtained
+ * from an external Kerberos 5->4 conversion service. If the code is built
+ * with HAVE_NO_KRB5_524 then this is a stub function which will always
+ * return success without a token.
+ *
+ * @param[in] context
+ *     An initialised Kerberos 5 context
+ * @param[in] v5cred
+ *     A Kerberos credentials structure containing a suitable service ticket
+ * @param[out] tokenPtr
+ *     An AFS token structure containing an rxkad token. This is a malloc'd
+ *     structure which should be freed by the caller.
+ * @param[out[ userPtr
+ *     A string containing the principal of the user to whom the token was
+ *     issued. This is a malloc'd block which should be freed by the caller.
+ *
+ * @returns
+ *     0 on success, an error value upon failure
+ */
+
+#ifdef HAVE_NO_KRB5_524
+static int
+rxkad_get_converted_token(krb5_context context, krb5_creds *v5cred,
+                         struct ktc_tokenUnion **tokenPtr, char **userPtr) {
+    *tokenPtr = NULL;
+    *userPtr = NULL;
+
+    return 0;
+}
+#else
+static int
+rxkad_get_converted_token(krb5_context context, krb5_creds *v5cred,
+                         struct ktc_tokenUnion **tokenPtr, char **userPtr) {
+    CREDENTIALS cred;
+    char username[BUFSIZ];
+    struct ktc_token token;
+    int status;
+
+    *tokenPtr = NULL;
+    *userPtr = NULL;
+
+    afs_dprintf("Using Kerberos 524 translator service\n");
+
+    status = krb5_524_convert_creds(context, v5cred, &cred);
+
+    if (status) {
+       afs_com_err(progname, status, "while converting tickets "
+               "to Kerberos V4 format");
+       return AKLOG_KERBEROS;
+    }
+
+    strcpy (username, cred.pname);
+    if (cred.pinst[0]) {
+       strcat (username, ".");
+       strcat (username, cred.pinst);
+    }
+
+    memset(&token, 0, sizeof(struct ktc_token));
+
+    token.kvno = cred.kvno;
+    token.startTime = cred.issue_date;
+    /*
+     * It seems silly to go through a bunch of contortions to
+     * extract the expiration time, when the v5 credentials already
+     * has the exact time!  Let's use that instead.
+     *
+     * 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);
+
+    status = token_importRxkadViceId(tokenPtr, &token, 0);
+    if (status) {
+       return status;
+    }
+
+    *userPtr = strdup(username);
+
+    return 0;
+}
+#endif
+
+/*!
+ * This function gets an rxkad token for a given cell.
+ *
+ * @param[in] context
+ *     An initialized Kerberos v5 context
+ * @param[in] cell
+ *     The cell information for the cell which we're obtaining a token for
+ * @param[in] realm
+ *     The realm to look in for the service principal. If NULL, then the
+ *     realm is determined from the cell name or the user's credentials
+ *     (see the documentation for rxkad_get_ticket)
+ * @param[out] token
+ *     The rxkad token produced. This is a malloc'd structure which should
+ *     be freed by the caller.
+ * @parma[out] authuser
+ *     A string containing the principal of the user to whom the token was
+ *     issued. This is a malloc'd block which should be freed by the caller.
+ * @param[out] foreign
+ *     Whether the user is considered as 'foreign' to the realm of the cell.
+ *
+ * @returns
+ *     0 on success, an error value upon failuer
+ */
+static int
+rxkad_get_token(krb5_context context, struct afsconf_cell *cell, char *realm,
+               struct ktc_tokenUnion **token, char **authuser, int *foreign) {
+    krb5_creds *v5cred;
+    char *realmUsed = NULL;
+    char *username = NULL;
+    int status;
+
+    *token = NULL;
+    *authuser = NULL;
+    *foreign = 0;
+
+    status = rxkad_get_ticket(context, realm, cell, &v5cred, &realmUsed);
+    if (status)
+       return status;
+
+    if (do524)
+       status = rxkad_get_converted_token(context, v5cred, token, &username);
+    else
+       status = rxkad_build_native_token(context, v5cred, token, &username);
+
+    if (status)
+       goto out;
+
+    /* We now have the username, plus the realm name, so stitch them together
+     * to give us the name that the ptserver will know the user by */
+    if (realmUsed == NULL) {
+       *authuser = username;
+       username = NULL;
+       *foreign = 0;
+    } else {
+       asprintf(authuser, "%s@%s", username, realmUsed);
+       *foreign = 1;
+    }
+
+out:
+    if (realmUsed)
+       free(realmUsed);
+    if (username)
+       free(username);
+
+    return status;
+}
+
 /* 
  * 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
  * to.
  */
 static int
-auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
+auth_to_cell(krb5_context context, const char *config, 
+            char *cell, char *realm, char **linkedcell)
 {
     int status = AKLOG_SUCCESS;
-    char username[BUFSIZ];     /* To hold client username structure */
+    int isForeign = 0;
+    char *username = NULL;     /* To hold client username structure */
     afs_int32 viceId;          /* AFS uid of user */
 
-    char *realm_of_user = NULL;    /* Kerberos realm of user */
-    char *realm_from_princ = NULL; /* Calculated realm data */
-    char *realm_of_cell = NULL;    /* Pointer to realm we're using */
-    int retry;                    /* round, and round we go ... */
-    
     char *local_cell = NULL;
-    krb5_creds *v5cred = NULL;
-    struct ktc_principal aserver;
-    struct ktc_principal aclient;
-    struct ktc_token atoken, 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)))
+    if ((status = get_cellconfig(config, 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;
        }
     }
 
     if (ll_string(&authedcells, ll_s_check, cellconf.name)) {
-       if (dflag) {
-           printf("Already authenticated to %s (or tried to)\n", 
-                  cellconf.name);
-       }
+       afs_dprintf("Already authenticated to %s (or tried to)\n", cellconf.name);
        status = AKLOG_SUCCESS;
        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
@@ -501,326 +937,73 @@ 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);
     }
 
     if (!noauth) {
-       if (dflag) {
-           printf("Authenticating to cell %s (server %s).\n",
-                  cellconf.name, cellconf.hostName[0]);
-       }
+       afs_dprintf("Authenticating to cell %s (server %s).\n", cellconf.name,
+               cellconf.hostName[0]);
 
-       if ((status = get_user_realm(context, &realm_of_user))) {
-           fprintf(stderr, "%s: Couldn't determine realm of user:)",
-                   progname);
-           afs_com_err(progname, status, " while getting realm");
-           status = AKLOG_KERBEROS;
+       token = token_buildTokenJar(cellconf.name);
+       if (token == NULL) {
+           status = ENOMEM;
            goto out;
        }
 
-       retry = 1;
-       
-       while(retry) {
-
-           /* This code tries principals in the following, much debated,
-            * order:
-            * 
-            * If the realm is specified on the command line we do
-            *    - afs/cell@COMMAND-LINE-REALM
-            *    - afs@COMMAND-LINE-REALM
-            * 
-            * Otherwise, we do
-            *    - afs/cell@REALM-FROM-USERS-PRINCIPAL
-            *    - afs/cell@krb5_get_host_realm(db-server)
-            *   Then, if krb5_get_host_realm(db-server) is non-empty
-            *      - afs@ krb5_get_host_realm(db-server)
-            *   Otherwise
-            *      - afs/cell@ upper-case-domain-of-db-server
-            *      - afs@ upper-case-domain-of-db-server
-            * 
-            * In all cases, the 'afs@' variant is only tried where the
-            * cell and the realm match case-insensitively.
-            */
-               
-           /* Cell on command line - use that one */
-           if (realm && realm[0]) {
-               realm_of_cell = realm;
-               status = AKLOG_TRYAGAIN;
-               if (dflag) {
-                   printf("We were told to authenticate to realm %s.\n", 
-                          realm);
-               }
-           } else {
-               /* Initially, try using afs/cell@USERREALM */
-               if (dflag) {
-                   printf("Trying to authenticate to user's realm %s.\n",
-                          realm_of_user);
-               }
-               
-               realm_of_cell = realm_of_user;
-               status = get_credv5(context, AFSKEY, cellconf.name,
-                                   realm_of_cell, &v5cred);
-           
-               /* If that failed, try to determine the realm from the name of 
-                * one of the DB servers */
-               if (TRYAGAIN(status)) {
-                   realm_of_cell = afs_realm_of_cell(context, &cellconf,
-                                                     FALSE);
-                   if (!realm_of_cell) {
-                       fprintf(stderr, 
-                               "%s: Couldn't figure out realm for cell %s.\n",
-                               progname, cellconf.name);
-                       exit(AKLOG_MISC);
-                   }
-
-                   if (dflag) {
-                       if (realm_of_cell[0])
-                           printf("We've deduced that we need to authenticate"
-                                  " to realm %s.\n", realm_of_cell);
-                   else
-                       printf("We've deduced that we need to authenticate "
-                              "using referrals.\n");
-                   }
-               }
-           }
-       
-           if (TRYAGAIN(status)) {
-               /* If we've got the full-princ-first option, or we're in a
-                * different realm from the cell - use the cell name as the
-                * instance */
-               if (AFS_TRY_FULL_PRINC || 
-                   strcasecmp(cellconf.name, realm_of_cell)!=0) {
-                   status = get_credv5(context, AFSKEY, cellconf.name,
-                                       realm_of_cell, &v5cred);
-
-                   /* If we failed & we've got an empty realm, then try 
-                    * calling afs_realm_for_cell again. */
-                   if (TRYAGAIN(status) && !realm_of_cell[0]) {
-                       /* This time, get the realm by taking the domain 
-                        * component of the db server and make it upper case */
-                       realm_of_cell = afs_realm_of_cell(context,
-                                                         &cellconf, TRUE);
-                       if (!realm_of_cell) {
-                           fprintf(stderr,
-                                   "%s: Couldn't figure out realm for cell "
-                                   "%s.\n", progname, cellconf.name);
-                           exit(AKLOG_MISC);
-                       }
-                       if (dflag) {
-                           printf("We've deduced that we need to authenticate"
-                                  " to realm %s.\n", realm_of_cell);
-                       }
-                   }
-                   status = get_credv5(context, AFSKEY, cellconf.name,
-                                       realm_of_cell, &v5cred);
-               }
-          
-               /* If the realm and cell name match, then try without an 
-                * instance, but only if realm is non-empty */
-               
-               if (TRYAGAIN(status) && 
-                   strcasecmp(cellconf.name, realm_of_cell) == 0) {
-                   status = get_credv5(context, AFSKEY, NULL, 
-                                       realm_of_cell, &v5cred);
-                   if (!AFS_TRY_FULL_PRINC && TRYAGAIN(status)) {
-                       status = get_credv5(context, AFSKEY, cellconf.name,
-                                           realm_of_cell, &v5cred);
-                   }
-               }
-           }
-
-           /* Try to find a service principal for this cell.
-            * Some broken MIT libraries return KRB5KRB_AP_ERR_MSG_TYPE upon 
-            * the first attempt, so we try twice to be sure */
+       status = rxkad_get_token(context, &cellconf, realm, &rxkadToken,
+                                &username, &isForeign);
+       if (status)
+           goto out;
 
-           if (status == KRB5KRB_AP_ERR_MSG_TYPE && retry == 1)
-               retry++;
-           else
-               retry = 0;
-       } 
-       
-       if (status != 0) {
-           if (dflag) {
-               printf("Kerberos error code returned by get_cred : %d\n",
-                      status);
-           }
-           fprintf(stderr, "%s: Couldn't get %s AFS tickets:\n",
-                   progname, cellconf.name);
-           afs_com_err(progname, status, "while getting AFS tickets");
-           status = AKLOG_KERBEROS;
+       /* 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 we've got a valid ticket, and we still don't know the realm name
-        * try to figure it out from the contents of the ticket
-        */
-       if (strcmp(realm_of_cell, "") == 0) {
-           status = get_realm_from_cred(context, v5cred, &realm_from_princ);
-           if (status) {
-               fprintf(stderr,
-                       "%s: Couldn't decode ticket to determine realm for "
-                       "cell %s.\n",
-                       progname, cellconf.name);
-           } else {
-               if (realm_from_princ)
-                   realm_of_cell = realm_from_princ;
-           }
-       }
-
-       strncpy(aserver.name, AFSKEY, MAXKTCNAMELEN - 1);
-       strncpy(aserver.instance, AFSINST, MAXKTCNAMELEN - 1);
-       strncpy(aserver.cell, cellconf.name, MAXKTCREALMLEN - 1);
-
-       /*
-        * The default is to use rxkad2b, which means we put in a full
-        * V5 ticket.  If the user specifies -524, we talk to the
-        * 524 ticket converter.
-        */
-
-       if (! do524) {
-           char k4name[ANAME_SZ], k4inst[INST_SZ], k4realm[REALM_SZ];
-#ifdef HAVE_NO_KRB5_524
-           char *p;
-           int len;
-#endif
 
-           if (dflag)
-               printf("Using Kerberos V5 ticket natively\n");
-
-#ifndef HAVE_NO_KRB5_524
-           status = krb5_524_conv_principal (context, v5cred->client,
-                                             (char *) &k4name,
-                                             (char *) &k4inst,
-                                             (char *) &k4realm);
-           if (status) {
-               afs_com_err(progname, status, "while converting principal "
-                       "to Kerberos V4 format");
-               status = AKLOG_KERBEROS;
-               goto out;
-           }
-           strcpy (username, k4name);
-           if (k4inst[0]) {
-               strcat (username, ".");
-               strcat (username, k4inst);
-           }
-#else
-           len = min(get_princ_len(context, v5cred->client, 0),
-                     second_comp(context, v5cred->client) ?
-                     MAXKTCNAMELEN - 2 : MAXKTCNAMELEN - 1);
-           strncpy(username, get_princ_str(context, v5cred->client, 0), len);
-           username[len] = '\0';
-           
-           if (second_comp(context, v5cred->client)) {
-               strcat(username, ".");
-               p = username + strlen(username);
-               len = min(get_princ_len(context, v5cred->client, 1),
-                         MAXKTCNAMELEN - strlen(username) - 1);
-               strncpy(p, get_princ_str(context, v5cred->client, 1), len);
-               p[len] = '\0';
-           }
-#endif
-
-           memset(&atoken, 0, sizeof(atoken));
-           atoken.kvno = RXKAD_TKT_TYPE_KERBEROS_V5;
-           atoken.startTime = v5cred->times.starttime;;
-           atoken.endTime = v5cred->times.endtime;
-           memcpy(&atoken.sessionKey, get_cred_keydata(v5cred),
-                  get_cred_keylen(v5cred));
-           atoken.ticketLen = v5cred->ticket.length;
-           memcpy(atoken.ticket, v5cred->ticket.data, atoken.ticketLen);
-#ifndef HAVE_NO_KRB5_524
-       } else {
-           CREDENTIALS cred;
-
-           if (dflag)
-               printf("Using Kerberos 524 translator service\n");
-
-           status = krb5_524_convert_creds(context, v5cred, &cred);
-
-           if (status) {
-               afs_com_err(progname, status, "while converting tickets "
-                       "to Kerberos V4 format");
-               status = AKLOG_KERBEROS;
-               goto out;
-           }
-
-           strcpy (username, cred.pname);
-           if (cred.pinst[0]) {
-               strcat (username, ".");
-               strcat (username, cred.pinst);
-           }
-
-           atoken.kvno = cred.kvno;
-           atoken.startTime = cred.issue_date;
-           /*
-            * It seems silly to go through a bunch of contortions to
-            * extract the expiration time, when the v5 credentials already
-            * has the exact time!  Let's use that instead.
-            *
-            * Note that this isn't a security hole, as the expiration time
-            * is also contained in the encrypted token
-            */
-           atoken.endTime = v5cred->times.endtime;
-           memcpy(&atoken.sessionKey, cred.session, 8);
-           atoken.ticketLen = cred.ticket_st.length;
-           memcpy(atoken.ticket, cred.ticket_st.dat, atoken.ticketLen);
-#endif /* HAVE_NO_KRB5_524 */
-       }
-       
        if (!force &&
-           !ktc_GetToken(&aserver, &btoken, sizeof(btoken), &aclient) &&
-           atoken.kvno == btoken.kvno &&
-           atoken.ticketLen == btoken.ticketLen &&
-           !memcmp(&atoken.sessionKey, &btoken.sessionKey, sizeof(atoken.sessionKey)) &&
-           !memcmp(atoken.ticket, btoken.ticket, atoken.ticketLen)) {
-
-           if (dflag) {
-               printf("Identical tokens already exist; skipping.\n");
-           }
+           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
 
        if (noprdb) {
-           if (dflag) {
-               printf("Not resolving name %s to id (-noprdb set)\n",
-                       username);
-           }
+           afs_dprintf("Not resolving name %s to id (-noprdb set)\n", username);
        }
        else {
-           if (strcmp(realm_of_user, realm_of_cell)) {
-               strcat(username, "@");
-               strcat(username, realm_of_user);
-           }
+           afs_dprintf("About to resolve name %s to id in cell %s.\n", username,
+                   cellconf.name);
 
-           if (dflag) {
-               printf("About to resolve name %s to id in cell %s.\n", 
-                       username, aserver.cell);
-           }
-
-           if (!pr_Initialize (0,  AFSDIR_CLIENT_ETC_DIRPATH, aserver.cell))
+           if (!pr_Initialize (0,  AFSDIR_CLIENT_ETC_DIRPATH, cellconf.name))
                status = pr_SNameToId (username, &viceId);
-           
-           if (dflag) {
-               if (status) 
-                   printf("Error %d\n", status);
-               else
-                   printf("Id %d\n", (int) viceId);
-           }
-           
+
+           if (status)
+               afs_dprintf("Error %d\n", status);
+           else
+               afs_dprintf("Id %d\n", (int) viceId);
+
 
            /*
             * This code is taken from cklog -- it lets people
@@ -828,17 +1011,13 @@ auth_to_cell(krb5_context context, char *cell, char *realm, char **linkedcell)
             */
 
 #ifdef ALLOW_REGISTER
-           if ((status == 0) && (viceId == ANONYMOUSID) &&
-               (strcmp(realm_of_user, realm_of_cell) != 0)) {
-               if (dflag) {
-                   printf("doing first-time registration of %s "
-                           "at %s\n", username, cellconf.name);
-               }
+           if ((status == 0) && (viceId == ANONYMOUSID) && isForeign) {
+               afs_dprintf("doing first-time registration of %s at %s\n",
+                       username, cellconf.name);
                viceId = 0;
-               strncpy(aclient.name, username, MAXKTCNAMELEN - 1);
-               strcpy(aclient.instance, "");
-               strncpy(aclient.cell, realm_of_user, MAXKTCREALMLEN - 1);
-               if ((status = ktc_SetToken(&aserver, &atoken, &aclient, 0))) {
+
+               status = ktc_SetTokenEx(token);
+               if (status) {
                    afs_com_err(progname, status,
                                "while obtaining tokens for cell %s",
                                cellconf.name);
@@ -848,12 +1027,12 @@ 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
                 */
 
                if ((status = pr_Initialize(1L,  AFSDIR_CLIENT_ETC_DIRPATH,
-                                           aserver.cell))) {
+                                           cellconf.name))) {
                    printf("Error %d\n", status);
                }
 
@@ -870,62 +1049,47 @@ 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)) {
-               sprintf(username, "AFS ID %d", (int) viceId);
+               status = token_setRxkadViceId(rxkadToken, viceId);
+               if (status) {
+                   fprintf(stderr, "Error %d setting rxkad ViceId\n", status);
+                   status = AKLOG_SUCCESS;
+               } else {
+                   token_replaceToken(token, rxkadToken);
+               }
            }
        }
 
-       if (dflag) {
-           fprintf(stdout, "Set username to %s\n", username);
-       }
+       afs_dprintf("Setting tokens. %s @ %s \n", username, cellconf.name);
 
-       /* Reset the "aclient" structure before we call ktc_SetToken.
-        * This structure was first set by the ktc_GetToken call when
-        * we were comparing whether identical tokens already existed.
-        */
-       strncpy(aclient.name, username, MAXKTCNAMELEN - 1);
-       strcpy(aclient.instance, "");
-       strncpy(aclient.cell, realm_of_user, MAXKTCREALMLEN - 1);
-
-       if (dflag) {
-           printf("Setting tokens. %s / %s @ %s \n",
-                   aclient.name, aclient.instance, aclient.cell );
-       }
 #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 
+       /* 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
         * 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
-       if ((status = ktc_SetToken(&aserver, &atoken, &aclient, afssetpag))) {
-           afs_com_err(progname, status, "while obtaining tokens for cell %s",
+       token_setPag(token, afssetpag);
+       status = ktc_SetTokenEx(token);
+       if (status) {
+           afs_com_err(progname, status, "while setting tokens for cell %s",
                        cellconf.name);
            status = AKLOG_TOKEN;
        }
     }
     else
-       if (dflag) {
-           printf("Noauth mode; not authenticating.\n");
-       }
+       afs_dprintf("Noauth mode; not authenticating.\n");
 
 out:
+    if (rxkadToken) {
+       token_freeToken(&rxkadToken);
+    }
+
     if (local_cell)
        free(local_cell);
-    if (realm_from_princ)
-       free(realm_from_princ);
-    if (realm_of_user)
-       free(realm_of_user);
+    if (username)
+       free(username);
 
     return(status);
 }
@@ -952,8 +1116,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;
@@ -967,7 +1131,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);
@@ -981,8 +1145,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
@@ -1002,9 +1166,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));
@@ -1016,7 +1180,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);
@@ -1024,22 +1188,26 @@ 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, 
-                                   sizeof(linkbuf)) > 0))) {
+       link = readlink(pathtocheck, linkbuf, sizeof(linkbuf)-1);
+
+       if (link > 0) {
+           linkbuf[link] = '\0'; /* NUL terminate string */
+
            if (++symlinkcount > MAXSYMLINKS) {
                fprintf(stderr, "%s: %s\n", progname, strerror(ELOOP));
                exit(AKLOG_BADPATH);
            }
+
            memset(tmpbuf, 0, sizeof(tmpbuf));
            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));
@@ -1050,8 +1218,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);
@@ -1085,16 +1253,15 @@ add_hosts(char *file)
     int i;
     struct hostent *hp;
     struct in_addr in;
-    
+
     memset(outbuf, 0, sizeof(outbuf));
 
     vio.out_size = sizeof(outbuf);
     vio.in_size = 0;
     vio.out = outbuf;
 
-    if (dflag) {
-       printf("Getting list of hosts for %s\n", file);
-    }
+    afs_dprintf("Getting list of hosts for %s\n", file);
+
     /* Don't worry about errors. */
     if (!pioctl(file, VIOCWHEREIS, &vio, 1)) {
        phosts = (long *) outbuf;
@@ -1108,7 +1275,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
@@ -1118,15 +1285,11 @@ add_hosts(char *file)
        for (i = 0; phosts[i]; i++) {
            if (hosts) {
                in.s_addr = phosts[i];
-               if (dflag) {
-                   printf("Got host %s\n", inet_ntoa(in));
-               }
+               afs_dprintf("Got host %s\n", inet_ntoa(in));
                ll_string(&hostlist, ll_s_add, (char *)inet_ntoa(in));
            }
            if (zsubs && (hp=gethostbyaddr((char *) &phosts[i],sizeof(long),AF_INET))) {
-               if (dflag) {
-                   printf("Got host %s\n", hp->h_name);
-               }
+               afs_dprintf("Got host %s\n", hp->h_name);
                ll_string(&zsublist, ll_s_add, hp->h_name);
            }
        }
@@ -1134,11 +1297,11 @@ 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
-auth_to_path(krb5_context context, char *path)
+auth_to_path(krb5_context context, const char *config, char *path)
 {
     int status = AKLOG_SUCCESS;
     int auth_status = AKLOG_SUCCESS;
@@ -1172,12 +1335,10 @@ auth_to_path(krb5_context context, char *path)
     /* Go on to the next level down the path */
     while ((nextpath = next_path(NULL))) {
        strcpy(pathtocheck, nextpath);
-       if (dflag) {
-           printf("Checking directory %s\n", pathtocheck);
-       }
-       /* 
-        * If this is an afs mountpoint, determine what cell from 
-        * the mountpoint name which is of the form 
+       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
         * #cellname:volumename or %cellname:volumename.
         */
        if (get_afs_mountpoint(pathtocheck, mountpoint, sizeof(mountpoint))) {
@@ -1190,7 +1351,8 @@ auth_to_path(krb5_context context, char *path)
                add_hosts(pathtocheck);
            if ((endofcell = strchr(mountpoint, VOLMARKER))) {
                *endofcell = '\0';
-               if ((auth_status = auth_to_cell(context, cell, NULL, NULL))) {
+               auth_status = auth_to_cell(context, config, cell, NULL, NULL);
+               if (auth_status) {
                    if (status == AKLOG_SUCCESS)
                        status = auth_status;
                    else if (status != auth_status)
@@ -1202,9 +1364,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);
            }
@@ -1216,7 +1378,7 @@ auth_to_path(krb5_context context, char *path)
            }
        }
     }
-    
+
 
     return(status);
 }
@@ -1267,8 +1429,6 @@ main(int argc, char *argv[])
 
     extern char *progname;     /* Name of this program */
 
-    extern int dflag;          /* Debug mode */
-
     int cmode = FALSE;         /* Cellname mode */
     int pmode = FALSE;         /* Path name mode */
 
@@ -1280,6 +1440,7 @@ main(int argc, char *argv[])
     linked_list paths;         /* List of paths to log to */
     ll_node *cur_node;
     char *linkedcell;
+    const char *config = AFSDIR_CLIENT_ETC_DIRPATH;
 
     memset(&cellinfo, 0, sizeof(cellinfo));
 
@@ -1299,8 +1460,35 @@ main(int argc, char *argv[])
     else
        progname = argv[0];
 
-    krb5_init_context(&context);
-    initialize_ktc_error_table ();
+#if defined(KRB5_PROG_ETYPE_NOSUPP) && !(defined(HAVE_KRB5_ENCTYPE_ENABLE) || defined(HAVE_KRB5_ALLOW_WEAK_CRYPTO))
+    {
+       char *filepath = NULL, *newpath = NULL;
+#ifndef AFS_DARWIN_ENV
+       char *defaultpath = "/etc/krb5.conf";
+#else
+       char *defaultpath = "~/Library/Preferences/edu.mit.Kerberos:/Library/Preferences/edu.mit.Kerberos";
+#endif
+       filepath = getenv("KRB5_CONFIG");
+       asprintf(&newpath, "%s:%s/krb5-weak.conf",
+                filepath ? filepath : defaultpath,
+                AFSDIR_CLIENT_ETC_DIRPATH);
+       setenv("KRB5_CONFIG", newpath, 1);
+#endif
+       krb5_init_context(&context);
+
+#if defined(KRB5_PROG_ETYPE_NOSUPP) && !(defined(HAVE_KRB5_ENCTYPE_ENABLE) || defined(HAVE_KRB5_ALLOW_WEAK_CRYPTO))
+       free(newpath);
+       if (filepath)
+           setenv("KRB5_CONFIG", filepath, 1);
+       else
+           unsetenv("KRB5_CONFIG");
+    }
+#endif
+    initialize_KTC_error_table ();
+    initialize_U_error_table();
+    initialize_RXK_error_table();
+    initialize_ACFG_error_table();
+    initialize_PT_error_table();
     afs_set_com_err_hook(redirect_errors);
 
     /*
@@ -1308,12 +1496,12 @@ main(int argc, char *argv[])
      * krb5_allow_weak_crypto is MIT Kerberos 1.8.  krb5_enctype_enable is
      * Heimdal.
      */
-#if defined(HAVE_KRB5_ALLOW_WEAK_CRYPTO)
-    krb5_allow_weak_crypto(context, 1);
-#elif defined(HAVE_KRB5_ENCTYPE_ENABLE)
+#if defined(HAVE_KRB5_ENCTYPE_ENABLE)
     i = krb5_enctype_valid(context, ETYPE_DES_CBC_CRC);
     if (i)
         krb5_enctype_enable(context, ETYPE_DES_CBC_CRC);
+#elif defined(HAVE_KRB5_ALLOW_WEAK_CRYPTO)
+    krb5_allow_weak_crypto(context, 1);
 #endif
 
     /* Initialize list of cells to which we have authenticated */
@@ -1323,7 +1511,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++;
@@ -1369,7 +1557,12 @@ main(int argc, char *argv[])
            }
            else
                usage();
-           
+       else if (strcmp(argv[i], "-config") == 0)
+           if (++i < argc) {
+               config = argv[i];
+           }
+           else
+               usage();
        else if (argv[i][0] == '-')
            usage();
        else if (!pmode && !cmode) {
@@ -1378,7 +1571,7 @@ main(int argc, char *argv[])
                pmode++;
                strcpy(path, argv[i]);
            }
-           else { 
+           else {
                cmode++;
                strcpy(cell, argv[i]);
            }
@@ -1402,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);
                }
@@ -1443,18 +1636,16 @@ main(int argc, char *argv[])
     if ((cells.nelements + paths.nelements) == 0) {
        struct passwd *pwd;
 
-       status = auth_to_cell(context, NULL, NULL, &linkedcell);
-       
+       status = auth_to_cell(context, config, 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.
         */
 
        if (!status && linked && linkedcell != NULL) {
-           if (dflag) {
-               printf("Linked cell: %s\n", linkedcell);
-           }
-           status = auth_to_cell(context, linkedcell, NULL, NULL);
+           afs_dprintf("Linked cell: %s\n", linkedcell);
+           status = auth_to_cell(context, config, linkedcell, NULL, NULL);
        }
        if (linkedcell) {
            free(linkedcell);
@@ -1478,21 +1669,17 @@ main(int argc, char *argv[])
            if ((stat(xlog_path, &sbuf) == 0) &&
                ((f = fopen(xlog_path, "r")) != NULL)) {
 
-               if (dflag) {
-                   printf("Reading %s for cells to authenticate to.\n",
-                          xlog_path);
-               }
+               afs_dprintf("Reading %s for cells to authenticate to.\n",
+                       xlog_path);
 
                while (fgets(fcell, 100, f) != NULL) {
                    int auth_status;
 
                    fcell[strlen(fcell) - 1] = '\0';
 
-                   if (dflag) {
-                       printf("Found cell %s in %s.\n", fcell, xlog_path);
-                   }
+                   afs_dprintf("Found cell %s in %s.\n", fcell, xlog_path);
 
-                   auth_status = auth_to_cell(context, fcell, NULL, NULL);
+                   auth_status = auth_to_cell(context, config, fcell, NULL, NULL);
                    if (status == AKLOG_SUCCESS)
                        status = auth_status;
                    else
@@ -1505,16 +1692,16 @@ main(int argc, char *argv[])
        /* Log to all cells in the cells list first */
        for (cur_node = cells.first; cur_node; cur_node = cur_node->next) {
            memcpy((char *)&cellinfo, cur_node->data, sizeof(cellinfo));
-           if ((status = auth_to_cell(context, cellinfo.cell, cellinfo.realm,
-                                      &linkedcell)))
+           status = auth_to_cell(context, config, cellinfo.cell,
+                                 cellinfo.realm, &linkedcell);
+           if (status) {
                somethingswrong++;
-           else {
+           } else {
                if (linked && linkedcell != NULL) {
-                   if (dflag) {
-                       printf("Linked cell: %s\n", linkedcell);
-                    }
-                   if ((status = auth_to_cell(context, linkedcell,
-                                              cellinfo.realm, NULL)))
+                   afs_dprintf("Linked cell: %s\n", linkedcell);
+                   status = auth_to_cell(context, config, linkedcell,
+                                         cellinfo.realm, NULL);
+                   if (status)
                        somethingswrong++;
                }
                if (linkedcell != NULL) {
@@ -1526,12 +1713,13 @@ main(int argc, char *argv[])
 
        /* Then, log to all paths in the paths list */
        for (cur_node = paths.first; cur_node; cur_node = cur_node->next) {
-           if ((status = auth_to_path(context, cur_node->data)))
+           status = auth_to_path(context, config, cur_node->data);
+           if (status)
                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.
         */
@@ -1540,7 +1728,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);
        }
@@ -1562,12 +1750,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
@@ -1596,14 +1784,14 @@ get_credv5_akimpersonate(krb5_context context,
     EncTicketPart enc_tkt_reply[1];
     krb5_address address[30];
     krb5_addresses faddr[1];
-    int temp_vno[1];
+    unsigned int temp_vno[1];
     time_t temp_time[2];
 #else
     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;
@@ -1726,7 +1914,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) {
@@ -1778,8 +1966,8 @@ get_credv5_akimpersonate(krb5_context context,
 
         if(buf_len != buf_size) {
             afs_com_err(progname, code,
-                   "%d != %d while encoding ticket (internal ASN.1 encoder error",
-                   buf_len, buf_size);
+                   "%u != %u while encoding ticket (internal ASN.1 encoder error",
+                   (unsigned int)buf_len, (unsigned int)buf_size);
             goto cleanup;
         }
         what = "krb5_crypto_init";
@@ -1800,7 +1988,7 @@ get_credv5_akimpersonate(krb5_context context,
         }
     } /* crypto block */
     ticket_reply->enc_part.etype = deref_entry_enctype(entry);
-    ticket_reply->enc_part.kvno = temp_vno;
+    ticket_reply->enc_part.kvno = (void *)temp_vno;
     *ticket_reply->enc_part.kvno = entry->vno;
     ticket_reply->tkt_vno = 5;
 #else
@@ -1884,7 +2072,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;
@@ -1900,21 +2087,19 @@ get_credv5(krb5_context context, char *name, char *inst, char *realm,
     krb5_error_code r;
     static krb5_principal client_principal = 0;
 
-    if (dflag) {
-       printf("Getting tickets: %s%s%s@%s\n", name, (inst && inst[0])
-              ? "/" : "", inst ? inst : "", 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,
                                  strlen(realm), realm,
                                  name,
-                                 (inst && strlen(inst)) ? inst : (void *) NULL,
-                                 (void *) NULL))) {
+                                 (inst && strlen(inst)) ? inst : NULL,
+                                 NULL))) {
         return r;
     }
-    
+
 
     if (!_krb425_ccache) {
         r = krb5_cc_default(context, &_krb425_ccache);
@@ -1930,12 +2115,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
@@ -1951,7 +2136,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;
 }
 
@@ -1964,8 +2149,11 @@ get_user_realm(krb5_context context, char **realm)
 
     *realm = NULL;
 
-    if (!_krb425_ccache)
-        krb5_cc_default(context, &_krb425_ccache);
+    if (!_krb425_ccache) {
+       r = krb5_cc_default(context, &_krb425_ccache);
+       if (r)
+           return r;
+    }
     if (!client_principal) {
        if (client) {
            r = krb5_parse_name(context, client,  &client_principal);