aklog: Avoid misleading AFSCELL message
[openafs.git] / src / aklog / aklog.c
index 0a1f915..988d54c 100644 (file)
@@ -1,94 +1,79 @@
-/* 
+/*
  * $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>
+#define KERBEROS_APPLE_DEPRECATED(x)
 #include <krb5.h>
-#if defined(HAVE_ET_COM_ERR_H)
-#include <et/com_err.h>
+#ifdef HAVE_COM_ERR_H
+# include <com_err.h>
+#elif HAVE_ET_COM_ERR_H
+# include <et/com_err.h>
+#elif HAVE_KRB5_COM_ERR_H
+# include <krb5/com_err.h>
 #else
-#include <com_err.h>
+# error No com_err.h? We need some kind of com_err.h
 #endif
 
 #ifndef HAVE_KERBEROSV_HEIM_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
@@ -144,9 +125,6 @@ typedef struct {
     char realm[REALM_SZ];
 } cellinfo_t;
 
-struct afsconf_cell ak_cellconfig; /* General information about the cell */
-static char linkedcell[MAXCELLCHARS+1];
-static char linkedcell2[MAXCELLCHARS+1];
 static krb5_ccache  _krb425_ccache = NULL;
 
 /*
@@ -163,7 +141,7 @@ extern char *afs_realm_of_cell(krb5_context, struct afsconf_cell *, int);
 static int isdir(char *, unsigned char *);
 static krb5_error_code get_credv5(krb5_context context, char *, char *,
                                  char *, krb5_creds **);
-static int get_user_realm(krb5_context, char *);
+static int get_user_realm(krb5_context, char **);
 
 #define TRYAGAIN(x) (x == AKLOG_TRYAGAIN || \
                     x == KRB5KDC_ERR_S_PRINCIPAL_UNKNOWN || \
@@ -189,8 +167,9 @@ 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) 
-extern krb5_error_code encode_krb5_enc_tkt_part (const krb5_enc_tkt_part *rep, krb5_data **code);
+#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);
 
 krb5_error_code
 krb5_encrypt_tkt_part(krb5_context context,
@@ -200,7 +179,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,
@@ -242,10 +221,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
@@ -333,10 +313,23 @@ 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;
+           if (krb5_init_context(&context) == 0) {
+                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);
@@ -345,560 +338,804 @@ redirect_errors(const char *who, afs_int32 code, const char *fmt, va_list ap)
     fflush(stderr);
 }
 
-/* ANL - CMU lifetime convert routine */
-/* for K5.4.1 don't use this for now. Need to see if it is needed */
-/* maybe needed in the krb524d module as well */
-/* extern unsigned long krb_life_to_time(); */
+static void
+afs_dprintf(char *fmt, ...) {
+    va_list ap;
 
-static char *copy_cellinfo(cellinfo_t *cellinfo)
+    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, char *linkedcell)
+static int
+get_cellconfig(const char *config, char *cell,
+              struct afsconf_cell *cellconfig, char **local_cell)
 {
     int status = AKLOG_SUCCESS;
     struct afsconf_dir *configdir;
 
-    memset(local_cell, 0, sizeof(local_cell));
     memset(cellconfig, 0, sizeof(*cellconfig));
 
-    if (!(configdir = afsconf_Open(AFSDIR_CLIENT_ETC_DIRPATH))) {
-       fprintf(stderr, 
-               "%s: can't get afs configuration (afsconf_Open(%s))\n",
-               progname, AFSDIR_CLIENT_ETC_DIRPATH);
+    *local_cell = malloc(MAXCELLCHARS);
+    if (*local_cell == NULL) {
+       fprintf(stderr, "%s: can't allocate memory for local cell name\n",
+               progname);
        exit(AKLOG_AFS);
     }
 
-    if (afsconf_GetLocalCell(configdir, local_cell, MAXCELLCHARS)) {
-       fprintf(stderr, "%s: can't determine local cell.\n", progname);
+    if (!(configdir = afsconf_Open(config))) {
+       fprintf(stderr,
+               "%s: can't get afs configuration (afsconf_Open(%s))\n",
+               progname, config);
        exit(AKLOG_AFS);
     }
 
-    if ((cell == NULL) || (cell[0] == 0))
-       cell = local_cell;
+    if (cell != NULL && cell[0] == '\0') {
+       /* Use the local cell */
+       cell = NULL;
+    }
 
-       linkedcell[0] = '\0';
+    /* XXX - This function modifies 'cell' by passing it through lcstring */
     if (afsconf_GetCellInfo(configdir, cell, NULL, cellconfig)) {
-       fprintf(stderr, "%s: Can't get information about cell %s.\n",
-               progname, cell);
+       if (cell != NULL) {
+           fprintf(stderr, "%s: Can't get information about cell %s.\n",
+                   progname, cell);
+       } else {
+           fprintf(stderr, "%s: Can't get information about the local cell.\n",
+                   progname);
+       }
        status = AKLOG_AFS;
+    } else if (afsconf_GetLocalCell(configdir, *local_cell, MAXCELLCHARS)) {
+       fprintf(stderr, "%s: can't determine local cell.\n", progname);
+       exit(AKLOG_AFS);
     }
-       if (cellconfig->linkedCell) 
-               strncpy(linkedcell,cellconfig->linkedCell,MAXCELLCHARS);
 
-    (void) afsconf_Close(configdir);
+    afsconf_Close(configdir);
 
     return(status);
 }
 
+static char *
+extract_realm(krb5_context context, krb5_principal princ) {
+    int len;
+    char *realm;
+
+    len = realm_len(context, princ);
+    if (len > REALM_SZ-1)
+       len = REALM_SZ-1;
+
+    realm = malloc(sizeof(char) * (len+1));
+    if (realm == NULL)
+       return NULL;
+
+    strncpy(realm, realm_data(context, princ), len);
+    realm[len] = '\0';
+
+    return realm;
+}
+
+static int
+get_realm_from_cred(krb5_context context, krb5_creds *v5cred, char **realm) {
+#if !defined(HEIMDAL) && defined(HAVE_KRB5_DECODE_TICKET)
+    krb5_error_code code;
+    krb5_ticket *ticket;
+
+    *realm = NULL;
+
+    code = krb5_decode_ticket(&v5cred->ticket, &ticket);
+    if (code)
+       return code;
+
+    *realm = extract_realm(context, ticket->server);
+    if (*realm == NULL)
+       code = ENOMEM;
+
+    krb5_free_ticket(context, ticket);
+
+    return code;
+#else
+    *realm = NULL;
+    return 0;
+#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,
+ *      if set.
+ *
+ * @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
+    void *inkey = get_cred_keydata(v5cred);
+    size_t inkey_sz = get_cred_keylen(v5cred);
+
+    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) {
+       if (!noprdb)
+           afs_com_err(progname, status,
+                       "while converting principal to Kerberos V4 format");
+    } else {
+       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;
+    if (tkt_DeriveDesKey(get_creds_enctype(v5cred), inkey, inkey_sz,
+                        &token.sessionKey) != 0) {
+       return RXKADBADKEY;
+    }
+    token.ticketLen = v5cred->ticket.length;
+    memcpy(token.ticket, v5cred->ticket.data, token.ticketLen);
+
+    status = token_importRxkadViceId(tokenPtr, &token, 0);
+    if (status) {
+       return status;
+    }
+
+    if (username[0] != '\0')
+       *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,
+ *      if set.
+ *
+ * @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,
+ *      if set.
+ * @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 || username == NULL) {
+       *authuser = username;
+       username = NULL;
+       *foreign = 0;
+    } else {
+       if (asprintf(authuser, "%s@%s", username, realmUsed) < 0) {
+           fprintf(stderr, "%s: Out of memory building PTS name\n", progname);
+           *authuser = NULL;
+           status = AKLOG_MISC;
+           goto out;
+       }
+       *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)
+static int
+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[REALM_SZ]; /* Kerberos realm of user */
-    char *realm_from_princ = 0 ;  /* Calculated realm data */
-    char *realm_of_cell = 0;     /* Pointer to realm we're using */    
-    int retry;                   /* round, and round we go ... */
-    
-    char local_cell[MAXCELLCHARS+1];
-    char cell_to_use[MAXCELLCHARS+1]; /* Cell to authenticate to */
-    static char lastcell[MAXCELLCHARS+1] = { 0 };
-    static char confname[512] = { 0 };
-    krb5_creds *v5cred = NULL;
-    struct ktc_principal aserver;
-    struct ktc_principal aclient;
-    struct ktc_token atoken, btoken;
-
-    memset(realm_of_user, 0, sizeof(realm_of_user));
-
-    if (confname[0] == '\0') {
-       strncpy(confname, AFSDIR_CLIENT_ETC_DIRPATH, sizeof(confname));
-       confname[sizeof(confname) - 2] = '\0';
-    }
+    char *local_cell = NULL;
+    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, &ak_cellconfig,
-                        local_cell, linkedcell)))
+    if ((status = get_cellconfig(config, cell, &cellconf, &local_cell)))
        return(status);
 
-    strncpy(cell_to_use, ak_cellconfig.name, MAXCELLCHARS);
-    cell_to_use[MAXCELLCHARS] = 0;
-
-    if (ll_string(&authedcells, ll_s_check, cell_to_use)) {
-       if (dflag) {
-           printf("Already authenticated to %s (or tried to)\n", 
-                  cell_to_use);
+    if (linkedcell != NULL) {
+       if (cellconf.linkedCell != NULL) {
+           *linkedcell = strdup(cellconf.linkedCell);
+           if (*linkedcell == NULL) {
+               status = ENOMEM;
+               goto out;
+           }
+       } else {
+           *linkedcell = NULL;
        }
-       return(AKLOG_SUCCESS);
     }
 
-    /* 
+    if (ll_string(&authedcells, ll_s_check, 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.
      */
-    (void)ll_string(&authedcells, ll_s_add, cell_to_use);
+    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
      * can return something different depending on whether or not we
      * are in -noauth mode.
      */
-    if (ll_string(&zsublist, ll_s_add, cell_to_use) == LL_FAILURE) {
-       fprintf(stderr, 
+    if (ll_string(&zsublist, ll_s_add, cellconf.name) == LL_FAILURE) {
+       fprintf(stderr,
                "%s: failure adding cell %s to zephyr subscriptions list.\n",
-               progname, cell_to_use);
+               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",
-                  cell_to_use, ak_cellconfig.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");
-           return(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, cell_to_use, 
-                                   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, &ak_cellconfig, 
-                                                     FALSE);
-                   if (!realm_of_cell) {
-                       fprintf(stderr, 
-                               "%s: Couldn't figure out realm for cell %s.\n",
-                               progname, cell_to_use);
-                       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(cell_to_use, realm_of_cell)!=0) {
-                   status = get_credv5(context, AFSKEY, cell_to_use, 
-                                       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, 
-                                                         &ak_cellconfig, TRUE);
-                       if (!realm_of_cell) {
-                           fprintf(stderr,
-                                   "%s: Couldn't figure out realm for cell "
-                                   "%s.\n", progname, cell_to_use);
-                           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, cell_to_use, 
-                                       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_to_use, 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_to_use,
-                                           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) {
-           if (dflag) {
-               printf("Kerberos error code returned by get_cred : %d\n",
-                      status);
-           }
-           fprintf(stderr, "%s: Couldn't get %s AFS tickets:\n",
-                   progname, cell_to_use);
-           afs_com_err(progname, status, "while getting AFS tickets");
-           return(AKLOG_KERBEROS);
-       }
-       
-       /* 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 !defined(USING_HEIMDAL) && defined(HAVE_KRB5_DECODE_TICKET)
-       if (strcmp(realm_of_cell, "") == 0) {
-           krb5_error_code code;
-           krb5_ticket *ticket;
-
-           code = krb5_decode_ticket(&v5cred->ticket, &ticket);
-
-           if (code != 0) {
-               fprintf(stderr,
-                       "%s: Couldn't decode ticket to determine realm for "
-                       "cell %s.\n",
-                       progname, cell_to_use);
-           } else {
-               int len = realm_len(context, ticket->server);
-               /* This really shouldn't happen. */
-               if (len > REALM_SZ-1)
-                   len = REALM_SZ-1;
-
-               realm_from_princ = (char *) malloc(sizeof(char) * (len+1));
-               
-               strncpy(realm_from_princ, realm_data(context, ticket->server), 
-                       len);
-               realm_from_princ[len] = 0;
-               realm_of_cell = realm_from_princ;
-               
-               krb5_free_ticket(context, ticket);
-           }
+       status = rxkad_get_token(context, &cellconf, realm, &rxkadToken,
+                                &username, &isForeign);
+       if (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;
        }
-#endif
-
-       strncpy(aserver.name, AFSKEY, MAXKTCNAMELEN - 1);
-       strncpy(aserver.instance, AFSINST, MAXKTCNAMELEN - 1);
-       strncpy(aserver.cell, cell_to_use, 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");
-               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(&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");
-               return(AKLOG_KERBEROS);
-           }
-
-           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");
-           }
-           return 0;
+           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);
-           }
+       if (username == NULL) {
+           afs_dprintf("Not resolving name to id\n");
+       }
+       else if (noprdb) {
+           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, cellconf.name))
+               status = pr_SNameToId (username, &viceId);
 
-           strcpy(lastcell, aserver.cell);
+           if (status)
+               afs_dprintf("Error %d\n", status);
+           else
+               afs_dprintf("Id %d\n", (int) viceId);
 
-           if (!pr_Initialize (0, confname, aserver.cell))
-                   status = pr_SNameToId (username, &viceId);
-           
-           if (dflag) {
-               if (status) 
-                   printf("Error %d\n", status);
-               else
-                   printf("Id %d\n", (int) viceId);
-           }
-           
-               /*
-                * 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.
-                */
 
-               /*
-                * This code is taken from cklog -- it lets people
-                * automatically register with the ptserver in foreign cells
-                */
+           /*
+            * This code is taken from cklog -- it lets people
+            * automatically register with the ptserver in foreign cells
+            */
 
 #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, cell_to_use);
-               }
+           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",
-                               cell_to_use);
+                               cellconf.name);
                    status = AKLOG_TOKEN;
                }
 
                /*
                 * 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, confname, aserver.cell))) {
+               if ((status = pr_Initialize(1L,  AFSDIR_CLIENT_ETC_DIRPATH,
+                                           cellconf.name))) {
                    printf("Error %d\n", status);
                }
 
                if ((status = pr_CreateUser(username, &viceId))) {
                    fprintf(stderr, "%s: %s so unable to create remote PTS "
                            "user %s in cell %s (status: %d).\n", progname,
-                           afs_error_message(status), username, cell_to_use,
+                           afs_error_message(status), username, cellconf.name,
                            status);
                    viceId = ANONYMOUSID;
                } else {
                    printf("created cross-cell entry for %s (Id %d) at %s\n",
-                          username, viceId, cell_to_use);
+                          username, viceId, cellconf.name);
                }
            }
 #endif /* ALLOW_REGISTER */
 
            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);
+       if (username) {
+           afs_dprintf("Setting tokens. %s @ %s\n",
+                       username, cellconf.name);
+       } else {
+           afs_dprintf("Setting tokens for cell %s\n", 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.
+        * We don't care about the return value, but need to collect it
+        * to avoid compiler warnings.
         */
-       write(2,"",0); /* dummy write */
+       if (write(2,"",0) < 0) {
+           /* dummy write, don't care */
+       }
 #endif
-       if ((status = ktc_SetToken(&aserver, &atoken, &aclient, afssetpag))) {
-           afs_com_err(progname, status, "while obtaining tokens for cell %s",
-                       cell_to_use);
+       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 (username)
+       free(username);
+
     return(status);
 }
 
-static int get_afs_mountpoint(char *file, char *mountpoint, int size)
+static int
+get_afs_mountpoint(char *file, char *mountpoint, int size)
 {
 #ifdef AFS_SUN_ENV
-       char V ='V'; /* AFS has problem on Sun with pioctl */
+    char V ='V'; /* AFS has problem on Sun with pioctl */
 #endif
     char our_file[MAXPATHLEN + 1];
     char *parent_dir;
@@ -906,8 +1143,7 @@ static int get_afs_mountpoint(char *file, char *mountpoint, int size)
     struct ViceIoctl vio;
     char cellname[BUFSIZ];
 
-    memset(our_file, 0, sizeof(our_file));
-    strcpy(our_file, file);
+    strlcpy(our_file, file, sizeof(our_file));
 
     if ((last_component = strrchr(our_file, DIR))) {
        *last_component++ = 0;
@@ -916,8 +1152,8 @@ static int 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;
@@ -931,10 +1167,10 @@ static int 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);
+               strlcat(cellname, VOLMARKERSTRING, sizeof(cellname));
+               strlcat(cellname, mountpoint + 1, sizeof(cellname));
                memset(mountpoint + 1, 0, size - 1);
                strcpy(mountpoint + 1, cellname);
            }
@@ -945,19 +1181,20 @@ static int 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
  * NULL.
  */
-static char *next_path(char *origpath)
+static char *
+next_path(char *origpath)
 {
     static char path[MAXPATHLEN + 1];
     static char pathtocheck[MAXPATHLEN + 1];
 
-    int link = FALSE;          /* Is this a symbolic link? */
+    ssize_t link;              /* Return value from readlink */
     char linkbuf[MAXPATHLEN + 1];
     char tmpbuf[MAXPATHLEN + 1];
 
@@ -965,21 +1202,21 @@ static char *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));
        memset(pathtocheck, 0, sizeof(pathtocheck));
-       strcpy(path, origpath);
+       strlcpy(path, origpath, sizeof(path));
        last_comp = path;
        symlinkcount = 0;
        return(NULL);
     }
 
     /* 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);
@@ -987,22 +1224,26 @@ static char *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);
+               strlcpy(tmpbuf, elast_comp, sizeof(tmpbuf));
            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));
@@ -1013,8 +1254,8 @@ static char *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);
@@ -1031,45 +1272,16 @@ static char *next_path(char *origpath)
        else
            last_comp = elast_comp;
     }
-    while(link);
+    while(link > 0);
 
     return(pathtocheck);
 }
 
-#if 0
-/*****************************************/
-int dee_gettokens()
+static void
+add_hosts(char *file)
 {
 #ifdef AFS_SUN_ENV
-       char V = 'V'; /* AFS has problem on SunOS */
-#endif
-   struct ViceIoctl vio;
-   char outbuf[BUFSIZ];
-   long ind;
-   int fd;
-
-   memset(outbuf, 0, sizeof(outbuf));
-
-   vio.out_size = sizeof(outbuf);
-   vio.in_size = sizeof(ind);
-   vio.out = outbuf;
-   vio.in = &ind;
-
-   ind = 0;
-   fd = open("dee.tok",O_WRONLY);
-   while(!pioctl(0,VIOCGETTOK,&vio,0)) {
-       write(fd,&outbuf,sizeof(outbuf)); 
-       ind++;
-   }
-   close(fd);
-}
-/*****************************************/
-#endif
-
-static void add_hosts(char *file)
-{
-#ifdef AFS_SUN_ENV
-       char V = 'V'; /* AFS has problem on SunOS */
+    char V = 'V'; /* AFS has problem on SunOS */
 #endif
     struct ViceIoctl vio;
     char outbuf[BUFSIZ];
@@ -1077,16 +1289,15 @@ static void 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;
@@ -1100,7 +1311,7 @@ static void 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
@@ -1110,15 +1321,11 @@ static void 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);
            }
        }
@@ -1126,13 +1333,14 @@ static void 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)
+static int
+auth_to_path(krb5_context context, const char *config, char *path)
 {
     int status = AKLOG_SUCCESS;
-    int auth_to_cell_status = AKLOG_SUCCESS;
+    int auth_status = AKLOG_SUCCESS;
 
     char *nextpath;
     char pathtocheck[MAXPATHLEN + 1];
@@ -1145,7 +1353,7 @@ static int auth_to_path(krb5_context context, char *path)
 
     /* Initialize */
     if (path[0] == DIR)
-       strcpy(pathtocheck, path);
+       strlcpy(pathtocheck, path, sizeof(pathtocheck));
     else {
        if (getcwd(pathtocheck, sizeof(pathtocheck)) == NULL) {
            fprintf(stderr, "Unable to find current working directory:\n");
@@ -1154,21 +1362,19 @@ static int auth_to_path(krb5_context context, char *path)
            exit(AKLOG_BADPATH);
        }
        else {
-           strcat(pathtocheck, DIRSTRING);
-           strcat(pathtocheck, path);
+           strlcat(pathtocheck, DIRSTRING, sizeof(pathtocheck));
+           strlcat(pathtocheck, path, sizeof(pathtocheck));
        }
     }
     next_path(pathtocheck);
 
     /* 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 
+       strlcpy(pathtocheck, nextpath, sizeof(pathtocheck));
+       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))) {
@@ -1181,10 +1387,11 @@ static int auth_to_path(krb5_context context, char *path)
                add_hosts(pathtocheck);
            if ((endofcell = strchr(mountpoint, VOLMARKER))) {
                *endofcell = '\0';
-               if ((auth_to_cell_status = auth_to_cell(context, cell, NULL))) {
+               auth_status = auth_to_cell(context, config, cell, NULL, NULL);
+               if (auth_status) {
                    if (status == AKLOG_SUCCESS)
-                       status = auth_to_cell_status;
-                   else if (status != auth_to_cell_status)
+                       status = auth_status;
+                   else if (status != auth_status)
                        status = AKLOG_SOMETHINGSWRONG;
                }
            }
@@ -1193,9 +1400,9 @@ static int 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);
            }
@@ -1207,14 +1414,15 @@ static int auth_to_path(krb5_context context, char *path)
            }
        }
     }
-    
+
 
     return(status);
 }
 
 
 /* Print usage message and exit */
-static void usage(void)
+static void
+usage(void)
 {
     fprintf(stderr, "\nUsage: %s %s%s%s\n", progname,
            "[-d] [[-cell | -c] cell [-k krb_realm]] ",
@@ -1245,7 +1453,8 @@ static void usage(void)
     exit(AKLOG_USAGE);
 }
 
-int main(int argc, char *argv[])
+int
+main(int argc, char *argv[])
 {
     krb5_context context;
     int status = AKLOG_SUCCESS;
@@ -1256,8 +1465,6 @@ int 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 */
 
@@ -1268,6 +1475,8 @@ int main(int argc, char *argv[])
     linked_list cells;         /* List of cells to log to */
     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));
 
@@ -1287,8 +1496,47 @@ int 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:/etc/krb5/krb5.conf";
+#else
+       char *defaultpath = "~/Library/Preferences/edu.mit.Kerberos:/Library/Preferences/edu.mit.Kerberos";
+#endif
+       filepath = getenv("KRB5_CONFIG");
+
+       /* only fiddle with KRB5_CONFIG if krb5-weak.conf actually exists */
+       if (asprintf(&newpath, "%s/krb5-weak.conf",
+                    AFSDIR_CLIENT_ETC_DIRPATH) < 0)
+           newpath = NULL;
+       if (newpath != NULL && access(newpath, R_OK) == 0) {
+           free(newpath);
+           newpath = NULL;
+           if (asprintf(&newpath, "%s:%s/krb5-weak.conf",
+                        filepath ? filepath : defaultpath,
+                        AFSDIR_CLIENT_ETC_DIRPATH) < 0)
+               newpath = NULL;
+           else
+               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))
+       if (newpath)
+           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);
 
     /*
@@ -1296,22 +1544,22 @@ int 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 */
-    (void)ll_init(&authedcells);
+    ll_init(&authedcells);
 
     /* Parse commandline arguments and make list of what to do. */
     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++;
@@ -1333,7 +1581,7 @@ int main(int argc, char *argv[])
                  (strcmp(argv[i], "-c") == 0)) && !pmode)
            if (++i < argc) {
                cmode++;
-               strcpy(cell, argv[i]);
+               strlcpy(cell, argv[i], sizeof(cell));
            }
            else
                usage();
@@ -1353,22 +1601,27 @@ int main(int argc, char *argv[])
                  (strcmp(argv[i], "-p") == 0)) && !cmode)
            if (++i < argc) {
                pmode++;
-               strcpy(path, argv[i]);
+               strlcpy(path, argv[i], sizeof(path));
+           }
+           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) {
            if (strchr(argv[i], DIR) || (strcmp(argv[i], ".") == 0) ||
                (strcmp(argv[i], "..") == 0)) {
                pmode++;
-               strcpy(path, argv[i]);
+               strlcpy(path, argv[i], sizeof(path));
            }
-           else { 
+           else {
                cmode++;
-               strcpy(cell, argv[i]);
+               strlcpy(cell, argv[i], sizeof(cell));
            }
        }
        else
@@ -1378,7 +1631,7 @@ int main(int argc, char *argv[])
            if (((i + 1) < argc) && (strcmp(argv[i + 1], "-k") == 0)) {
                i+=2;
                if (i < argc)
-                   strcpy(realm, argv[i]);
+                   strlcpy(realm, argv[i], sizeof(realm));
                else
                    usage();
            }
@@ -1390,7 +1643,7 @@ int 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);
                }
@@ -1427,124 +1680,94 @@ int main(int argc, char *argv[])
        }
     }
 
-    /*
-     * The code that _used_ to be here called setpag().  When you think
-     * about this, doing this makes no sense!  setpag() allocates a PAG
-     * only for the current process, so the token installed would have
-     * not be usable in the parent!  Since ktc_SetToken() now takes a
-     * 4th argument to control whether or not we're going to allocate
-     * a PAG (and since when you do it _that_ way, it modifies the cred
-     * structure of your parent)), why don't we use that instead?
-     */
-
-#if 0
-    if (afssetpag) {
-          status = setpag();
-          if (dflag) { 
-            int i,j;
-                int gidsetlen = 50;
-         int gidset[50];
-
-                printf("setpag %d\n",status);
-            j = getgroups(gidsetlen,gidset);
-         printf("Groups(%d):",j);
-         for (i = 0; i<j; i++) {
-           printf("%d",gidset[i]);
-           if((i+1)<j) printf(",");
-         }
-         printf("\n");
-          }
-       }
-#endif
     /* If nothing was given, log to the local cell. */
     if ((cells.nelements + paths.nelements) == 0) {
-               struct passwd *pwd;
-
-               status = auth_to_cell(context, NULL, NULL);
-       
-               /* If this cell is linked to a DCE cell, and user 
-                * requested -linked, get tokens for both 
-                * This is very usefull when the AFS cell is linked to a DFS 
-                * cell and this system does not also have DFS. 
-                */
-
-               if (!status && linked && linkedcell[0]) {
-                               strncpy(linkedcell2,linkedcell,MAXCELLCHARS);
-                           if (dflag) {
-                               printf("Linked cell: %s\n", linkedcell);
-                           }
-                               status = auth_to_cell(context, linkedcell2, NULL);
-               }
+       struct passwd *pwd;
 
-               /*
-                * Local hack - if the person has a file in their home
-                * directory called ".xlog", read that for a list of
-                * extra cells to authenticate to
-                */
+       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) {
+           afs_dprintf("Linked cell: %s\n", linkedcell);
+           status = auth_to_cell(context, config, linkedcell, NULL, NULL);
+       }
+       if (linkedcell) {
+           free(linkedcell);
+           linkedcell = NULL;
+       }
 
-               if ((pwd = getpwuid(getuid())) != NULL) {
-                   struct stat sbuf;
-                   FILE *f;
-                   char fcell[100], xlog_path[512];
+       /*
+        * Local hack - if the person has a file in their home
+        * directory called ".xlog", read that for a list of
+        * extra cells to authenticate to
+        */
 
-                   strcpy(xlog_path, pwd->pw_dir);
-                   strcat(xlog_path, "/.xlog");
+       if ((pwd = getpwuid(getuid())) != NULL) {
+           struct stat sbuf;
+           FILE *f;
+           char fcell[100], xlog_path[512];
 
-                   if ((stat(xlog_path, &sbuf) == 0) &&
-                       ((f = fopen(xlog_path, "r")) != NULL)) {
+           strlcpy(xlog_path, pwd->pw_dir, sizeof(xlog_path));
+           strlcat(xlog_path, "/.xlog", sizeof(xlog_path));
 
-                       if (dflag) {
-                           printf("Reading %s for cells to "
-                                   "authenticate to.\n", xlog_path);
-                       }
+           if ((stat(xlog_path, &sbuf) == 0) &&
+               ((f = fopen(xlog_path, "r")) != NULL)) {
 
-                       while (fgets(fcell, 100, f) != NULL) {
-                           int auth_status;
+               afs_dprintf("Reading %s for cells to authenticate to.\n",
+                       xlog_path);
 
-                           fcell[strlen(fcell) - 1] = '\0';
+               while (fgets(fcell, 100, f) != NULL) {
+                   int auth_status;
 
-                           if (dflag) {
-                               printf("Found cell %s in %s.\n",
-                                       fcell, xlog_path);
-                           }
+                   fcell[strlen(fcell) - 1] = '\0';
 
-                           auth_status = auth_to_cell(context, fcell, NULL);
-                           if (status == AKLOG_SUCCESS)
-                               status = auth_status;
-                           else
-                               status = AKLOG_SOMETHINGSWRONG;
-                       }
-                   }
+                   afs_dprintf("Found cell %s in %s.\n", fcell, xlog_path);
+
+                   auth_status = auth_to_cell(context, config, fcell, NULL, NULL);
+                   if (status == AKLOG_SUCCESS)
+                       status = auth_status;
+                   else
+                       status = AKLOG_SOMETHINGSWRONG;
                }
+           }
        }
+    }
     else {
        /* 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)))
+           status = auth_to_cell(context, config, cellinfo.cell,
+                                 cellinfo.realm, &linkedcell);
+           if (status) {
                somethingswrong++;
-               else {
-                       if (linked && linkedcell[0]) {
-                               strncpy(linkedcell2,linkedcell,MAXCELLCHARS);
-                if (dflag) {
-                    printf("Linked cell: %s\n",
-                        linkedcell);
-                }
-                               if ((status = auth_to_cell(context,linkedcell2,
-                                                        cellinfo.realm)))
-                               somethingswrong++;
-                       }
+           } else {
+               if (linked && linkedcell != NULL) {
+                   afs_dprintf("Linked cell: %s\n", linkedcell);
+                   status = auth_to_cell(context, config, linkedcell,
+                                         cellinfo.realm, NULL);
+                   if (status)
+                       somethingswrong++;
+               }
+               if (linkedcell != NULL) {
+                   free(linkedcell);
+                   linkedcell = NULL;
                }
+           }
        }
-       
+
        /* 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.
         */
@@ -1553,7 +1776,7 @@ int 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);
        }
@@ -1567,30 +1790,32 @@ int main(int argc, char *argv[])
     exit(status);
 }
 
-static int isdir(char *path, unsigned char *val)
+static int
+isdir(char *path, unsigned char *val)
 {
     struct stat statbuf;
 
     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 get_credv5_akimpersonate(krb5_context context,
-                                               char* keytab,
-                                               krb5_principal service_principal,
-                                               krb5_principal client_principal,
-                                               time_t starttime,
-                                               time_t endtime,
-                                               int *allowed_enctypes,
-                                               int *paddress,
-                                               krb5_creds** out_creds /* out */ )
+static krb5_error_code
+get_credv5_akimpersonate(krb5_context context,
+                        char* keytab,
+                        krb5_principal service_principal,
+                        krb5_principal client_principal,
+                        time_t starttime,
+                        time_t endtime,
+                        int *allowed_enctypes,
+                        int *paddress,
+                        krb5_creds** out_creds /* out */ )
 {
 #if defined(USING_HEIMDAL) || (defined(HAVE_ENCODE_KRB5_ENC_TKT) && defined(HAVE_ENCODE_KRB5_TICKET) && defined(HAVE_KRB5_C_ENCRYPT))
     krb5_error_code code;
@@ -1607,14 +1832,14 @@ static krb5_error_code 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;
@@ -1664,7 +1889,7 @@ static krb5_error_code get_credv5_akimpersonate(krb5_context context,
         krb5_keytab_entry new[1];
         int best = -1;
         memset(new, 0, sizeof *new);
-        if ((code == krb5_kt_start_seq_get(context, kt, cursor))) {
+        if ((code = krb5_kt_start_seq_get(context, kt, cursor))) {
             afs_com_err(progname, code, "while starting keytab scan");
             goto cleanup;
         }
@@ -1737,7 +1962,7 @@ static krb5_error_code 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) {
@@ -1789,8 +2014,8 @@ static krb5_error_code 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";
@@ -1811,7 +2036,7 @@ static krb5_error_code 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
@@ -1895,7 +2120,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;
@@ -1903,29 +2127,27 @@ out:
 }
 
 
-static krb5_error_code get_credv5(krb5_context context, 
-                                 char *name, char *inst, char *realm,
-                                 krb5_creds **creds)
+static krb5_error_code
+get_credv5(krb5_context context, char *name, char *inst, char *realm,
+          krb5_creds **creds)
 {
     krb5_creds increds;
     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);
@@ -1941,12 +2163,13 @@ static krb5_error_code get_credv5(krb5_context context,
        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 (do524)
+       /* 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
@@ -1962,19 +2185,24 @@ static krb5_error_code get_credv5(krb5_context context,
                                     creds /* out */);
     } else {
        r = krb5_get_credentials(context, 0, _krb425_ccache, &increds, creds);
-    }    
+    }
     return r;
 }
 
 
-static int get_user_realm(krb5_context context, char *realm)
+static int
+get_user_realm(krb5_context context, char **realm)
 {
     static krb5_principal client_principal = 0;
-    int i;
     krb5_error_code r = 0;
 
-    if (!_krb425_ccache)
-        krb5_cc_default(context, &_krb425_ccache);
+    *realm = NULL;
+
+    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);
@@ -1985,10 +2213,9 @@ static int get_user_realm(krb5_context context, char *realm)
            return r;
     }
 
-    i = realm_len(context, client_principal);
-    if (i > REALM_SZ-1) i = REALM_SZ-1;
-    strncpy(realm,realm_data(context, client_principal), i);
-    realm[i] = 0;
+    *realm = extract_realm(context, client_principal);
+    if (*realm == NULL)
+       return ENOMEM;
 
     return(r);
 }