Windows: remove unused variables in fs.c
[openafs.git] / src / WINNT / afsd / fs.c
index d1f437d..8423fb9 100644 (file)
@@ -1,18 +1,23 @@
 /*
  * Copyright 2000, International Business Machines Corporation and others.
  * All Rights Reserved.
- * 
+ *
  * This software has been released under the terms of the IBM Public
  * License.  For details, see the LICENSE file in the top-level source
  * directory or online at http://www.openafs.org/dl/license10.html
  */
 
+#include <afsconfig.h>
 #include <afs/param.h>
+#include <roken.h>
+
 #include <afs/stds.h>
 #include <afs/com_err.h>
+#include <afs/cmd.h>
 
 #include <windows.h>
 #include <stdlib.h>
+#include <sys/stat.h>
 #include <malloc.h>
 #include <string.h>
 #include <strsafe.h>
@@ -20,7 +25,7 @@
 #include <time.h>
 #include <winsock2.h>
 #include <errno.h>
-#include <assert.h>
+#include <afs/afs_assert.h>
 #include <rx/rx_globals.h>
 
 #include <osi.h>
 
 #include "fs.h"
 #include "fs_utils.h"
+#include "fs_acl.h"
 #include "cmd.h"
 #include "afsd.h"
 #include "cm_ioctl.h"
+#include "parsemode.h"
 
-#define MAXNAME 100
 #define MAXINSIZE 1300    /* pioctl complains if data is larger than this */
 #define VMSGSIZE 128      /* size of msg buf in volume hdr */
 #define CELL_MAXNAMELEN                256
 #define MAXHOSTCHARS           64
 
+static char pn[] = "fs";
+
 static char space[AFS_PIOCTL_MAXSIZE];
-static char tspace[1024];
 
 static struct ubik_client *uclient;
 
+/* some forward references */
+static int SetVolCmd(struct cmd_syndesc *as, void *arock);
+
+static int GetCellName(char *cellNamep, struct afsconf_cell *infop);
+static afs_int32 GetCell(char *fname, char *cellname, size_t cell_len);
+
+static int VLDBInit(int noAuthFlag, struct afsconf_cell *infop);
 static int GetClientAddrsCmd(struct cmd_syndesc *asp, void *arock);
 static int SetClientAddrsCmd(struct cmd_syndesc *asp, void *arock);
 static int FlushMountCmd(struct cmd_syndesc *asp, void *arock);
 static int RxStatProcCmd(struct cmd_syndesc *asp, void *arock);
 static int RxStatPeerCmd(struct cmd_syndesc *asp, void *arock);
 
-extern struct cmd_syndesc *cmd_CreateSyntax();
-
 static int MemDumpCmd(struct cmd_syndesc *asp, void *arock);
 static int CSCPolicyCmd(struct cmd_syndesc *asp, void *arock);
 static int MiniDumpCmd(struct cmd_syndesc *asp, void *arock);
 
-static char pn[] = "fs";
 static int rxInitDone = 0;
 
 /*
- * Character to use between name and rights in printed representation for
- * DFS ACL's.
- */
-#define DFS_SEPARATOR  ' '
-
-typedef char sec_rgy_name_t[1025];     /* A DCE definition */
-
-struct Acl {
-    int dfs;           /* Originally true if a dfs acl; now also the type
-                         * of the acl (1, 2, or 3, corresponding to object,
-                         * initial dir, or initial object). */
-    sec_rgy_name_t cell; /* DFS cell name */
-    int nplus;
-    int nminus;
-    struct AclEntry *pluslist;
-    struct AclEntry *minuslist;
-};
-
-struct AclEntry {
-    struct AclEntry *next;
-    char name[MAXNAME];
-    afs_int32 rights;
-};
-
-static void 
-ZapAcl (struct Acl *acl)
-{
-    if (!acl)
-        return;
-
-    ZapList(acl->pluslist);
-    ZapList(acl->minuslist);
-    free(acl);
-}
-
-/*
- * Mods for the AFS/DFS protocol translator.
- *
- * DFS rights. It's ugly to put these definitions here, but they 
- * *cannot* change, because they're part of the wire protocol.
- * In any event, the protocol translator will guarantee these
- * assignments for AFS cache managers.
- */
-#define DFS_READ          0x01
-#define DFS_WRITE         0x02
-#define DFS_EXECUTE       0x04
-#define DFS_CONTROL       0x08
-#define DFS_INSERT        0x10
-#define DFS_DELETE        0x20
-
-/* the application definable ones (backwards from AFS) */
-#define DFS_USR0 0x80000000      /* "A" bit */
-#define DFS_USR1 0x40000000      /* "B" bit */
-#define DFS_USR2 0x20000000      /* "C" bit */
-#define DFS_USR3 0x10000000      /* "D" bit */
-#define DFS_USR4 0x08000000      /* "E" bit */
-#define DFS_USR5 0x04000000      /* "F" bit */
-#define DFS_USR6 0x02000000      /* "G" bit */
-#define DFS_USR7 0x01000000      /* "H" bit */
-#define DFS_USRALL     (DFS_USR0 | DFS_USR1 | DFS_USR2 | DFS_USR3 |\
-                        DFS_USR4 | DFS_USR5 | DFS_USR6 | DFS_USR7)
-
-/*
  * Offset of -id switch in command structure for various commands.
  * The -if switch is the next switch always.
  */
@@ -138,7 +86,7 @@ static int parm_setacl_id, parm_copyacl_id, parm_listacl_id;
  * return 0, 1 or 2, as appropriate. Abort if both switches are present.
  */
 /* int id; Offset of -id switch; -if is next switch */
-static int 
+static int
 getidf(struct cmd_syndesc *as, int id)
 {
     int idf = 0;
@@ -162,182 +110,129 @@ static int
 PRights(afs_int32 arights, int dfs)
 {
     if (!dfs) {
-       if (arights & PRSFS_READ) 
+       if (arights & PRSFS_READ)
             printf("r");
-       if (arights & PRSFS_LOOKUP) 
+       if (arights & PRSFS_LOOKUP)
             printf("l");
-       if (arights & PRSFS_INSERT) 
+       if (arights & PRSFS_INSERT)
             printf("i");
-       if (arights & PRSFS_DELETE) 
+       if (arights & PRSFS_DELETE)
             printf("d");
-       if (arights & PRSFS_WRITE) 
+       if (arights & PRSFS_WRITE)
             printf("w");
-       if (arights & PRSFS_LOCK) 
+       if (arights & PRSFS_LOCK)
             printf("k");
-       if (arights & PRSFS_ADMINISTER) 
+       if (arights & PRSFS_ADMINISTER)
             printf("a");
-       if (arights & PRSFS_USR0) 
+       if (arights & PRSFS_USR0)
             printf("A");
-       if (arights & PRSFS_USR1) 
+       if (arights & PRSFS_USR1)
             printf("B");
-       if (arights & PRSFS_USR2) 
+       if (arights & PRSFS_USR2)
             printf("C");
-       if (arights & PRSFS_USR3) 
+       if (arights & PRSFS_USR3)
             printf("D");
-       if (arights & PRSFS_USR4) 
+       if (arights & PRSFS_USR4)
             printf("E");
-       if (arights & PRSFS_USR5) 
+       if (arights & PRSFS_USR5)
             printf("F");
-       if (arights & PRSFS_USR6) 
+       if (arights & PRSFS_USR6)
             printf("G");
-       if (arights & PRSFS_USR7) 
+       if (arights & PRSFS_USR7)
             printf("H");
     } else {
-       if (arights & DFS_READ) 
+       if (arights & DFS_READ)
             printf("r");
-        else 
+        else
             printf("-");
-       if (arights & DFS_WRITE) 
-            printf("w"); 
-        else 
+       if (arights & DFS_WRITE)
+            printf("w");
+        else
             printf("-");
-       if (arights & DFS_EXECUTE) 
-            printf("x"); 
-        else 
+       if (arights & DFS_EXECUTE)
+            printf("x");
+        else
             printf("-");
-       if (arights & DFS_CONTROL) 
-            printf("c"); 
-        else 
+       if (arights & DFS_CONTROL)
+            printf("c");
+        else
             printf("-");
-       if (arights & DFS_INSERT) 
-            printf("i"); 
-        else 
+       if (arights & DFS_INSERT)
+            printf("i");
+        else
             printf("-");
-       if (arights & DFS_DELETE) 
-            printf("d"); 
-        else 
+       if (arights & DFS_DELETE)
+            printf("d");
+        else
             printf("-");
-       if (arights & (DFS_USRALL)) 
+       if (arights & (DFS_USRALL))
             printf("+");
-       if (arights & DFS_USR0) 
+       if (arights & DFS_USR0)
             printf("A");
-       if (arights & DFS_USR1) 
+       if (arights & DFS_USR1)
             printf("B");
-       if (arights & DFS_USR2) 
+       if (arights & DFS_USR2)
             printf("C");
-       if (arights & DFS_USR3) 
+       if (arights & DFS_USR3)
             printf("D");
-       if (arights & DFS_USR4) 
+       if (arights & DFS_USR4)
             printf("E");
-       if (arights & DFS_USR5) 
+       if (arights & DFS_USR5)
             printf("F");
-       if (arights & DFS_USR6) 
+       if (arights & DFS_USR6)
             printf("G");
-       if (arights & DFS_USR7) 
+       if (arights & DFS_USR7)
             printf("H");
-    }  
-    return 0;
-}
-
-/* this function returns TRUE (1) if the file is in AFS, otherwise false (0) */
-static int 
-InAFS(char *apath)
-{
-    struct ViceIoctl blob;
-    cm_ioctlQueryOptions_t options;
-    cm_fid_t fid;
-    afs_int32 code;
-
-    memset(&options, 0, sizeof(options));
-    options.size = sizeof(options);
-    options.field_flags |= CM_IOCTL_QOPTS_FIELD_LITERAL;
-    options.literal = 1;
-    blob.in_size = options.size;    /* no variable length data */
-    blob.in = &options;
-    blob.out_size = sizeof(cm_fid_t);
-    blob.out = (char *) &fid;
-
-    code = pioctl_utf8(apath, VIOCGETFID, &blob, 1);
-    if (code) {
-       if ((errno == EINVAL) || (errno == ENOENT)) 
-            return 0;
-    }
-    return 1;
-}
-
-static int 
-IsFreelanceRoot(char *apath)
-{
-    struct ViceIoctl blob;
-    afs_int32 code;
-
-    blob.in_size = 0;
-    blob.out_size = AFS_PIOCTL_MAXSIZE;
-    blob.out = space;
-
-    code = pioctl_utf8(apath, VIOC_FILE_CELL_NAME, &blob, 1);
-    if (code == 0) {
-        return !cm_strnicmp_utf8N("Freelance.Local.Root",space, blob.out_size);
-    }
-    return 1;   /* assume it is because it is more restrictive that way */
-}
-
-/* return a static pointer to a buffer */
-static char *
-Parent(char *apath)
-{
-    char *tp;
-    if( FAILED(StringCbCopy(tspace, sizeof(tspace), apath))) {
-        fprintf (stderr, "tspace - not enough space");
-        exit(1);
     }
-    tp = strrchr(tspace, '\\');
-    if (tp) {
-       *(tp+1) = 0;    /* lv trailing slash so Parent("k:\foo") is "k:\" not "k:" */
-    }
-    else {
-       fs_ExtractDriveLetter(apath, tspace);
-       if( FAILED(StringCbCat(tspace, sizeof(tspace), "."))) {
-           fprintf (stderr, "tspace - not enough space");
-           exit(1);
-       }
-    }
-    return tspace;
+    return 0;
 }
 
-enum rtype {add, destroy, deny};
-
-static afs_int32 
+static afs_int32
 Convert(char *arights, int dfs, enum rtype *rtypep)
 {
     afs_int32 mode;
     char tc;
+    char *tcp;
 
     *rtypep = add;     /* add rights, by default */
 
+                                /* analyze last character of string   */
+    tcp = arights + strlen(arights);
+    if ( tcp-- > arights ) {    /* assure non-empty string            */
+        if ( *tcp == '+' )
+            *rtypep = reladd;   /* '+' indicates more rights          */
+        else if ( *tcp == '-' )
+            *rtypep = reldel;   /* '-' indicates less rights          */
+        else if ( *tcp == '=' )
+            *rtypep = add;      /* '=' also allows old behaviour      */
+        else
+            tcp++;              /* back to original null byte         */
+        *tcp = '\0';            /* do not disturb old strcmp-s        */
+    }
+
     if (dfs) {
        if (!strcmp(arights, "null")) {
            *rtypep = deny;
            return 0;
        }
-       if (!strcmp(arights,"read")) 
+       if (!strcmp(arights,"read"))
             return DFS_READ | DFS_EXECUTE;
-       if (!strcmp(arights, "write")) 
-            return DFS_READ | DFS_EXECUTE | DFS_INSERT | DFS_DELETE | 
+       if (!strcmp(arights, "write"))
+            return DFS_READ | DFS_EXECUTE | DFS_INSERT | DFS_DELETE |
                 DFS_WRITE;
-       if (!strcmp(arights, "all")) 
-            return DFS_READ | DFS_EXECUTE | DFS_INSERT | DFS_DELETE | 
+       if (!strcmp(arights, "all"))
+            return DFS_READ | DFS_EXECUTE | DFS_INSERT | DFS_DELETE |
                 DFS_WRITE | DFS_CONTROL;
     } else {
-       if (!strcmp(arights,"read")) 
+       if (!strcmp(arights,"read"))
             return PRSFS_READ | PRSFS_LOOKUP;
-       if (!strcmp(arights, "write")) 
-            return PRSFS_READ | PRSFS_LOOKUP | PRSFS_INSERT | PRSFS_DELETE | 
+       if (!strcmp(arights, "write"))
+            return PRSFS_READ | PRSFS_LOOKUP | PRSFS_INSERT | PRSFS_DELETE |
                 PRSFS_WRITE | PRSFS_LOCK;
-       if (!strcmp(arights, "mail")) 
+       if (!strcmp(arights, "mail"))
             return PRSFS_INSERT | PRSFS_LOCK | PRSFS_LOOKUP;
-       if (!strcmp(arights, "all")) 
-            return PRSFS_READ | PRSFS_LOOKUP | PRSFS_INSERT | PRSFS_DELETE | 
+       if (!strcmp(arights, "all"))
+            return PRSFS_READ | PRSFS_LOOKUP | PRSFS_INSERT | PRSFS_DELETE |
                 PRSFS_WRITE | PRSFS_LOCK | PRSFS_ADMINISTER;
     }
     if (!strcmp(arights, "none")) {
@@ -345,76 +240,77 @@ Convert(char *arights, int dfs, enum rtype *rtypep)
        return 0;
     }
     mode = 0;
-    for(; (tc = *arights) != '\0'; arights++) {
+    tcp = arights;
+    while ((tc = *tcp++)) {
        if (dfs) {
-           if (tc == '-') 
+           if (tc == '-')
                 continue;
-           else if (tc == 'r') 
+           else if (tc == 'r')
                 mode |= DFS_READ;
-           else if (tc == 'w') 
+           else if (tc == 'w')
                 mode |= DFS_WRITE;
-           else if (tc == 'x') 
+           else if (tc == 'x')
                 mode |= DFS_EXECUTE;
-           else if (tc == 'c') 
+           else if (tc == 'c')
                 mode |= DFS_CONTROL;
-           else if (tc == 'i') 
+           else if (tc == 'i')
                 mode |= DFS_INSERT;
-           else if (tc == 'd') 
+           else if (tc == 'd')
                 mode |= DFS_DELETE;
-           else if (tc == 'A') 
+           else if (tc == 'A')
                 mode |= DFS_USR0;
-           else if (tc == 'B') 
+           else if (tc == 'B')
                 mode |= DFS_USR1;
-           else if (tc == 'C') 
+           else if (tc == 'C')
                 mode |= DFS_USR2;
-           else if (tc == 'D') 
+           else if (tc == 'D')
                 mode |= DFS_USR3;
-           else if (tc == 'E') 
+           else if (tc == 'E')
                 mode |= DFS_USR4;
-           else if (tc == 'F') 
+           else if (tc == 'F')
                 mode |= DFS_USR5;
-           else if (tc == 'G') 
+           else if (tc == 'G')
                 mode |= DFS_USR6;
-           else if (tc == 'H') 
+           else if (tc == 'H')
                 mode |= DFS_USR7;
            else {
-               fprintf(stderr, "%s: illegal DFS rights character '%c'.\n", 
+               fprintf(stderr, "%s: illegal DFS rights character '%c'.\n",
                          pn, tc);
                exit(1);
            }
        } else {
-           if (tc == 'r') 
+           if (tc == 'r')
                 mode |= PRSFS_READ;
-           else if (tc == 'l') 
+           else if (tc == 'l')
                 mode |= PRSFS_LOOKUP;
-           else if (tc == 'i') 
+           else if (tc == 'i')
                 mode |= PRSFS_INSERT;
-           else if (tc == 'd') 
+           else if (tc == 'd')
                 mode |= PRSFS_DELETE;
-           else if (tc == 'w') 
+           else if (tc == 'w')
                 mode |= PRSFS_WRITE;
-           else if (tc == 'k') 
+           else if (tc == 'k')
                 mode |= PRSFS_LOCK;
-           else if (tc == 'a') 
+           else if (tc == 'a')
                 mode |= PRSFS_ADMINISTER;
-           else if (tc == 'A') 
+           else if (tc == 'A')
                 mode |= PRSFS_USR0;
-           else if (tc == 'B') 
+           else if (tc == 'B')
                 mode |= PRSFS_USR1;
-           else if (tc == 'C') 
+           else if (tc == 'C')
                 mode |= PRSFS_USR2;
-           else if (tc == 'D') 
+           else if (tc == 'D')
                 mode |= PRSFS_USR3;
-           else if (tc == 'E') 
+           else if (tc == 'E')
                 mode |= PRSFS_USR4;
-           else if (tc == 'F') 
+           else if (tc == 'F')
                 mode |= PRSFS_USR5;
-           else if (tc == 'G') 
+           else if (tc == 'G')
                 mode |= PRSFS_USR6;
-           else if (tc == 'H') 
+           else if (tc == 'H')
                 mode |= PRSFS_USR7;
            else {
-               fprintf(stderr, "%s: illegal rights character '%c'.\n", pn, 
+               fprintf(stderr, "%s: illegal rights character '%c'.\n", pn,
                          tc);
                exit(1);
            }
@@ -423,25 +319,14 @@ Convert(char *arights, int dfs, enum rtype *rtypep)
     return mode;
 }
 
-static struct AclEntry *
-FindList (struct AclEntry *alist, char *aname)
-{
-    while (alist) {
-        if (!strcasecmp(alist->name, aname)) 
-            return alist;
-        alist = alist->next;
-    }
-    return 0;
-}
-
 /* if no parm specified in a particular slot, set parm to be "." instead */
-static void 
+static void
 SetDotDefault(struct cmd_item **aitemp)
 {
     struct cmd_item *ti;
     int len_data = 2;
 
-    if (*aitemp) 
+    if (*aitemp)
         return;                        /* already has value */
     /* otherwise, allocate an item representing "." */
     ti = (struct cmd_item *) malloc(sizeof(struct cmd_item));
@@ -456,232 +341,6 @@ SetDotDefault(struct cmd_item **aitemp)
     *aitemp = ti;
 }
 
-static void 
-ChangeList (struct Acl *al, afs_int32 plus, char *aname, afs_int32 arights)
-{
-    struct AclEntry *tlist;
-    tlist = (plus ? al->pluslist : al->minuslist);
-    tlist = FindList (tlist, aname);
-    if (tlist) {
-        /* Found the item already in the list. */
-        tlist->rights = arights;
-        if (plus)
-            al->nplus -= PruneList(&al->pluslist, al->dfs);
-        else
-            al->nminus -= PruneList(&al->minuslist, al->dfs);
-        return;
-    }
-    /* Otherwise we make a new item and plug in the new data. */
-    tlist = (struct AclEntry *) malloc(sizeof (struct AclEntry));
-    assert(tlist);
-    if( FAILED(StringCbCopy(tlist->name, sizeof(tlist->name), aname))) {
-       fprintf (stderr, "name - not enough space");
-        exit(1);
-    }
-    tlist->rights = arights;
-    if (plus) {
-        tlist->next = al->pluslist;
-        al->pluslist = tlist;
-        al->nplus++;
-        if (arights == 0 || arights == -1)
-           al->nplus -= PruneList(&al->pluslist, al->dfs);
-    } else {
-        tlist->next = al->minuslist;
-        al->minuslist = tlist;
-        al->nminus++;
-        if (arights == 0) 
-            al->nminus -= PruneList(&al->minuslist, al->dfs);
-    }
-}
-
-static void 
-ZapList (struct AclEntry *alist)
-{
-    struct AclEntry *tp, *np;
-    for (tp = alist; tp; tp = np) {
-        np = tp->next;
-        free(tp);
-    }
-}
-
-static int 
-PruneList (struct AclEntry **ae, int dfs)
-{
-    struct AclEntry **lp;
-    struct AclEntry *te, *ne;
-    afs_int32 ctr;
-    ctr = 0;
-    lp = ae;
-    for(te = *ae;te;te=ne) {
-        if ((!dfs && te->rights == 0) || te->rights == -1) {
-            *lp = te->next;
-            ne = te->next;
-            free(te);
-            ctr++;
-       } else {
-            ne = te->next;
-            lp = &te->next;
-       }
-    }
-    return ctr;
-}
-
-static char *
-SkipLine (char *astr)
-{
-    while (*astr !='\n') 
-        astr++;
-    astr++;
-    return astr;
-}
-
-/*
- * Create an empty acl, taking into account whether the acl pointed
- * to by astr is an AFS or DFS acl. Only parse this minimally, so we
- * can recover from problems caused by bogus ACL's (in that case, always
- * assume that the acl is AFS: for DFS, the user can always resort to
- * acl_edit, but for AFS there may be no other way out).
- */
-static struct Acl *
-EmptyAcl(char *astr)
-{
-    struct Acl *tp;
-    int junk;
-
-    tp = (struct Acl *)malloc(sizeof (struct Acl));
-    assert(tp);
-    tp->nplus = tp->nminus = 0;
-    tp->pluslist = tp->minuslist = 0;
-    tp->dfs = 0;
-#if _MSC_VER < 1400
-    if (astr == NULL || sscanf(astr, "%d dfs:%d %s", &junk, &tp->dfs, tp->cell) <= 0) {
-        tp->dfs = 0;
-        tp->cell[0] = '\0';
-    }
-#else
-    if (astr == NULL || sscanf_s(astr, "%d dfs:%d %s", &junk, &tp->dfs, tp->cell, sizeof(tp->cell)) <= 0) {
-        tp->dfs = 0;
-        tp->cell[0] = '\0';
-    }
-#endif
-    return tp;
-}
-
-static struct Acl *
-ParseAcl (char *astr, int astr_size)
-{
-    int nplus, nminus, i, trights, ret;
-    size_t len;
-    char tname[MAXNAME];
-    struct AclEntry *first, *next, *last, *tl;
-    struct Acl *ta;
-
-    ta = EmptyAcl(NULL);
-    if( FAILED(StringCbLength(astr, astr_size, &len))) {
-        fprintf (stderr, "StringCbLength failure on astr");
-        exit(1);
-    }
-    if (astr == NULL || len == 0)
-        return ta;
-
-#if _MSC_VER < 1400
-    ret = sscanf(astr, "%d dfs:%d %s", &ta->nplus, &ta->dfs, ta->cell);
-#else
-    ret = sscanf_s(astr, "%d dfs:%d %s", &ta->nplus, &ta->dfs, ta->cell, sizeof(ta->cell));
-#endif
-    if (ret <= 0) {
-        free(ta);
-        return NULL;
-    }
-    astr = SkipLine(astr);
-#if _MSC_VER < 1400
-    ret = sscanf(astr, "%d", &ta->nminus);
-#else
-    ret = sscanf_s(astr, "%d", &ta->nminus);
-#endif
-    if (ret <= 0) {
-        free(ta);
-        return NULL;
-    }
-    astr = SkipLine(astr);
-
-    nplus = ta->nplus;
-    nminus = ta->nminus;
-
-    last = 0;
-    first = 0;
-    for(i=0;i<nplus;i++) {
-#if _MSC_VER < 1400
-        ret = sscanf(astr, "%100s %d", tname, &trights); 
-#else
-        ret = sscanf_s(astr, "%100s %d", tname, sizeof(tname), &trights);
-#endif
-        if (ret <= 0)
-            goto nplus_err;
-        astr = SkipLine(astr);
-        tl = (struct AclEntry *) malloc(sizeof (struct AclEntry));
-        if (tl == NULL)
-            goto nplus_err;
-        if (!first) 
-            first = tl;
-        if( FAILED(StringCbCopy(tl->name, sizeof(tl->name), tname))) {
-            fprintf (stderr, "name - not enough space");
-            exit(1);
-        }
-        tl->rights = trights;
-        tl->next = 0;
-        if (last) 
-            last->next = tl;
-        last = tl;
-    }
-    ta->pluslist = first;
-
-    last = 0;
-    first = 0;
-    for(i=0;i<nminus;i++) {
-#if _MSC_VER < 1400
-        ret = sscanf(astr, "%100s %d", tname, &trights);
-#else
-        ret = sscanf_s(astr, "%100s %d", tname, sizeof(tname), &trights);
-#endif
-        if (ret <= 0)
-            goto nminus_err;
-        astr = SkipLine(astr);
-        tl = (struct AclEntry *) malloc(sizeof (struct AclEntry));
-        if (tl == NULL)
-            goto nminus_err;
-        if (!first) 
-            first = tl;
-        if( FAILED(StringCbCopy(tl->name, sizeof(tl->name), tname))) {
-            fprintf (stderr, "name - not enough space");
-            exit(1);
-        }
-        tl->rights = trights;
-        tl->next = 0;
-        if (last) 
-            last->next = tl;
-        last = tl;
-    }
-    ta->minuslist = first;
-
-    return ta;
-
-  nminus_err:
-    for (;first; first = next) {
-        next = first->next;
-        free(first);
-    }   
-    first = ta->pluslist;
-
-  nplus_err:
-    for (;first; first = next) {
-        next = first->next;
-        free(first);
-    }   
-    free(ta);
-    return NULL;
-}
-
 static int
 PrintStatus(VolumeStatus *status, char *name, char *motd, char *offmsg)
 {
@@ -691,9 +350,9 @@ PrintStatus(VolumeStatus *status, char *name, char *motd, char *offmsg)
     if (*motd != 0)
        printf("Current message of the day is %s\n",motd);
     printf("Current disk quota is ");
-    if (status->MaxQuota != 0) 
+    if (status->MaxQuota != 0)
         printf("%d\n", status->MaxQuota);
-    else 
+    else
         printf("unlimited\n");
     printf("Current blocks used are %d\n",status->BlocksInUse);
     printf("The partition has %d blocks available out of %d\n",
@@ -718,17 +377,17 @@ QuickPrintStatus(VolumeStatus *status, char *name)
     if (QuotaUsed > 90.0){
        printf(" %5.0f%%<<", QuotaUsed);
        WARN = 1;
-    } else 
+    } else
         printf(" %5.0f%%  ", QuotaUsed);
     PartUsed = (100.0 - ((((double)status->PartBlocksAvail)/status->PartMaxBlocks) * 100.0));
     if (PartUsed > 97.0){
        printf(" %9.0f%%<<", PartUsed);
        WARN = 1;
-    } else 
+    } else
         printf(" %9.0f%%  ", PartUsed);
     if (WARN){
        printf("  <<WARNING\n");
-    } else 
+    } else
         printf("\n");
     return 0;
 }
@@ -741,63 +400,20 @@ QuickPrintSpace(VolumeStatus *status, char *name)
     printf("%-25.25s",name);
 
     printf("%10d%10d%10d", status->PartMaxBlocks, status->PartMaxBlocks - status->PartBlocksAvail, status->PartBlocksAvail);
-       
+
     PartUsed = (100.0 - ((((double)status->PartBlocksAvail)/status->PartMaxBlocks) * 100.0));
     if (PartUsed > 90.0){
        printf(" %4.0f%%<<", PartUsed);
        WARN = 1;
-    } else 
+    } else
         printf(" %4.0f%%  ", PartUsed);
     if (WARN){
        printf("  <<WARNING\n");
-    } else 
+    } else
         printf("\n");
     return 0;
 }
 
-static char *
-AclToString(struct Acl *acl)
-{
-    static char mydata[AFS_PIOCTL_MAXSIZE];
-    char tstring[AFS_PIOCTL_MAXSIZE];
-    char dfsstring[30];
-    struct AclEntry *tp;
-
-    if (acl->dfs) {
-        if( FAILED(StringCbPrintf(dfsstring, sizeof(dfsstring), " dfs:%d %s", acl->dfs, acl->cell))) {
-            fprintf (stderr, "dfsstring - cannot be populated");
-            exit(1);
-        }
-    } else {
-        dfsstring[0] = '\0';
-    }
-    if( FAILED(StringCbPrintf(mydata, sizeof(mydata), "%d%s\n%d\n", acl->nplus, dfsstring, acl->nminus))) {
-        fprintf (stderr, "mydata - cannot be populated");
-        exit(1);
-    }
-    for (tp = acl->pluslist;tp;tp=tp->next) {
-        if( FAILED(StringCbPrintf(tstring, sizeof(tstring), "%s %d\n", tp->name, tp->rights))) {
-            fprintf (stderr, "tstring - cannot be populated");
-            exit(1);
-        }
-        if( FAILED(StringCbCat(mydata, sizeof(mydata), tstring))) {
-            fprintf (stderr, "mydata - not enough space");
-            exit(1);
-        }
-    }
-    for (tp = acl->minuslist;tp;tp=tp->next) {
-        if( FAILED(StringCbPrintf(tstring, sizeof(tstring), "%s %d\n", tp->name, tp->rights))) {
-            fprintf (stderr, "tstring - cannot be populated");
-            exit(1);
-        }
-        if( FAILED(StringCbCat(mydata, sizeof(mydata), tstring))) {
-            fprintf (stderr, "mydata - not enough space");
-            exit(1);
-        }
-    }
-    return mydata;
-}
-
 static DWORD IsFreelance(void)
 {
     HKEY  parmKey;
@@ -816,177 +432,6 @@ static DWORD IsFreelance(void)
     return enabled;
 }
 
-#define NETBIOSNAMESZ 1024
-static const char * NetbiosName(void)
-{
-    static char buffer[NETBIOSNAMESZ] = "AFS";
-    HKEY  parmKey;
-    DWORD code;
-    DWORD dummyLen;
-    DWORD enabled = 0;
-
-    code = RegOpenKeyEx(HKEY_LOCAL_MACHINE, AFSREG_CLT_SVC_PARAM_SUBKEY,
-                         0, (IsWow64()?KEY_WOW64_64KEY:0)|KEY_QUERY_VALUE, &parmKey);
-    if (code == ERROR_SUCCESS) {
-        dummyLen = sizeof(buffer);
-        code = RegQueryValueEx(parmKey, "NetbiosName", NULL, NULL,
-                              buffer, &dummyLen);
-        RegCloseKey (parmKey);
-    } else {
-       if( FAILED(StringCbCopy(buffer, sizeof(buffer), "AFS"))) {
-           fprintf (stderr, "buffer - not enough space");
-            exit(1);
-       }
-    }
-    return buffer;
-}
-
-#define AFSCLIENT_ADMIN_GROUPNAME "AFS Client Admins"
-
-static BOOL IsAdmin (void)
-{
-    static BOOL fAdmin = FALSE;
-    static BOOL fTested = FALSE;
-
-    if (!fTested)
-    {
-        /* Obtain the SID for the AFS client admin group.  If the group does
-         * not exist, then assume we have AFS client admin privileges.
-         */
-        PSID psidAdmin = NULL;
-        DWORD dwSize, dwSize2;
-        char pszAdminGroup[ MAX_COMPUTERNAME_LENGTH + sizeof(AFSCLIENT_ADMIN_GROUPNAME) + 2 ];
-        char *pszRefDomain = NULL;
-        SID_NAME_USE snu = SidTypeGroup;
-
-        dwSize = sizeof(pszAdminGroup);
-
-        if (!GetComputerName(pszAdminGroup, &dwSize)) {
-            /* Can't get computer name.  We return false in this case.
-               Retain fAdmin and fTested. This shouldn't happen.*/
-            return FALSE;
-        }
-
-        dwSize = 0;
-        dwSize2 = 0;
-
-       if( FAILED(StringCbCat(pszAdminGroup, MAX_COMPUTERNAME_LENGTH + sizeof(AFSCLIENT_ADMIN_GROUPNAME) + 2,"\\"))) {
-           fprintf (stderr, "pszAdminGroup - not enough space");
-            exit(1);
-       }
-       if( FAILED(StringCbCat(pszAdminGroup, MAX_COMPUTERNAME_LENGTH +
-           sizeof(AFSCLIENT_ADMIN_GROUPNAME) + 2, AFSCLIENT_ADMIN_GROUPNAME))) {
-           fprintf (stderr, "pszAdminGroup - not enough space");
-            exit(1);
-       }
-
-        LookupAccountName(NULL, pszAdminGroup, NULL, &dwSize, NULL, &dwSize2, &snu);
-        /* that should always fail. */
-
-        if (GetLastError() != ERROR_INSUFFICIENT_BUFFER) {
-            /* if we can't find the group, then we allow the operation */
-            fAdmin = TRUE;
-            return TRUE;
-        }
-
-        if (dwSize == 0 || dwSize2 == 0) {
-            /* Paranoia */
-            fAdmin = TRUE;
-            return TRUE;
-        }
-
-        psidAdmin = (PSID)malloc(dwSize); memset(psidAdmin,0,dwSize);
-        assert(psidAdmin);
-        pszRefDomain = (char *)malloc(dwSize2);
-        assert(pszRefDomain);
-
-        if (!LookupAccountName(NULL, pszAdminGroup, psidAdmin, &dwSize, pszRefDomain, &dwSize2, &snu)) {
-            /* We can't lookup the group now even though we looked it up earlier.  
-               Could this happen? */
-            fAdmin = TRUE;
-        } else {
-            /* Then open our current ProcessToken */
-            HANDLE hToken;
-
-            if (OpenProcessToken (GetCurrentProcess(), TOKEN_QUERY, &hToken))
-            {
-
-                if (!CheckTokenMembership(hToken, psidAdmin, &fAdmin)) {
-                    /* We'll have to allocate a chunk of memory to store the list of
-                     * groups to which this user belongs; find out how much memory
-                     * we'll need.
-                     */
-                    DWORD dwSize = 0;
-                    PTOKEN_GROUPS pGroups;
-
-                    GetTokenInformation (hToken, TokenGroups, NULL, dwSize, &dwSize);
-
-                    pGroups = (PTOKEN_GROUPS)malloc(dwSize);
-                    assert(pGroups);
-
-                    /* Allocate that buffer, and read in the list of groups. */
-                    if (GetTokenInformation (hToken, TokenGroups, pGroups, dwSize, &dwSize))
-                    {
-                        /* Look through the list of group SIDs and see if any of them
-                         * matches the AFS Client Admin group SID.
-                         */
-                        size_t iGroup = 0;
-                        for (; (!fAdmin) && (iGroup < pGroups->GroupCount); ++iGroup)
-                        {
-                            if (EqualSid (psidAdmin, pGroups->Groups[ iGroup ].Sid)) {
-                                fAdmin = TRUE;
-                            }
-                        }
-                    }
-
-                    if (pGroups)
-                        free(pGroups);
-                }
-
-                /* if do not have permission because we were not explicitly listed
-                 * in the Admin Client Group let's see if we are the SYSTEM account
-                 */
-                if (!fAdmin) {
-                    PTOKEN_USER pTokenUser;
-                    SID_IDENTIFIER_AUTHORITY SIDAuth = SECURITY_NT_AUTHORITY;
-                    PSID pSidLocalSystem = 0;
-                    DWORD gle;
-
-                    GetTokenInformation(hToken, TokenUser, NULL, 0, &dwSize);
-
-                    pTokenUser = (PTOKEN_USER)malloc(dwSize);
-                    assert(pTokenUser);
-
-                    if (!GetTokenInformation(hToken, TokenUser, pTokenUser, dwSize, &dwSize))
-                        gle = GetLastError();
-
-                    if (AllocateAndInitializeSid( &SIDAuth, 1,
-                                                  SECURITY_LOCAL_SYSTEM_RID,
-                                                  0, 0, 0, 0, 0, 0, 0,
-                                                  &pSidLocalSystem))
-                    {
-                        if (EqualSid(pTokenUser->User.Sid, pSidLocalSystem)) {
-                            fAdmin = TRUE;
-                        }
-
-                        FreeSid(pSidLocalSystem);
-                    }
-
-                    if ( pTokenUser )
-                        free(pTokenUser);
-                }
-            }
-        }
-
-        free(psidAdmin);
-        free(pszRefDomain);
-
-        fTested = TRUE;
-    }
-
-    return fAdmin;
-}
-
 static int
 SetACLCmd(struct cmd_syndesc *as, void *arock)
 {
@@ -1000,6 +445,7 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
     int idf = getidf(as, parm_setacl_id);
     size_t len;
     int error = 0;
+    char cell[CELL_MAXNAMELEN];
 
     if (as->parms[2].items)
         clear = 1;
@@ -1007,12 +453,24 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
         clear = 0;
     plusp = !(as->parms[3].items);
     for(ti=as->parms[0].items; ti;ti=ti->next) {
+        code = GetCell(ti->data, cell, sizeof(cell));
+       if (code) {
+           fs_Die(errno, ti->data);
+           error = 1;
+            continue;
+       }
+
+        if ( fs_IsFreelanceRoot(ti->data) ) {
+            fprintf(stderr,"%s: ACLs cannot be set on the Freelance root.afs volume.\n", pn);
+            error = 1;
+            continue;
+        }
        blob.out_size = AFS_PIOCTL_MAXSIZE;
        blob.in_size = idf;
        blob.in = blob.out = space;
        code = pioctl_utf8(ti->data, VIOCGETAL, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
             continue;
        }
@@ -1036,9 +494,9 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
        }
         if (ta)
             ZapAcl(ta);
-       if (clear) 
+       if (clear)
             ta = EmptyAcl(space);
-       else 
+       else
             ta = ParseAcl(space, AFS_PIOCTL_MAXSIZE);
         if (!ta) {
             fprintf(stderr,
@@ -1047,7 +505,7 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
             error = 1;
             continue;
         }
-       CleanAcl(ta, ti->data);
+       CleanAcl(ta, cell);
        for(ui=as->parms[1].items; ui; ui=ui->next->next) {
            enum rtype rtype;
            if (!ui->next) {
@@ -1064,11 +522,11 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
                if (!FindList(tlist, ui->data))
                     continue;
            }
-           if (rtype == deny && !ta->dfs) 
+           if (rtype == deny && !ta->dfs)
                 plusp = 0;
-           if (rtype == destroy && ta->dfs) 
+           if (rtype == destroy && ta->dfs)
                 rights = -1;
-           ChangeList(ta, plusp, ui->data, rights);
+           ChangeList(ta, plusp, ui->data, rights, &rtype);
        }
        blob.in = AclToString(ta);
        blob.out_size=0;
@@ -1089,7 +547,7 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
                    if (!fsenv) {
                        fprintf(stderr,
     "%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s",
-    "\nPossible reasons for this include:\n\n",                            
+    "\nPossible reasons for this include:\n\n",
     " -You may have specified an inappropriate combination of rights.\n",
     "  For example, some DFS-supported filesystems may not allow you to\n",
     "  drop the \"c\" right from \"user_obj\".\n\n",
@@ -1106,7 +564,7 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
     " -If you add a user or group to a DFS ACL, remember that it must be\n",
     "  fully specified as \"user:username\" or \"group:groupname\". In addition, there\n",
     "  may be local requirements on the format of the user or group name.\n",
-    "  Check with your cell administrator.\n\n",                           
+    "  Check with your cell administrator.\n\n",
     " -Or numerous other possibilities. It would be great if we could be more\n",
     "  precise about the actual problem, but for various reasons, this is\n",
     "  impractical via this interface.  If you can't figure it out, you\n",
@@ -1116,7 +574,7 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
                    }
                } else {
                    fprintf(stderr,
-                            "%s: Invalid argument, possible reasons include:\n", 
+                            "%s: Invalid argument, possible reasons include:\n",
                              pn);
                    fprintf(stderr,"\t-File not in AFS\n");
                    fprintf(stderr,
@@ -1125,7 +583,7 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
                             "\t-Tried to add non-existent user to access control list\n");
                }
             } else {
-               Die(errno, ti->data);
+               fs_Die(errno, ti->data);
             }
             error = 1;
         }
@@ -1135,7 +593,7 @@ SetACLCmd(struct cmd_syndesc *as, void *arock)
     return error;
 }
 
-static int 
+static int
 CopyACLCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
@@ -1147,62 +605,78 @@ CopyACLCmd(struct cmd_syndesc *as, void *arock)
     int idf = getidf(as, parm_copyacl_id);
     int error = 0;
     size_t len;
+    char cell[CELL_MAXNAMELEN];
 
-    if (as->parms[2].items) 
+    if (as->parms[2].items)
         clear=1;
-    else 
+    else
         clear=0;
     blob.out_size = AFS_PIOCTL_MAXSIZE;
     blob.in_size = idf;
     blob.in = blob.out = space;
     code = pioctl_utf8(as->parms[0].items->data, VIOCGETAL, &blob, 1);
     if (code) {
-       Die(errno, as->parms[0].items->data);
+       fs_Die(errno, as->parms[0].items->data);
        return 1;
     }
     fa = ParseAcl(space, AFS_PIOCTL_MAXSIZE);
     if (!fa) {
         fprintf(stderr,
-                 "fs: %s: invalid acl data returned from VIOCGETAL\n",
-                 as->parms[0].items->data);
+                 "%s: %s: invalid acl data returned from VIOCGETAL\n",
+                 pn, as->parms[0].items->data);
         return 1;
     }
-    CleanAcl(fa, as->parms[0].items->data);
+    code = GetCell(as->parms[0].items->data, cell, sizeof(cell));
+    if (code) {
+        fprintf(stderr,
+                "%s: %s: unable to obtain cell name\n",
+                pn, as->parms[0].items->data);
+        return 1;
+    }
+    CleanAcl(fa, cell);
     for (ti=as->parms[1].items; ti;ti=ti->next) {
        blob.out_size = AFS_PIOCTL_MAXSIZE;
        blob.in_size = idf;
        blob.in = blob.out = space;
        code = pioctl_utf8(ti->data, VIOCGETAL, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
             continue;
        }
         if (ta)
             ZapAcl(ta);
-       if (clear) 
+       if (clear)
             ta = EmptyAcl(space);
-       else 
+       else
             ta = ParseAcl(space, AFS_PIOCTL_MAXSIZE);
         if (!ta) {
             fprintf(stderr,
-                    "fs: %s: invalid acl data returned from VIOCGETAL\n",
-                     ti->data);
+                    "%s: %s: invalid acl data returned from VIOCGETAL\n",
+                     pn, ti->data);
             error = 1;
             continue;
         }
-       CleanAcl(ta, ti->data);
+        code = GetCell(ti->data, cell, sizeof(cell));
+        if (code) {
+            fprintf(stderr,
+                    "%s: %s: unable to obtain cell name\n",
+                     pn, ti->data);
+            error = 1;
+            continue;
+        }
+       CleanAcl(ta, cell);
        if (ta->dfs != fa->dfs) {
-           fprintf(stderr, 
-                    "%s: incompatible file system types: acl not copied to %s; aborted\n", 
+           fprintf(stderr,
+                    "%s: incompatible file system types: acl not copied to %s; aborted\n",
                     pn, ti->data);
            error = 1;
             continue;
        }
        if (ta->dfs) {
            if (! clear && strcmp(ta->cell, fa->cell) != 0) {
-               fprintf(stderr, 
-                        "%s: default DCE cell differs for file %s: use \"-clear\" switch; acl not merged\n", 
+               fprintf(stderr,
+                        "%s: default DCE cell differs for file %s: use \"-clear\" switch; acl not merged\n",
                         pn, ti->data);
                 error = 1;
                continue;
@@ -1212,10 +686,10 @@ CopyACLCmd(struct cmd_syndesc *as, void *arock)
                 exit(1);
            }
        }
-       for (tp = fa->pluslist;tp;tp=tp->next) 
-           ChangeList(ta, 1, tp->name, tp->rights);
-       for (tp = fa->minuslist;tp;tp=tp->next) 
-           ChangeList(ta, 0, tp->name, tp->rights);
+       for (tp = fa->pluslist;tp;tp=tp->next)
+           ChangeList(ta, 1, tp->name, tp->rights, NULL);
+       for (tp = fa->minuslist;tp;tp=tp->next)
+           ChangeList(ta, 0, tp->name, tp->rights, NULL);
        blob.in = AclToString(ta);
        blob.out_size=0;
        if( FAILED(StringCbLength(blob.in, sizeof(space), &len))) {
@@ -1230,11 +704,11 @@ CopyACLCmd(struct cmd_syndesc *as, void *arock)
                         "%s: Invalid argument, possible reasons include:\n", pn);
                fprintf(stderr,"\t-File not in AFS\n");
            } else {
-               Die(errno, ti->data);
+               fs_Die(errno, ti->data);
            }
             error = 1;
        }
-    } 
+    }
     if (ta)
        ZapAcl(ta);
     ZapAcl(fa);
@@ -1243,13 +717,13 @@ CopyACLCmd(struct cmd_syndesc *as, void *arock)
 
 /* pioctl_utf8() call to get the cellname of a pathname */
 static afs_int32
-GetCell(char *fname, char *cellname)
+GetCell(char *fname, char *cellname, size_t cell_len)
 {
     afs_int32 code;
     struct ViceIoctl blob;
 
     blob.in_size = 0;
-    blob.out_size = CELL_MAXNAMELEN;
+    blob.out_size = cell_len;
     blob.out = cellname;
 
     code = pioctl_utf8(fname, VIOC_FILE_CELL_NAME, &blob, 1);
@@ -1258,86 +732,8 @@ GetCell(char *fname, char *cellname)
     return code;
 }
 
-/* Check if a username is valid: If it contains only digits (or a
- * negative sign), then it might be bad.  We then query the ptserver
- * to see.
- */
-static int
-BadName(char *aname, char *fname)
-{
-    afs_int32 tc, code, id;
-    char *nm;
-    char cell[CELL_MAXNAMELEN];
-    char confDir[257];
-
-    for ( nm = aname; tc = *nm; nm++) {
-       /* all must be '-' or digit to be bad */
-       if (tc != '-' && (tc < '0' || tc > '9'))
-            return 0;
-    }
-
-    /* Go to the PRDB and see if this all number username is valid */
-    code = GetCell(fname, cell);
-    if (code)
-        return 0;
-
-    cm_GetConfigDir(confDir, sizeof(confDir));
-
-    pr_Initialize(1, confDir, cell);
-    code = pr_SNameToId(aname, &id);
-    pr_End();
-
-    /* 1=>Not-valid; 0=>Valid */
-    return ((!code && (id == ANONYMOUSID)) ? 1 : 0);
-}
-
-
-/* clean up an access control list of its bad entries; return 1 if we made
-   any changes to the list, and 0 otherwise */
-static int 
-CleanAcl(struct Acl *aa, char *fname)
-{
-    struct AclEntry *te, **le, *ne;
-    int changes;
-
-    /* Don't correct DFS ACL's for now */
-    if (aa->dfs)
-       return 0;
-
-    /* prune out bad entries */
-    changes = 0;           /* count deleted entries */
-    le = &aa->pluslist;
-    for(te = aa->pluslist; te; te=ne) {
-       ne = te->next;
-       if (BadName(te->name, fname)) {
-           /* zap this dude */
-           *le = te->next;
-           aa->nplus--;
-           free(te);
-           changes++;
-       } else {
-           le = &te->next;
-       }
-    }
-    le = &aa->minuslist;
-    for(te = aa->minuslist; te; te=ne) {
-       ne = te->next;
-       if (BadName(te->name, fname)) {
-           /* zap this dude */
-           *le = te->next;
-           aa->nminus--;
-           free(te);
-           changes++;
-       } else {
-           le = &te->next;
-       }
-    }
-    return changes;
-}
-
-
 /* clean up an acl to not have bogus entries */
-static int 
+static int
 CleanACLCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
@@ -1356,7 +752,7 @@ CleanACLCmd(struct cmd_syndesc *as, void *arock)
        blob.out = space;
        code = pioctl_utf8(ti->data, VIOCGETAL, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
             error = 1;
            continue;
        }
@@ -1393,14 +789,14 @@ CleanACLCmd(struct cmd_syndesc *as, void *arock)
            if (code) {
                if (errno == EINVAL) {
                    fprintf(stderr,
-                            "%s: Invalid argument, possible reasons include\n", 
+                            "%s: Invalid argument, possible reasons include\n",
                              pn);
                    fprintf(stderr,"%s: File not in vice or\n", pn);
                    fprintf(stderr,
-                            "%s: Too many users on access control list or\n", 
+                            "%s: Too many users on access control list or\n",
                             pn);
                } else {
-                   Die(errno, ti->data);
+                   fs_Die(errno, ti->data);
                }
                 error = 1;
                 continue;
@@ -1409,7 +805,7 @@ CleanACLCmd(struct cmd_syndesc *as, void *arock)
            /* now list the updated acl */
            printf("Access list for %s is now\n", ti->data);
            if (ta->nplus > 0) {
-               if (!ta->dfs) 
+               if (!ta->dfs)
                     printf("Normal rights:\n");
                for(te = ta->pluslist;te;te=te->next) {
                    printf("  %s ", te->name);
@@ -1425,7 +821,7 @@ CleanACLCmd(struct cmd_syndesc *as, void *arock)
                    printf("\n");
                }
            }
-           if (ti->next) 
+           if (ti->next)
                 printf("\n");
        } else
            printf("Access list for %s is fine.\n", ti->data);
@@ -1435,8 +831,8 @@ CleanACLCmd(struct cmd_syndesc *as, void *arock)
     return error;
 }
 
-static int 
-ListACLCmd(struct cmd_syndesc *as, void *arock) 
+static int
+ListACLCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
     struct Acl *ta = 0;
@@ -1449,12 +845,19 @@ ListACLCmd(struct cmd_syndesc *as, void *arock)
     SetDotDefault(&as->parms[0].items);
     for(ti=as->parms[0].items; ti; ti=ti->next) {
        char separator;
-       blob.out_size = AFS_PIOCTL_MAXSIZE;
+
+        if ( fs_IsFreelanceRoot(ti->data) ) {
+            fprintf(stderr,"%s: ACLs are not set on the Freelance root.afs volume.\n", pn);
+            error = 1;
+            continue;
+        }
+
+        blob.out_size = AFS_PIOCTL_MAXSIZE;
        blob.in_size = idf;
        blob.in = blob.out = space;
        code = pioctl_utf8(ti->data, VIOCGETAL, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
             error = 1;
            continue;
        }
@@ -1564,8 +967,8 @@ FlushVolumeCmd(struct cmd_syndesc *as, void *arock)
     return error;
 }
 
-static int 
-FlushCmd(struct cmd_syndesc *as, void *arock) 
+static int
+FlushCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
     struct ViceIoctl blob;
@@ -1576,7 +979,7 @@ FlushCmd(struct cmd_syndesc *as, void *arock)
 
     if (as->parms[1].items)
         literal = 1;
-    
+
     for(ti=as->parms[0].items; ti; ti=ti->next) {
         /* once per file */
         memset(&options, 0, sizeof(options));
@@ -1590,7 +993,7 @@ FlushCmd(struct cmd_syndesc *as, void *arock)
        code = pioctl_utf8(ti->data, VIOCFLUSH, &blob, 0);
        if (code) {
            if (errno == EMFILE) {
-               fprintf(stderr, "%s: Can't flush active file %s\n", pn, 
+               fprintf(stderr, "%s: Can't flush active file %s\n", pn,
                         ti->data);
            } else {
                fprintf(stderr, "%s: Error flushing file ", pn);
@@ -1651,9 +1054,9 @@ SetVolCmd(struct cmd_syndesc *as, void *arock) {
                 continue;
            }
        }
-       if (as->parms[2].items) 
+       if (as->parms[2].items)
             motd = as->parms[2].items->data;
-       if (as->parms[3].items) 
+       if (as->parms[3].items)
             offmsg = as->parms[3].items->data;
        input = (char *)status + sizeof(*status);
        *(input++) = '\0';      /* never set name: this call doesn't change vldb */
@@ -1695,40 +1098,14 @@ SetVolCmd(struct cmd_syndesc *as, void *arock) {
         }
        code = pioctl_utf8(ti->data,VIOCSETVOLSTAT, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
        }
     }
-    return error;
-}
-
-/* values match cache manager File Types */
-static char *
-filetypestr(afs_uint32 type)
-{
-    char * s = "Object";
-
-    switch (type) {
-    case 1:     /* file */
-        s = "File";
-        break;
-    case 2:
-        s = "Directory";
-        break;
-    case 3:
-        s = "Symlink";
-        break;
-    case 4:
-        s = "Mountpoint";
-        break;
-    case 5:
-        s = "DfsLink";
-        break;
-    }
-    return s;
+    return error;
 }
 
-static int 
+static int
 ExamineCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
@@ -1749,6 +1126,7 @@ ExamineCmd(struct cmd_syndesc *as, void *arock)
         cm_fid_t fid;
         afs_uint32 filetype;
        afs_int32 owner[2];
+        afs_uint32 unixModeBits;
        char cell[CELL_MAXNAMELEN];
 
         /* once per file */
@@ -1768,7 +1146,7 @@ ExamineCmd(struct cmd_syndesc *as, void *arock)
             options.field_flags |= CM_IOCTL_QOPTS_FIELD_FID;
             options.fid = fid;
         } else {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
@@ -1778,7 +1156,7 @@ ExamineCmd(struct cmd_syndesc *as, void *arock)
 
         code = pioctl_utf8(ti->data, VIOC_GETFILETYPE, &blob, 1);
         if (code || blob.out_size != sizeof(filetype)) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
@@ -1790,7 +1168,7 @@ ExamineCmd(struct cmd_syndesc *as, void *arock)
         if (code == 0)
             cell[blob.out_size-1] = '\0';
         printf("%s %s (%u.%u.%u) contained in cell %s\n",
-                filetypestr(filetype),
+                fs_filetypestr(filetype),
                 ti->data, fid.volume, fid.vnode, fid.unique,
                 code ? "unknown-cell" : cell);
 
@@ -1811,6 +1189,13 @@ ExamineCmd(struct cmd_syndesc *as, void *arock)
            printf("Owner %s (%d) Group %s (%d)\n", oname, owner[0], gname, owner[1]);
         }
 
+       blob.out_size = sizeof(afs_uint32);
+        blob.out = (char *) &unixModeBits;
+       if (0 == pioctl_utf8(ti->data, VIOC_GETUNIXMODE, &blob, 1) &&
+            blob.out_size == sizeof(afs_uint32)) {
+           printf("UNIX mode 0%o\n", unixModeBits);
+        }
+
        blob.out = space;
        blob.out_size = AFS_PIOCTL_MAXSIZE;
        code = pioctl_utf8(ti->data, VIOCGETVOLSTAT, &blob, 1);
@@ -1830,7 +1215,7 @@ ExamineCmd(struct cmd_syndesc *as, void *arock)
             motd = offmsg + len + 1;
             PrintStatus(status, name, motd, offmsg);
         } else {
-            Die(errno, ti->data);
+            fs_Die(errno, ti->data);
         }
 
         errno = 0;
@@ -1850,7 +1235,7 @@ ExamineCmd(struct cmd_syndesc *as, void *arock)
             break;
         default:
             printf("Unknown volume state\n");
-            Die(errno, ti->data);
+            fs_Die(errno, ti->data);
         }
         printf("\n");
     }
@@ -1858,7 +1243,7 @@ ExamineCmd(struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ListQuotaCmd(struct cmd_syndesc *as, void *arock) 
+ListQuotaCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
     struct ViceIoctl blob;
@@ -1867,7 +1252,7 @@ ListQuotaCmd(struct cmd_syndesc *as, void *arock)
     char *name;
 
     int error = 0;
-    
+
     printf("%-25s%-11s%-11s%-7s%-13s\n", "Volume Name", "      Quota",
           "       Used", "  %Used", "    Partition");
     SetDotDefault(&as->parms[0].items);
@@ -1878,7 +1263,7 @@ ListQuotaCmd(struct cmd_syndesc *as, void *arock)
        blob.out = space;
        code = pioctl_utf8(ti->data, VIOCGETVOLSTAT, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
             error = 1;
            continue;
        }
@@ -1905,7 +1290,7 @@ WhereIsCmd(struct cmd_syndesc *as, void *arock)
 
     if (as->parms[1].items)
         literal = 1;
-    
+
     SetDotDefault(&as->parms[0].items);
     for(ti=as->parms[0].items; ti; ti=ti->next) {
         cm_fid_t fid;
@@ -1920,7 +1305,7 @@ WhereIsCmd(struct cmd_syndesc *as, void *arock)
         options.literal = literal;
        blob.in_size = options.size;    /* no variable length data */
         blob.in = &options;
-        
+
         blob.out_size = sizeof(cm_fid_t);
         blob.out = (char *) &fid;
         if (0 == pioctl_utf8(ti->data, VIOCGETFID, &blob, 1) &&
@@ -1928,7 +1313,7 @@ WhereIsCmd(struct cmd_syndesc *as, void *arock)
             options.field_flags |= CM_IOCTL_QOPTS_FIELD_FID;
             options.fid = fid;
         } else {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
@@ -1938,7 +1323,7 @@ WhereIsCmd(struct cmd_syndesc *as, void *arock)
 
         code = pioctl_utf8(ti->data, VIOC_GETFILETYPE, &blob, 1);
         if (code || blob.out_size != sizeof(filetype)) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
             error = 1;
            continue;
         }
@@ -1947,17 +1332,17 @@ WhereIsCmd(struct cmd_syndesc *as, void *arock)
        memset(space, 0, sizeof(space));
        code = pioctl_utf8(ti->data, VIOCWHEREIS, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
             error = 1;
            continue;
        }
         hosts = (afs_int32 *) space;
-       printf("%s %s is on host%s ", 
-                filetypestr(filetype),
+       printf("%s %s is on host%s ",
+                fs_filetypestr(filetype),
                 ti->data,
                 (hosts[0] && !hosts[1]) ? "": "s");
        for(j=0; j<AFS_MAXHOSTS; j++) {
-           if (hosts[j] == 0) 
+           if (hosts[j] == 0)
                 break;
            tp = hostutil_GetNameByINet(hosts[j]);
            printf("%s ", tp);
@@ -1977,7 +1362,7 @@ DiskFreeCmd(struct cmd_syndesc *as, void *arock)
     char *name;
     struct VolumeStatus *status;
     int error = 0;
-    
+
     printf("%-25s%-10s%-10s%-10s%-6s\n", "Volume Name", "    kbytes",
           "      used", "     avail", " %used");
     SetDotDefault(&as->parms[0].items);
@@ -1988,7 +1373,7 @@ DiskFreeCmd(struct cmd_syndesc *as, void *arock)
        blob.out = space;
        code = pioctl_utf8(ti->data, VIOCGETVOLSTAT, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
             error = 1;
            continue;
        }
@@ -2009,7 +1394,7 @@ QuotaCmd(struct cmd_syndesc *as, void *arock)
     double quotaPct;
     struct VolumeStatus *status;
     int error = 0;
-    
+
     SetDotDefault(&as->parms[0].items);
     for(ti=as->parms[0].items; ti; ti=ti->next) {
        /* once per file */
@@ -2021,15 +1406,15 @@ QuotaCmd(struct cmd_syndesc *as, void *arock)
          * The response is VolumeStatus, volume name, offline message, and motd
          */
        if (code || blob.out_size < sizeof(*status)) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
             error = 1;
            continue;
        }
 
         status = (VolumeStatus *)space;
-       if (status->MaxQuota) 
+       if (status->MaxQuota)
             quotaPct = ((((double)status->BlocksInUse)/status->MaxQuota) * 100.0);
-       else 
+       else
             quotaPct = 0.0;
        printf("%2.0f%% of quota used.\n", quotaPct);
     }
@@ -2055,7 +1440,7 @@ ListMountCmd(struct cmd_syndesc *as, void *arock)
     int link_chars_read;               /*Num chars read in readlink()*/
 #endif /* not WIN32 */
     int        thru_symlink;                   /*Did we get to a mount point via a symlink?*/
-    
+
     int error = 0;
     for(ti=as->parms[0].items; ti; ti=ti->next) {
        /* once per file */
@@ -2156,8 +1541,8 @@ ListMountCmd(struct cmd_syndesc *as, void *arock)
            parent_dir[last_component - true_name + 1] = 0;
            last_component++;   /*Skip the slash*/
 #ifdef WIN32
-           if (!InAFS(parent_dir)) {
-               const char * nbname = NetbiosName();
+           if (!fs_InAFS(parent_dir)) {
+               const char * nbname = fs_NetbiosName();
                if( FAILED(StringCbLength(nbname, NETBIOSNAMESZ, &len))) {
                    fprintf (stderr, "StringCbLength failure on nbname");
                    exit(1);
@@ -2218,7 +1603,7 @@ ListMountCmd(struct cmd_syndesc *as, void *arock)
            if (errno == EINVAL) {
                fprintf(stderr,"'%s' is not a mount point.\n", ti->data);
            } else {
-               Die(errno, (ti->data ? ti->data : parent_dir));
+               fs_Die(errno, (ti->data ? ti->data : parent_dir));
            }
            error = 1;
        }
@@ -2267,29 +1652,30 @@ MakeMountCmd(struct cmd_syndesc *as, void *arock)
         volName = ++tmpName;
     }
 
-    parent = Parent(as->parms[0].items->data);
-    if (!InAFS(parent)) {
+    parent = fs_GetParent(as->parms[0].items->data);
+    if (!fs_InAFS(parent)) {
 #ifdef WIN32
-       const char * nbname = NetbiosName();
+       const char * nbname = fs_NetbiosName();
        if ( FAILED(StringCbLength(nbname, NETBIOSNAMESZ, &len))) {
            fprintf (stderr, "StringCbLength failure on nbname");
            exit(1);
        }
        if (parent[0] == '\\' && parent[1] == '\\' &&
-           parent[len+2] == '\\' &&
-           parent[len+3] == '\0' &&
+           (parent[len+2] == '\\' && parent[len+3] == '\0' || parent[len+2] == '\0') &&
            !strnicmp(nbname,&parent[2],len))
        {
-           if( FAILED(StringCbPrintf(path, sizeof(path),"%sall\\%s", parent, &as->parms[0].items->data[len+2]))) {
+           if( FAILED(StringCbPrintf(path, sizeof(path),"%s%sall%s", parent,
+                                      parent[len+2]?"":"\\",
+                                      &as->parms[0].items->data[len+2]))) {
                fprintf (stderr, "path - cannot be populated");
                 exit(1);
             }
-           parent = Parent(path);
-           if (!InAFS(parent)) {
+           parent = fs_GetParent(path);
+           if (!fs_InAFS(parent)) {
                fprintf(stderr,"%s: mount points must be created within the AFS file system\n", pn);
                return 1;
            }
-       } else 
+       } else
 #endif
        {
            fprintf(stderr,"%s: mount points must be created within the AFS file system\n", pn);
@@ -2307,9 +1693,9 @@ MakeMountCmd(struct cmd_syndesc *as, void *arock)
             exit(1);
        }
     }
-    if ( IsFreelanceRoot(parent) ) {
-       if ( !IsAdmin() ) {
-           fprintf(stderr,"%s: Only AFS Client Administrators may alter the root.afs volume\n", pn);
+    if ( fs_IsFreelanceRoot(parent) ) {
+       if ( !fs_IsAdmin() ) {
+           fprintf(stderr,"%s: Only AFS Client Administrators may alter the Freelance root.afs volume\n", pn);
            return 1;
        }
 
@@ -2325,7 +1711,7 @@ MakeMountCmd(struct cmd_syndesc *as, void *arock)
        }
     } else {
        if (!cellName) {
-           code = GetCell(parent,space);
+           code = GetCell(parent, space, sizeof(space));
             if (code)
                 return 1;
         }
@@ -2402,7 +1788,7 @@ MakeMountCmd(struct cmd_syndesc *as, void *arock)
         free(info.linkedCell);
 
     if (code) {
-       Die(errno, path);
+       fs_Die(errno, path);
        return 1;
     }
     return 0;
@@ -2438,8 +1824,8 @@ RemoveMountCmd(struct cmd_syndesc *as, void *arock) {
            tp++;   /* skip the slash */
 
 #ifdef WIN32
-           if (!InAFS(tbuffer)) {
-               const char * nbname = NetbiosName();
+           if (!fs_InAFS(tbuffer)) {
+               const char * nbname = fs_NetbiosName();
                if( FAILED(StringCbLength(nbname, NETBIOSNAMESZ, &len))) {
                    fprintf (stderr, "StringCbLength failure on nbname");
                    exit(1);
@@ -2464,7 +1850,7 @@ RemoveMountCmd(struct cmd_syndesc *as, void *arock) {
                 exit(1);
            }
            tp = ti->data;
-            fs_StripDriveLetter(tp, tp, 0);
+            fs_StripDriveLetter(tp, tp, strlen(tp) + 1);
        }
        blob.in = tp;
        if( FAILED(StringCbLength(tp, AFS_PIOCTL_MAXSIZE, &len))) {
@@ -2479,14 +1865,14 @@ RemoveMountCmd(struct cmd_syndesc *as, void *arock) {
            if (errno == EINVAL) {
                fprintf(stderr,"%s: '%s' is not a mount point.\n", pn, ti->data);
             } else {
-               Die(errno, ti->data);
+               fs_Die(errno, ti->data);
            }
             error = 1;
            continue;   /* don't bother trying */
        }
 
-        if ( IsFreelanceRoot(tbuffer) && !IsAdmin() ) {
-            fprintf(stderr,"%s: Only AFS Client Administrators may alter the root.afs volume\n", pn);
+        if ( fs_IsFreelanceRoot(tbuffer) && !fs_IsAdmin() ) {
+            fprintf(stderr,"%s: Only AFS Client Administrators may alter the Freelance root.afs volume\n", pn);
             error = 1;
             continue;   /* skip */
         }
@@ -2500,7 +1886,7 @@ RemoveMountCmd(struct cmd_syndesc *as, void *arock) {
        blob.in_size = (long)len+1;
        code = pioctl_utf8(tbuffer, VIOC_AFS_DELETE_MT_PT, &blob, 0);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
             error = 1;
        }
     }
@@ -2534,11 +1920,11 @@ CheckServersCmd(struct cmd_syndesc *as, void *arock)
 
     /* prepare flags for checkservers command */
     temp = 2;  /* default to checking local cell only */
-    if (as->parms[2].items) 
+    if (as->parms[2].items)
         temp |= 1;     /* set fast flag */
-    if (as->parms[1].items) 
+    if (as->parms[1].items)
         temp &= ~2;    /* turn off local cell check */
-    
+
     checkserv.magic = 0x12345678;      /* XXX */
     checkserv.tflags=temp;
 
@@ -2577,14 +1963,14 @@ CheckServersCmd(struct cmd_syndesc *as, void *arock)
         } else if(checkserv.tinterval> 600) {
             printf("Warning: The maximum -interval value is 10 mins (600 secs)\n");
             checkserv.tinterval=600;   /* 10 min max interval */
-        }       
+        }
     } else {
         checkserv.tinterval = -1;      /* don't change current interval */
     }
 
     if ( checkserv.tinterval >= 0 ) {
 #ifdef WIN32
-        if ( !IsAdmin() ) {
+        if ( !fs_IsAdmin() ) {
             fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
             return EACCES;
         }
@@ -2602,7 +1988,7 @@ CheckServersCmd(struct cmd_syndesc *as, void *arock)
            printf("Must be root to change -interval\n");
            return code;
        }
-       Die(errno, 0);
+       fs_Die(errno, 0);
         return 1;
     }
 #if _MSC_VER < 1400
@@ -2616,10 +2002,10 @@ CheckServersCmd(struct cmd_syndesc *as, void *arock)
 #endif
 
     if (checkserv.tinterval >= 0) {
-       if (checkserv.tinterval > 0) 
-           printf("The new down server probe interval (%d secs) is now in effect (old interval was %d secs)\n", 
+       if (checkserv.tinterval > 0)
+           printf("The new down server probe interval (%d secs) is now in effect (old interval was %d secs)\n",
                   checkserv.tinterval, temp);
-       else 
+       else
            printf("The current down server probe interval is %d secs\n", temp);
        return 0;
     }
@@ -2638,7 +2024,7 @@ CheckServersCmd(struct cmd_syndesc *as, void *arock)
             }
 #endif
 
-           if (temp == 0) 
+           if (temp == 0)
                 break;
            tp = hostutil_GetNameByINet(temp);
            printf(" %s", tp);
@@ -2656,7 +2042,7 @@ MessagesCmd(struct cmd_syndesc *as, void *arock)
     struct ViceIoctl blob;
     struct gaginfo gagflags;
     struct cmd_item *show;
-    
+
     memset(&gagflags, 0, sizeof(struct gaginfo));
     blob.in_size = sizeof(struct gaginfo);
     blob.in = (caddr_t ) &gagflags;
@@ -2674,19 +2060,19 @@ MessagesCmd(struct cmd_syndesc *as, void *arock)
         else if (!strcasecmp (show->data, "none"))
             /* do nothing */ ;
         else {
-            fprintf(stderr, 
-                     "unrecognized flag %s: must be in {user,console,all,none}\n", 
+            fprintf(stderr,
+                     "unrecognized flag %s: must be in {user,console,all,none}\n",
                      show->data);
             code = EINVAL;
         }
     }
+
     if (code)
         return 1;
 
     code = pioctl_utf8(0, VIOC_GAG, &blob, 1);
     if (code) {
-       Die(errno, 0);
+       fs_Die(errno, 0);
         return 1;
     }
     return 0;
@@ -2697,16 +2083,16 @@ CheckVolumesCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
     struct ViceIoctl blob;
-    
+
     blob.in_size = 0;
     blob.out_size = 0;
     code = pioctl_utf8(0, VIOCCKBACK, &blob, 1);
     if (code) {
-       Die(errno, 0);
+       fs_Die(errno, 0);
        return 1;
     }
     printf("All volumeID/name mappings checked.\n");
-    
+
     return 0;
 }
 
@@ -2716,9 +2102,9 @@ SetCacheSizeCmd(struct cmd_syndesc *as, void *arock)
     afs_int32 code;
     struct ViceIoctl blob;
     afs_int32 temp;
-    
+
 #ifdef WIN32
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -2746,10 +2132,10 @@ SetCacheSizeCmd(struct cmd_syndesc *as, void *arock)
     blob.out_size = 0;
     code = pioctl_utf8(0, VIOCSETCACHESIZE, &blob, 1);
     if (code) {
-       Die(errno, (char *) 0);
+       fs_Die(errno, (char *) 0);
         return 1;
-    } 
-      
+    }
+
     printf("New cache size set.\n");
     return 0;
 }
@@ -2768,10 +2154,10 @@ GetCacheParmsCmd(struct cmd_syndesc *as, void *arock)
     blob.out = (char *) &parms;
     code = pioctl_utf8(0, VIOCGETCACHEPARMS, &blob, 1);
     if (code || blob.out_size != sizeof(parms)) {
-       Die(errno, NULL);
+       fs_Die(errno, NULL);
         return 1;
     }
-     
+
     printf("AFS using %I64u of the cache's available %I64u 1K byte blocks.\n",
            parms.parms[1], parms.parms[0]);
     if (parms.parms[1] > parms.parms[0])
@@ -2791,7 +2177,7 @@ ListCellsCmd(struct cmd_syndesc *as, void *arock)
     errno_t err;
 
     resolve = !(as->parms[0].items);    /* -numeric */
-    
+
     for(i=0;i<1000;i++) {
        tp = space;
 #if _MSC_VER < 1400
@@ -2813,11 +2199,11 @@ ListCellsCmd(struct cmd_syndesc *as, void *arock)
        blob.out = space;
        code = pioctl_utf8(0, VIOCGETCELL, &blob, 1);
        if (code < 0) {
-           if (errno == EDOM) 
+           if (errno == EDOM)
                 break; /* done with the list */
-            Die(errno, 0);
+            fs_Die(errno, 0);
             return 1;
-       }       
+       }
        tp = space;
 #if _MSC_VER < 1400
         memcpy(&magic, tp, sizeof(afs_int32));
@@ -2844,7 +2230,7 @@ ListCellsCmd(struct cmd_syndesc *as, void *arock)
                 exit(1);
            }
 #endif
-           if (addr == 0) 
+           if (addr == 0)
                 break;
 
             if (resolve) {
@@ -2894,7 +2280,7 @@ ListAliasesCmd(struct cmd_syndesc *as, void *arock)
        if (code < 0) {
            if (errno == EDOM)
                break;          /* done with the list */
-           Die(errno, 0);
+           fs_Die(errno, 0);
            return 1;
        }
         space[blob.out_size - 1] = '\0';
@@ -2921,7 +2307,7 @@ CallBackRxConnCmd(struct cmd_syndesc *as, void *arock)
     struct hostent *thp;
     char *tp;
     int setp;
-    
+
     ti = as->parms[0].items;
     setp = 1;
     if (ti) {
@@ -2943,16 +2329,16 @@ CallBackRxConnCmd(struct cmd_syndesc *as, void *arock)
         hostAddr = 0;   /* means don't set host */
        setp = 0;       /* aren't setting host */
     }
-    
+
     /* now do operation */
     blob.in_size = sizeof(afs_int32);
     blob.out_size = sizeof(afs_int32);
     blob.in = (char *) &hostAddr;
     blob.out = (char *) &hostAddr;
-    
+
     code = pioctl_utf8(0, VIOC_CBADDR, &blob, 1);
     if (code < 0) {
-       Die(errno, 0);
+       fs_Die(errno, 0);
        return 1;
     }
     return 0;
@@ -2962,114 +2348,182 @@ CallBackRxConnCmd(struct cmd_syndesc *as, void *arock)
 static int
 NewCellCmd(struct cmd_syndesc *as, void *arock)
 {
-#ifndef WIN32
-    afs_int32 code, linkedstate=0, size=0, *lp;
+    afs_uint32 code, linkedstate=0, size=0, count=0, *lp;
+    afs_uint32 usedns=0, useregistry=0;
     struct ViceIoctl blob;
     struct cmd_item *ti;
-    char *destEnd, *tp, *cellname=0;
-    struct hostent *thp;
-    afs_int32 fsport = 0, vlport = 0;
-    errno_t err;
+    char *tp, *cellname=0, *linked_cellname=0;
+    afs_uint32 fsport = 0, vlport = 0;
     size_t destRemaining;
 
-    memset(space, 0, AFS_MAXHOSTS * sizeof(afs_int32));
-    tp = space;
-    lp = (afs_int32 *)tp;
-    *lp++ = 0x12345678;
-    tp += sizeof(afs_int32);
-    for(ti=as->parms[1].items; ti; ti=ti->next) {
-       thp = hostutil_GetHostByName(ti->data);
-       if (!thp) {
-           fprintf(stderr,"%s: Host %s not found in host table, skipping it.\n",
-                  pn, ti->data);
-       }
-       else {
-#if _MSC_VER < 1400
-            memcpy(tp, thp->h_addr, sizeof(afs_int32));
-#else
-            err = memcpy_s(tp, sizeof(space) - (tp - space) * sizeof(char), thp->h_addr, sizeof(afs_int32));
-           if ( err ) {
-                fprintf (stderr, "memcpy_s failure on tp");
-                exit(1);
-            }
-#endif
-           tp += sizeof(afs_int32);
-       }
+    if ( !fs_IsAdmin() ) {
+        fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
+        return EACCES;
+    }
+
+    /* if there is no cell specified, use old Windows behavior */
+    if (as->parms[0].items == NULL) {
+        blob.in_size = 0;
+        blob.in = (char *) 0;
+        blob.out_size = AFS_PIOCTL_MAXSIZE;
+        blob.out = space;
+
+        code = pioctl_utf8((char *) 0, VIOCNEWCELL, &blob, 1);
+
+        if (code) {
+            fs_Die(errno, (char *) 0);
+            return 1;
+        }
+
+        printf("Cell servers information refreshed\n");
+        return 0;
+    } else {
+        cellname = as->parms[0].items->data;
     }
+
     if (as->parms[2].items) {
        /*
         * Link the cell, for the purposes of volume location, to the specified
         * cell.
         */
-       cellname = as->parms[2].items->data;
+       linked_cellname = as->parms[2].items->data;
        linkedstate = 1;
     }
-#ifdef FS_ENABLE_SERVER_DEBUG_PORTS
+
     if (as->parms[3].items) {
-       code = util_GetInt32(as->parms[3].items->data, &vlport);
+       code = util_GetInt32(as->parms[2].items->data, &vlport);
        if (code) {
            fprintf(stderr,"fs: bad integer specified for the fileserver port.\n");
            return code;
        }
     }
     if (as->parms[4].items) {
-       code = util_GetInt32(as->parms[4].items->data, &fsport);
+       code = util_GetInt32(as->parms[3].items->data, &fsport);
        if (code) {
            fprintf(stderr,"fs: bad integer specified for the vldb server port.\n");
            return code;
        }
     }
-#endif
-    tp = (char *)(space + (AFS_MAXHOSTS+1) *sizeof(afs_int32));
-    lp = (afs_int32 *)tp;    
+
+    if (as->parms[5].items) {
+        useregistry = 1;
+    }
+
+    if (as->parms[6].items) {
+        usedns = 1;
+    }
+
+    /* Count the number of hostnames */
+    for (ti=as->parms[1].items; ti && count < AFS_MAXHOSTS; ti=ti->next, count++);
+
+    if (!usedns && count == 0) {
+        fprintf( stderr, "fs: at least one vldb server must be specified.");
+        exit(1);
+    }
+
+    if (count > AFS_MAXHOSTS) {
+        fprintf( stderr, "fs: at most %u servers may be specified.", AFS_MAXHOSTS);
+        exit(1);
+    }
+
+    /*
+     * The pioctl data buffer consists of the following structure:
+     *
+     *  afs_uint32 flags
+     *  afs_uint32 alternative fs port
+     *  afs_uint32 alternative vl port
+     *  afs_uint32 count of vldb servers
+     *  char[]     cellname
+     *  char[]     linkedcell
+     *  n * char[] hostnames
+     */
+
+    memset(space, 0, sizeof(space));
+    tp = space;
+    lp = (afs_uint32 *)tp;
+
+    /* flags */
+    if (usedns)
+        *lp |= VIOC_NEWCELL2_FLAG_USEDNS;
+
+    if (useregistry)
+        *lp |= VIOC_NEWCELL2_FLAG_USEREG;
+
+    if (linkedstate)
+        *lp |= VIOC_NEWCELL2_FLAG_LINKED;
+    lp++;
+
+    /* alt fs port */
     *lp++ = fsport;
+
+    /* alt vl port */
     *lp++ = vlport;
-    *lp = linkedstate;
-    if( FAILED(StringCbCopyEx(space +  ((AFS_MAXHOSTS+4) * sizeof(afs_int32)), sizeof(space) - (AFS_MAXHOSTS+4) * sizeof(afs_int32)
-        , as->parms[0].items->data, &tp, &destRemaining, STRSAFE_FILL_ON_FAILURE))) {
-        fprintf (stderr, "var - not enough space");
+
+    /* count of server names */
+    *lp++ = count;
+
+    /* Switch back to char pointer */
+    tp = (char *)lp;
+
+    /* Add nul-terminated cellname */
+    destRemaining = sizeof(space) - (tp - space);
+    if( FAILED(StringCbCopyEx( tp,
+                               destRemaining,
+                               as->parms[0].items->data,
+                               &tp,
+                               &destRemaining,
+                               STRSAFE_FILL_ON_FAILURE))) {
+        fprintf (stderr, " not enough space for cellname");
+        exit(1);
+    }
+    /* Move beyond the terminating nul */
+    tp++;
+    destRemaining -= sizeof(char);
+
+    /* Add nul-terminated linkname */
+    if( FAILED(StringCbCopyEx( tp,
+                               destRemaining,
+                               linkedstate ? linked_cellname : "",
+                               &tp,
+                               &destRemaining,
+                               STRSAFE_FILL_ON_FAILURE))) {
+        fprintf (stderr, " not enough space for linked cellname");
         exit(1);
     }
-    tp++;   /* for null */
+    /* Move beyond the terminating nul */
+    tp++;
     destRemaining -= sizeof(char);
-    if (linkedstate) {
-        if( FAILED(StringCbCopyEx(tp, sizeof(space) - size, cellname, &destEnd, &destRemaining, STRSAFE_FILL_ON_FAILURE))) {
-            fprintf (tp, "space arr - not enough space");
+
+    /* Add the servers */
+    for (ti=as->parms[1].items; ti; ti=ti->next) {
+        if( FAILED(StringCbCopyEx( tp,
+                                   destRemaining,
+                                   ti->data,
+                                   &tp,
+                                   &destRemaining,
+                                   STRSAFE_FILL_ON_FAILURE))) {
+            fprintf (stderr, " not enough space for server %s", ti->data);
             exit(1);
-       }
-        size += destEnd - tp + 1;
+        }
+        /* Move beyond the terminating nul */
+        tp++;
+        destRemaining -= sizeof(char);
     }
-    blob.in_size = size;
+
+    blob.in_size = (tp - space);
     blob.in = space;
     blob.out_size = 0;
-    code = pioctl_utf8(0, VIOCNEWCELL, &blob, 1);
-    if (code < 0)
-       Die(errno, 0);
-    return 0;
-#else /* WIN32 */
-    afs_int32 code;
-    struct ViceIoctl blob;
-    
-    if ( !IsAdmin() ) {
-        fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
-        return EACCES;
-    }
-
-    blob.in_size = 0;
-    blob.in = (char *) 0;
-    blob.out_size = AFS_PIOCTL_MAXSIZE;
     blob.out = space;
-
-    code = pioctl_utf8((char *) 0, VIOCNEWCELL, &blob, 1);
+    code = pioctl_utf8(NULL, VIOCNEWCELL2, &blob, 1);
 
     if (code) {
-        Die(errno, (char *) 0);
+        fs_Die(errno, as->parms[0].items->data);
         return 1;
     }
-    
-    printf("Cell servers information refreshed\n");
+
+    printf("Cell servers information for %s added or updated.\n",
+           as->parms[0].items->data);
     return 0;
-#endif /* WIN32 */
 }
 
 #ifndef WIN32
@@ -3110,7 +2564,7 @@ NewAliasCmd(struct cmd_syndesc *as, void *arock)
                    "%s: cell name `%s' in use by an existing cell.\n", pn,
                    aliasName);
        } else {
-           Die(errno, 0);
+           fs_Die(errno, 0);
        }
        return 1;
     }
@@ -3130,7 +2584,7 @@ WhichCellCmd(struct cmd_syndesc *as, void *arock)
 
     if (as->parms[1].items)
         literal = 1;
-    
+
     SetDotDefault(&as->parms[0].items);
     for(ti=as->parms[0].items; ti; ti=ti->next) {
         cm_fid_t fid;
@@ -3154,7 +2608,7 @@ WhichCellCmd(struct cmd_syndesc *as, void *arock)
             options.field_flags |= CM_IOCTL_QOPTS_FIELD_FID;
             options.fid = fid;
         } else {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
@@ -3164,7 +2618,7 @@ WhichCellCmd(struct cmd_syndesc *as, void *arock)
 
         code = pioctl_utf8(ti->data, VIOC_GETFILETYPE, &blob, 1);
         if (code || blob.out_size != sizeof(filetype)) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
@@ -3176,13 +2630,13 @@ WhichCellCmd(struct cmd_syndesc *as, void *arock)
            if (errno == ENOENT)
                fprintf(stderr,"%s: no such cell as '%s'\n", pn, ti->data);
            else
-               Die(errno, ti->data);
+               fs_Die(errno, ti->data);
            error = 1;
            continue;
        }
         cell[CELL_MAXNAMELEN - 1] = '\0';
         printf("%s %s lives in cell '%s'\n",
-                filetypestr(filetype),
+                fs_filetypestr(filetype),
                 ti->data, cell);
     }
     return error;
@@ -3193,7 +2647,7 @@ WSCellCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
     struct ViceIoctl blob;
-    
+
     blob.in_size = 0;
     blob.in = NULL;
     blob.out_size = AFS_PIOCTL_MAXSIZE;
@@ -3202,7 +2656,7 @@ WSCellCmd(struct cmd_syndesc *as, void *arock)
     code = pioctl_utf8(NULL, VIOC_GET_WS_CELL, &blob, 1);
 
     if (code) {
-       Die(errno, NULL);
+       fs_Die(errno, NULL);
         return 1;
     }
     space[AFS_PIOCTL_MAXSIZE - 1] = '\0';
@@ -3231,7 +2685,7 @@ MonitorCmd(struct cmd_syndesc *as, void *arock)
     char *tp;
     int setp;
     errno_t err;
-    
+
     ti = as->parms[0].items;
     setp = 1;
     if (ti) {
@@ -3272,7 +2726,7 @@ MonitorCmd(struct cmd_syndesc *as, void *arock)
     blob.out = (char *) &hostAddr;
     code = pioctl_utf8(0, VIOC_AFS_MARINER_HOST, &blob, 1);
     if (code || blob.out_size != sizeof(afs_int32)) {
-       Die(errno, 0);
+       fs_Die(errno, 0);
        return 1;
     }
     if (setp) {
@@ -3301,11 +2755,11 @@ SysNameCmd(struct cmd_syndesc *as, void *arock)
     errno_t err;
     size_t destRemaining = sizeof(space);
     size_t len;
-    
+
     ti = as->parms[0].items;
     if (ti) {
 #ifdef WIN32
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -3355,9 +2809,9 @@ SysNameCmd(struct cmd_syndesc *as, void *arock)
 
     code = pioctl_utf8(0, VIOC_AFS_SYSNAME, &blob, 1);
     if (code) {
-        Die(errno, 0);
+        fs_Die(errno, 0);
         return 1;
-    }    
+    }
     if (setp) {
         printf("%s: new sysname%s set.\n", pn, setp > 1 ? " list" : "");
         return 0;
@@ -3377,7 +2831,7 @@ SysNameCmd(struct cmd_syndesc *as, void *arock)
     if (!setp) {
         fprintf(stderr,"No sysname name value was found\n");
         return 1;
-    } 
+    }
     space[blob.out_size - 1] = '\0';
     printf("Current sysname%s is", setp > 1 ? " list" : "");
     for (; setp > 0; --setp ) {
@@ -3401,9 +2855,9 @@ static int ExportAfsCmd(struct cmd_syndesc *as, void *arock)
     struct cmd_item *ti;
     int export = 0, type = 0, mode = 0, exp = 0, gstat = 0;
     int exportcall, pwsync = 0, smounts = 0;
-    
+
 #ifdef WIN32
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -3415,7 +2869,7 @@ static int ExportAfsCmd(struct cmd_syndesc *as, void *arock)
 #endif /* WIN32 */
 
     ti = as->parms[0].items;
-    if (strcmp(ti->data, "nfs")        == 0) 
+    if (strcmp(ti->data, "nfs")        == 0)
         type = 0x71; /* NFS */
     else {
        fprintf(stderr,
@@ -3424,9 +2878,9 @@ static int ExportAfsCmd(struct cmd_syndesc *as, void *arock)
     }
     ti = as->parms[1].items;
     if (ti) {
-       if (strcmp(ti->data, "on") == 0) 
+       if (strcmp(ti->data, "on") == 0)
             export = 3;
-       else if (strcmp(ti->data, "off") == 0) 
+       else if (strcmp(ti->data, "off") == 0)
             export = 2;
        else {
            fprintf(stderr, "Illegal argument %s\n", ti->data);
@@ -3435,9 +2889,9 @@ static int ExportAfsCmd(struct cmd_syndesc *as, void *arock)
        exp = 1;
     }
     if (ti = as->parms[2].items) {     /* -noconvert */
-       if (strcmp(ti->data, "on") == 0) 
+       if (strcmp(ti->data, "on") == 0)
             mode = 2;
-       else if (strcmp(ti->data, "off") == 0) 
+       else if (strcmp(ti->data, "off") == 0)
             mode = 3;
        else {
            fprintf(stderr, "Illegal argument %s\n", ti->data);
@@ -3445,9 +2899,9 @@ static int ExportAfsCmd(struct cmd_syndesc *as, void *arock)
        }
     }
     if (ti = as->parms[3].items) {     /* -uidcheck */
-       if (strcmp(ti->data, "on") == 0) 
+       if (strcmp(ti->data, "on") == 0)
             pwsync = 3;
-       else if (strcmp(ti->data, "off") == 0) 
+       else if (strcmp(ti->data, "off") == 0)
             pwsync = 2;
        else {
            fprintf(stderr, "Illegal argument %s\n", ti->data);
@@ -3455,9 +2909,9 @@ static int ExportAfsCmd(struct cmd_syndesc *as, void *arock)
        }
     }
     if (ti = as->parms[4].items) {     /* -submounts */
-       if (strcmp(ti->data, "on") == 0) 
+       if (strcmp(ti->data, "on") == 0)
             smounts = 3;
-       else if (strcmp(ti->data, "off") == 0) 
+       else if (strcmp(ti->data, "off") == 0)
             smounts = 2;
        else {
            fprintf(stderr, "Illegal argument %s\n", ti->data);
@@ -3477,13 +2931,13 @@ static int ExportAfsCmd(struct cmd_syndesc *as, void *arock)
            fprintf(stderr,
                     "Sorry, the %s-exporter type is currently not supported on this AFS client\n", exported_types[type]);
        } else {
-           Die(errno, 0);
+           fs_Die(errno, 0);
        }
         return 1;
     } else {
        if (!gstat) {
            if (exportcall & 1) {
-               printf("'%s' translator is enabled with the following options:\n\tRunning in %s mode\n\tRunning in %s mode\n\t%s\n", 
+               printf("'%s' translator is enabled with the following options:\n\tRunning in %s mode\n\tRunning in %s mode\n\t%s\n",
                       exported_types[type], (exportcall & 2 ? "strict unix" : "convert owner mode bits to world/other"),
                       (exportcall & 4 ? "strict 'passwd sync'" : "no 'passwd sync'"),
                       (exportcall & 8 ? "Allow mounts of /afs/.. subdirs" : "Only mounts to /afs allowed"));
@@ -3534,20 +2988,20 @@ GetCellCmd(struct cmd_syndesc *as, void *arock)
            if (errno == ENOENT)
                fprintf(stderr,"%s: the cell named '%s' does not exist\n", pn, info.name);
            else
-               Die(errno, info.name);
+               fs_Die(errno, info.name);
            error = 1;
             continue;
        }
        printf("Cell %s status: ", info.name);
 #ifdef notdef
-       if (args.stat & 1) 
+       if (args.stat & 1)
             printf("primary ");
 #endif
-       if (args.stat & 2) 
+       if (args.stat & 2)
             printf("no setuid allowed");
-       else 
+       else
             printf("setuid allowed");
-       if (args.stat & 4) 
+       if (args.stat & 4)
             printf(", using old VLDB");
        printf("\n");
     }
@@ -3580,7 +3034,7 @@ static int SetCellCmd(struct cmd_syndesc *as, void *arock)
     args.junk = 0;
 
 #ifdef WIN32
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -3591,7 +3045,7 @@ static int SetCellCmd(struct cmd_syndesc *as, void *arock)
     }
 #endif /* WIN32 */
 
-    if (! as->parms[1].items) 
+    if (! as->parms[1].items)
         args.stat |= CM_SETCELLFLAG_SUID; /* default to -nosuid */
 
     /* set stat for all listed cells */
@@ -3614,7 +3068,7 @@ static int SetCellCmd(struct cmd_syndesc *as, void *arock)
        blob.out = (caddr_t) 0;
        code = pioctl_utf8(0, VIOC_SETCELLSTATUS, &blob, 1);
        if (code) {
-           Die(errno, info.name);      /* XXX added cell name to Die() call */
+           fs_Die(errno, info.name);      /* XXX added cell name to fs_Die() call */
             error = 1;
        }
     }
@@ -3639,8 +3093,8 @@ VLDBInit(int noAuthFlag, struct afsconf_cell *info)
 
     cm_GetConfigDir(confDir, sizeof(confDir));
 
-    code = ugen_ClientInit(noAuthFlag, confDir, 
-                          info->name, 0, &uclient, 
+    code = ugen_ClientInit(noAuthFlag, confDir,
+                          info->name, 0, &uclient,
                            NULL, pn, rxkad_clear,
                            VLDB_MAXSERVERS, AFSCONF_VLDBSERVICE, 50,
                            0, 0, USER_SERVICE_ID);
@@ -3650,7 +3104,7 @@ VLDBInit(int noAuthFlag, struct afsconf_cell *info)
 
 static struct ViceIoctl gblob;
 static int debug = 0;
-/* 
+/*
  * here follow some routines in suport of the setserverprefs and
  * getserverprefs commands.  They are:
  * SetPrefCmd  "top-level" routine
@@ -3662,7 +3116,7 @@ static int debug = 0;
  */
 
 #ifdef WIN32
-static int 
+static int
 pokeServers(void)
 {
     int code;
@@ -3707,7 +3161,7 @@ pokeServers(void)
 #ifdef WIN32
 static int
 addServer(char *name, unsigned short rank)
-{  
+{
     int code;
     cm_SSetPref_t *ssp;
     cm_SPref_t *sp;
@@ -3857,7 +3311,7 @@ SetPrefCmd(struct cmd_syndesc *as, void * arock)
     char name[80];
     afs_int32 rank;
     cm_SSetPref_t *ssp;
-    
+
     ssp = (cm_SSetPref_t *)space;
     ssp->flags = 0;
     ssp->num_servers = 0;
@@ -3866,7 +3320,7 @@ SetPrefCmd(struct cmd_syndesc *as, void * arock)
     gblob.out = space;
     gblob.out_size = AFS_PIOCTL_MAXSIZE;
 
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -3878,7 +3332,7 @@ SetPrefCmd(struct cmd_syndesc *as, void * arock)
         if (debug) fprintf(stderr,"opening file %s\n",ti->data);
         if (!(infd = fopen(ti->data,"r" ))) {
             code = errno;
-            Die(errno,ti->data);
+            fs_Die(errno,ti->data);
         } else {
 #if _MSC_VER < 1400
             while ( fscanf(infd, "%79s%ld", name, &rank) != EOF) {
@@ -3918,7 +3372,7 @@ SetPrefCmd(struct cmd_syndesc *as, void * arock)
         }
     }
     code = pokeServers();
-    if (debug) 
+    if (debug)
         printf("now working on vlservers, code=%d, errno=%d\n",code,errno);
 
     ssp = (cm_SSetPref_t *)space;
@@ -3933,20 +3387,20 @@ SetPrefCmd(struct cmd_syndesc *as, void * arock)
                 break;
             }
             code = addServer (ti->data, (unsigned short) atol(ti->next->data));
-            if (debug) 
+            if (debug)
                 printf("set vl prefs %s %s\n", ti->data, ti->next->data);
             ti=ti->next;
         }
     }
 
     if (as->parms[1].items) {
-        if (debug) 
+        if (debug)
             printf("now poking vlservers\n");
         code = pokeServers();
     }
 
-    if (code) 
-        Die(errno,0);
+    if (code)
+        fs_Die(errno,0);
 
     return code;
 }
@@ -4067,14 +3521,14 @@ SetPrefCmd(struct cmd_syndesc *as, void *arock)
     }
 
     if (error > 0)
-       Die(error, 0);
+       fs_Die(error, 0);
 
     return error ? 1 : 0;
 }
 #endif /* WIN32 */
 
 #ifdef WIN32
-static int 
+static int
 GetPrefCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
@@ -4088,13 +3542,13 @@ GetPrefCmd(struct cmd_syndesc *as, void *arock)
     struct cm_SPrefRequest *in;
     struct cm_SPrefInfo *out;
     int i;
-    
+
     code = 0;
     ti = as->parms[0].items;  /* -file */
     if (ti) {
         if (debug) fprintf(stderr,"opening file %s\n",ti->data);
         if (!(outfd = freopen(ti->data,"w",stdout))) {
-            Die(errno,ti->data);
+            fs_Die(errno,ti->data);
             return errno;
         }
     }
@@ -4115,12 +3569,12 @@ GetPrefCmd(struct cmd_syndesc *as, void *arock)
         blob.out_size = AFS_PIOCTL_MAXSIZE;
 
         in->num_servers = (AFS_PIOCTL_MAXSIZE - 2*sizeof(short))/sizeof(struct cm_SPref);
-        in->flags = vlservers; 
+        in->flags = vlservers;
 
         code = pioctl_utf8(0, VIOC_GETSPREFS, &blob, 1);
         if (code){
             perror("getserverprefs pioctl");
-            Die (errno,0);
+            fs_Die (errno,0);
         }
         else {
             out = (struct cm_SPrefInfo *) blob.out;
@@ -4138,7 +3592,7 @@ GetPrefCmd(struct cmd_syndesc *as, void *arock)
                    }
                     name=tbuffer;
                 }
-                printf ("%-50s %5u\n",name,out->servers[i].rank);      
+                printf ("%-50s %5u\n",name,out->servers[i].rank);
             }
 
             in->offset = out->next_offset;
@@ -4241,7 +3695,7 @@ SmbUnicodeCmd(struct cmd_syndesc * asp, void * arock)
         inValue = 2;
     }
 
-    if (inValue != 0 && !IsAdmin()) {
+    if (inValue != 0 && !fs_IsAdmin()) {
         fprintf (stderr, "Permission denied: Requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -4253,7 +3707,7 @@ SmbUnicodeCmd(struct cmd_syndesc * asp, void * arock)
 
     code = pioctl_utf8(NULL, VIOC_UNICODECTL, &blob, 1);
     if (code) {
-        Die(errno, NULL);
+        fs_Die(errno, NULL);
         return code;
     }
 
@@ -4304,7 +3758,7 @@ GetFidCmd(struct cmd_syndesc *as, void *arock)
             options.field_flags |= CM_IOCTL_QOPTS_FIELD_FID;
             options.fid = fid;
         } else {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
@@ -4314,7 +3768,7 @@ GetFidCmd(struct cmd_syndesc *as, void *arock)
 
         code = pioctl_utf8(ti->data, VIOC_GETFILETYPE, &blob, 1);
         if (code || blob.out_size != sizeof(filetype)) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
@@ -4325,7 +3779,7 @@ GetFidCmd(struct cmd_syndesc *as, void *arock)
         if (code == 0)
             cell[CELL_MAXNAMELEN - 1] = '\0';
         printf("%s %s (%u.%u.%u) contained in cell %s\n",
-                filetypestr(filetype),
+                fs_filetypestr(filetype),
                 ti->data, fid.volume, fid.vnode, fid.unique,
                 code ? "unknown-cell" : cell);
     }
@@ -4343,7 +3797,7 @@ UuidCmd(struct cmd_syndesc *asp, void *arock)
 
     if (asp->parms[0].items) {
 #ifdef WIN32
-        if ( !IsAdmin() ) {
+        if ( !fs_IsAdmin() ) {
             fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
             return EACCES;
         }
@@ -4365,7 +3819,7 @@ UuidCmd(struct cmd_syndesc *asp, void *arock)
 
     code = pioctl_utf8(NULL, VIOC_UUIDCTL, &blob, 1);
     if (code || blob.out_size != sizeof(outValue)) {
-        Die(errno, NULL);
+        fs_Die(errno, NULL);
         return code;
     }
 
@@ -4388,9 +3842,9 @@ TraceCmd(struct cmd_syndesc *asp, void *arock)
     struct ViceIoctl blob;
     long inValue;
     long outValue;
-    
+
 #ifdef WIN32
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -4405,32 +3859,32 @@ TraceCmd(struct cmd_syndesc *asp, void *arock)
         fprintf(stderr, "fs trace: must use at most one of '-off' or '-on'\n");
         return EINVAL;
     }
-        
+
     /* determine if we're turning this tracing on or off */
     inValue = 0;
     if (asp->parms[0].items)
         inValue = 3;           /* enable */
-    else if (asp->parms[1].items) 
+    else if (asp->parms[1].items)
         inValue = 2;   /* disable */
-    if (asp->parms[2].items) 
+    if (asp->parms[2].items)
         inValue |= 4;          /* do reset */
-    if (asp->parms[3].items) 
+    if (asp->parms[3].items)
         inValue |= 8;          /* dump */
-        
+
     blob.in_size = sizeof(long);
     blob.in = (char *) &inValue;
     blob.out_size = sizeof(long);
     blob.out = (char *) &outValue;
-        
+
     code = pioctl_utf8(NULL, VIOC_TRACECTL, &blob, 1);
     if (code || blob.out_size != sizeof(long)) {
-        Die(errno, NULL);
+        fs_Die(errno, NULL);
         return code;
     }
 
-    if (outValue) 
+    if (outValue)
         printf("AFS tracing enabled.\n");
-    else 
+    else
         printf("AFS tracing disabled.\n");
 
     return 0;
@@ -4440,7 +3894,7 @@ static void sbusage(void)
 {
     fprintf(stderr, "example usage: %s storebehind -files *.o -kb 99999 -default 0\n", pn);
     fprintf(stderr, "               %s sb 50000 *.[ao] -default 10\n", pn);
-}       
+}
 
 /* fs sb -kbytes 9999 -files *.o -default 64 */
 static int
@@ -4456,8 +3910,8 @@ StoreBehindCmd(struct cmd_syndesc *as, void *arock)
     int error = 0;
 
 #ifdef WIN32
-    if ( !IsAdmin() ) {
-        fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");  
+    if ( !fs_IsAdmin() ) {
+        fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
 #endif /* WIN32 */
@@ -4513,14 +3967,14 @@ StoreBehindCmd(struct cmd_syndesc *as, void *arock)
        /* Do this solely to see if the file is there */
        code = pioctl_utf8(ti->data, VIOCWHEREIS, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
        }
 
        code = pioctl_utf8(ti->data, VIOC_STOREBEHIND, &blob, 1);
        if (code) {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
        }
@@ -4544,7 +3998,7 @@ StoreBehindCmd(struct cmd_syndesc *as, void *arock)
        tsb.sb_default = allfiles;
        code = pioctl_utf8(0, VIOC_STOREBEHIND, &blob, 1);
        if (code) {
-           Die(errno, ((allfiles == -1) ? 0 : "-allfiles"));
+           fs_Die(errno, ((allfiles == -1) ? 0 : "-allfiles"));
            error = 1;
        }
     }
@@ -4558,15 +4012,15 @@ StoreBehindCmd(struct cmd_syndesc *as, void *arock)
     return error;
 }
 
-static afs_int32 
+static afs_int32
 SetCryptCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code = 0, flag;
     struct ViceIoctl blob;
     char *tp;
+
 #ifdef WIN32
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -4589,18 +4043,18 @@ SetCryptCmd(struct cmd_syndesc *as, void *arock)
     blob.out_size = 0;
     code = pioctl_utf8(0, VIOC_SETRXKCRYPT, &blob, 1);
     if (code)
-        Die(code, NULL);
+        fs_Die(code, NULL);
     return 0;
 }
 
-static afs_int32 
+static afs_int32
 GetCryptCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code = 0, flag;
     struct ViceIoctl blob;
     char *tp;
     errno_t err;
+
     blob.in = NULL;
     blob.in_size = 0;
     blob.out_size = sizeof(flag);
@@ -4609,7 +4063,7 @@ GetCryptCmd(struct cmd_syndesc *as, void *arock)
     code = pioctl_utf8(0, VIOC_GETRXKCRYPT, &blob, 1);
 
     if (code || blob.out_size != sizeof(flag))
-        Die(code, NULL);
+        fs_Die(code, NULL);
     else {
         tp = space;
 #if _MSC_VER < 1400
@@ -4641,7 +4095,7 @@ MemDumpCmd(struct cmd_syndesc *asp, void *arock)
     long inValue = 0;
     long outValue;
 
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -4665,11 +4119,11 @@ MemDumpCmd(struct cmd_syndesc *asp, void *arock)
 
     code = pioctl_utf8(NULL, VIOC_TRACEMEMDUMP, &blob, 1);
     if (code || blob.out_size != sizeof(long)) {
-        Die(errno, NULL);
+        fs_Die(errno, NULL);
         return code;
     }
 
-    if (!outValue) { 
+    if (!outValue) {
         printf("AFS memdump created.\n");
         return 0;
     } else {
@@ -4686,7 +4140,7 @@ MiniDumpCmd(struct cmd_syndesc *asp, void *arock)
     SC_HANDLE hManager = NULL;
     SC_HANDLE hService = NULL;
 
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -4709,7 +4163,7 @@ MiniDumpCmd(struct cmd_syndesc *asp, void *arock)
         return 0;
     }
 
-  failure: 
+  failure:
     if (hService)
         CloseServiceHandle(hService);
     if (hManager)
@@ -4727,7 +4181,7 @@ CSCPolicyCmd(struct cmd_syndesc *asp, void *arock)
     HKEY hkCSCPolicy;
     size_t len;
 
-    if ( !IsAdmin() ) {
+    if ( !fs_IsAdmin() ) {
         fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
         return EACCES;
     }
@@ -4744,13 +4198,13 @@ CSCPolicyCmd(struct cmd_syndesc *asp, void *arock)
     {
         char *policy;
 
-        RegCreateKeyEx( HKEY_LOCAL_MACHINE, 
+        RegCreateKeyEx( HKEY_LOCAL_MACHINE,
                          AFSREG_CLT_OPENAFS_SUBKEY "\\CSCPolicy",
-                        0, 
-                        "AFS", 
+                        0,
+                        "AFS",
                         REG_OPTION_NON_VOLATILE,
                         (IsWow64()?KEY_WOW64_64KEY:0)|KEY_WRITE,
-                        NULL, 
+                        NULL,
                         &hkCSCPolicy,
                         NULL );
 
@@ -4759,7 +4213,7 @@ CSCPolicyCmd(struct cmd_syndesc *asp, void *arock)
             return EACCES;
         }
 
-        if ( !IsAdmin() ) {
+        if ( !fs_IsAdmin() ) {
             fprintf (stderr,"Permission denied: requires AFS Client Administrator access.\n");
             RegCloseKey(hkCSCPolicy);
             return EACCES;
@@ -4767,7 +4221,7 @@ CSCPolicyCmd(struct cmd_syndesc *asp, void *arock)
 
         policy = "manual";
         len = 6;
-               
+
         if (asp->parms[1].items) {
             policy = "manual";
             len = 6;
@@ -4785,9 +4239,9 @@ CSCPolicyCmd(struct cmd_syndesc *asp, void *arock)
             len = 7;
         }
         RegSetValueEx( hkCSCPolicy, share, 0, REG_SZ, policy, (DWORD)len+1);
-               
+
         printf("CSC policy on share \"%s\" changed to \"%s\".\n\n", share, policy);
-        printf("Close all applications that accessed files on this share or restart AFS Client for the change to take effect.\n"); 
+        printf("Close all applications that accessed files on this share or restart AFS Client for the change to take effect.\n");
     }
     else
     {
@@ -4800,13 +4254,13 @@ CSCPolicyCmd(struct cmd_syndesc *asp, void *arock)
 
         /* list current csc policies */
 
-        RegCreateKeyEx( HKEY_LOCAL_MACHINE, 
+        RegCreateKeyEx( HKEY_LOCAL_MACHINE,
                         AFSREG_CLT_OPENAFS_SUBKEY "\\CSCPolicy",
-                        0, 
-                        "AFS", 
+                        0,
+                        "AFS",
                         REG_OPTION_NON_VOLATILE,
                         (IsWow64()?KEY_WOW64_64KEY:0)|KEY_READ|KEY_QUERY_VALUE,
-                        NULL, 
+                        NULL,
                         &hkCSCPolicy,
                         NULL );
 
@@ -4823,7 +4277,7 @@ CSCPolicyCmd(struct cmd_syndesc *asp, void *arock)
                          NULL,  /* lpcbSecurityDescriptor */
                          NULL   /* lpftLastWriteTime */
                          );
-               
+
         printf("Current CSC policies:\n");
         for ( dwIndex = 0; dwIndex < dwPolicies; dwIndex ++ ) {
 
@@ -4958,7 +4412,7 @@ SetClientAddrsCmd(struct cmd_syndesc *as, void *arock)
 
     code = pioctl_utf8(0, VIOC_SETCPREFS, &blob, 1);   /* network order */
     if (code) {
-       Die(errno, 0);
+       fs_Die(errno, 0);
        error = 1;
     }
 
@@ -5096,7 +4550,7 @@ FlushMountCmd(struct cmd_syndesc *as, void *arock)
            if (errno == EINVAL) {
                fprintf(stderr, "'%s' is not a mount point.\n", ti->data);
            } else {
-               Die(errno, (ti->data ? ti->data : parent_dir));
+               fs_Die(errno, (ti->data ? ti->data : parent_dir));
            }
            error = 1;
        }
@@ -5132,7 +4586,7 @@ RxStatProcCmd(struct cmd_syndesc *as, void *arock)
 
     code = pioctl_utf8(NULL, VIOC_RXSTAT_PROC, &blob, 1);
     if (code != 0) {
-       Die(errno, NULL);
+       fs_Die(errno, NULL);
        return 1;
     }
 
@@ -5166,7 +4620,7 @@ RxStatPeerCmd(struct cmd_syndesc *as, void *arock)
 
     code = pioctl_utf8(NULL, VIOC_RXSTAT_PEER, &blob, 1);
     if (code != 0) {
-       Die(errno, NULL);
+       fs_Die(errno, NULL);
        return 1;
     }
 
@@ -5252,14 +4706,14 @@ TestVolStatCmd(struct cmd_syndesc *as, void *arock)
 
     code = pioctl_utf8(NULL, VIOC_VOLSTAT_TEST, &blob, 1);
     if (code != 0) {
-       Die(errno, NULL);
+       fs_Die(errno, NULL);
        return 1;
     }
 
     return 0;
 }
 
-static int 
+static int
 ChOwnCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
@@ -5267,7 +4721,7 @@ ChOwnCmd(struct cmd_syndesc *as, void *arock)
     struct cmd_item *ti;
     int error = 0;
     int literal = 0;
-    struct { 
+    struct {
         cm_ioctlQueryOptions_t options;
         afs_uint32 owner;
     } inData;
@@ -5306,14 +4760,14 @@ ChOwnCmd(struct cmd_syndesc *as, void *arock)
             inData.options.field_flags |= CM_IOCTL_QOPTS_FIELD_FID;
             inData.options.fid = fid;
         } else {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
 
-        /* 
+        /*
          * if the owner was specified as a numeric value,
-         * then we can just use it.  Otherwise, we need 
+         * then we can just use it.  Otherwise, we need
          * to know the cell of the path to determine which
          * ptserver to contact in order to convert the name
          * to a numeric value.
@@ -5324,21 +4778,21 @@ ChOwnCmd(struct cmd_syndesc *as, void *arock)
 
             code = pioctl_utf8(ti->data, VIOC_FILE_CELL_NAME, &blob, 1);
             if (code) {
-                Die(errno, ti->data);
+                fs_Die(errno, ti->data);
                 error = 1;
                 continue;
             }
             cell[CELL_MAXNAMELEN - 1] = '\0';
-            /* 
+            /*
              * We now know the cell for the target and we need to
-             * convert the ownerStr to the Id for this user 
+             * convert the ownerStr to the Id for this user
              */
             pr_Initialize(1, confDir, cell);
             code = pr_SNameToId(ownerStr, &inData.owner);
             pr_End();
 
             if (code || inData.owner == ANONYMOUSID ) {
-                Die(ECHILD, ti->data);
+                fs_Die(ECHILD, ti->data);
                 error = 1;
                 continue;
             }
@@ -5351,13 +4805,13 @@ ChOwnCmd(struct cmd_syndesc *as, void *arock)
        blob.out_size = 0;
        code = pioctl_utf8(ti->data, VIOC_SETOWNER, &blob, 1);
        if (code) {
-            Die(errno, ti->data);
+            fs_Die(errno, ti->data);
         }
     }
     return error;
 }
 
-static int 
+static int
 ChGrpCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code;
@@ -5365,7 +4819,7 @@ ChGrpCmd(struct cmd_syndesc *as, void *arock)
     struct cmd_item *ti;
     int error = 0;
     int literal = 0;
-    struct { 
+    struct {
         cm_ioctlQueryOptions_t options;
         afs_uint32 group;
     } inData;
@@ -5404,14 +4858,14 @@ ChGrpCmd(struct cmd_syndesc *as, void *arock)
             inData.options.field_flags |= CM_IOCTL_QOPTS_FIELD_FID;
             inData.options.fid = fid;
         } else {
-           Die(errno, ti->data);
+           fs_Die(errno, ti->data);
            error = 1;
            continue;
         }
 
-        /* 
+        /*
          * if the group was specified as a numeric value,
-         * then we can just use it.  Otherwise, we need 
+         * then we can just use it.  Otherwise, we need
          * to know the cell of the path to determine which
          * ptserver to contact in order to convert the name
          * to a numeric value.
@@ -5422,21 +4876,21 @@ ChGrpCmd(struct cmd_syndesc *as, void *arock)
 
             code = pioctl_utf8(ti->data, VIOC_FILE_CELL_NAME, &blob, 1);
             if (code) {
-                Die(errno, ti->data);
+                fs_Die(errno, ti->data);
                 error = 1;
                 continue;
             }
             cell[CELL_MAXNAMELEN - 1] = '\0';
-            /* 
+            /*
              * We now know the cell for the target and we need to
-             * convert the groupStr to the Id for this user 
+             * convert the groupStr to the Id for this user
              */
             pr_Initialize(1, confDir, cell);
             code = pr_SNameToId(groupStr, &inData.group);
             pr_End();
 
             if (code || inData.group == ANONYMOUSID ) {
-                Die(ECHILD, ti->data);
+                fs_Die(ECHILD, ti->data);
                 error = 1;
                 continue;
             }
@@ -5449,60 +4903,113 @@ ChGrpCmd(struct cmd_syndesc *as, void *arock)
        blob.out_size = 0;
        code = pioctl_utf8(ti->data, VIOC_SETGROUP, &blob, 1);
        if (code) {
-            Die(errno, ti->data);
+            fs_Die(errno, ti->data);
         }
     }
     return error;
 }
 
-#ifndef WIN32
-#include "AFS_component_version_number.c"
-#endif
 
-static void
-FreeUtf8CmdLine(int argc, char ** argv)
-{
-    int i;
-    for (i=0; i < argc; i++) {
-        if (argv[i])
-            free(argv[i]);
-    }
-    free(argv);
-}
 
-static char **
-MakeUtf8Cmdline(int argc, const wchar_t **wargv)
+static int
+ChModCmd(struct cmd_syndesc *as, void *arock)
 {
-    char ** argv;
-    int i;
+    afs_int32 code;
+    struct ViceIoctl blob;
+    struct cmd_item *ti;
+    int error = 0;
+    int literal = 0;
+    struct {
+        cm_ioctlQueryOptions_t options;
+        afs_uint32 unixModeBits;
+    } inData;
+    afs_uint32 unixModeBits;
+    afs_int32  absolute = 0;
+    char * unixModeStr;
+    char confDir[257];
 
-    argv = calloc(argc, sizeof(argv[0]));
-    if (argv == NULL)
-        return NULL;
+    cm_GetConfigDir(confDir, sizeof(confDir));
 
-    for (i=0; i < argc; i++) {
-        int s;
+    if (as->parms[2].items)
+        literal = 1;
 
-        s = WideCharToMultiByte(CP_UTF8, 0, wargv[i], -1, NULL, 0, NULL, FALSE);
-        if (s == 0 ||
-            (argv[i] = calloc(s+1, sizeof(char))) == NULL) {
-            break;
+    unixModeStr = as->parms[0].items->data;
+    if (*unixModeStr >= '0' && *unixModeStr <= '7') {
+        unixModeBits = 0;
+        absolute = 1;
+        while (*unixModeStr >= '0' && *unixModeStr <= '7')
+            unixModeBits = (unixModeBits << 3) | (*unixModeStr++ & 07);
+        if (*unixModeStr) {
+            fs_Die(EINVAL, "invalid mode");
+            return(1);
         }
+        unixModeBits &= ALL_MODES;
+    }
 
-        s = WideCharToMultiByte(CP_UTF8, 0, wargv[i], -1, argv[i], s+1, NULL, FALSE);
-        if (s == 0) {
-            break;
+    SetDotDefault(&as->parms[1].items);
+    for(ti=as->parms[1].items; ti; ti=ti->next) {
+        cm_fid_t fid;
+        afs_uint32 filetype;
+
+        /* once per file */
+        memset(&fid, 0, sizeof(fid));
+        memset(&inData, 0, sizeof(inData));
+        filetype = 0;
+        inData.options.size = sizeof(inData.options);
+        inData.options.field_flags |= CM_IOCTL_QOPTS_FIELD_LITERAL;
+        inData.options.literal = literal;
+       blob.in_size = inData.options.size;    /* no variable length data */
+        blob.in = &inData;
+
+        blob.out_size = sizeof(cm_fid_t);
+        blob.out = (char *) &fid;
+        if (0 == pioctl_utf8(ti->data, VIOCGETFID, &blob, 1) &&
+            blob.out_size == sizeof(cm_fid_t)) {
+            inData.options.field_flags |= CM_IOCTL_QOPTS_FIELD_FID;
+            inData.options.fid = fid;
+        } else {
+           fs_Die(errno, ti->data);
+           error = 1;
+           continue;
         }
-    }
 
-    if (i < argc) {
-        FreeUtf8CmdLine(argc, argv);
-        return NULL;
-    }
+        /*
+         * if the mode was specified as an absolute numeric,
+         * value we can simply apply it to all of the listed
+         * file paths.  Otherwise, we must obtain the old mode
+         * value in order to compute the new value from the
+         * symbolic representation.
+         */
+        if (!absolute) {
+            blob.in_size = 0;
+            blob.out_size = sizeof(afs_uint32);
+            blob.out = (char *)&unixModeBits;
+            if (pioctl_utf8(ti->data, VIOC_GETUNIXMODE, &blob, 1) != 0)
+            {
+                fs_Die(errno, ti->data);
+                error = 1;
+                continue;
+            }
+            inData.unixModeBits = parsemode(unixModeStr, unixModeBits);
+        } else {
+            inData.unixModeBits = unixModeBits;
+        }
 
-    return argv;
+        blob.in_size = sizeof(inData);
+       blob.out = NULL;
+       blob.out_size = 0;
+       code = pioctl_utf8(ti->data, VIOC_SETUNIXMODE, &blob, 1);
+       if (code) {
+            fs_Die(errno, ti->data);
+        }
+    }
+    return error;
 }
 
+#ifndef WIN32
+#include "AFS_component_version_number.c"
+#endif
+
 int wmain(int argc, wchar_t **wargv)
 {
     afs_int32 code;
@@ -5511,13 +5018,13 @@ int wmain(int argc, wchar_t **wargv)
 
 #ifdef AFS_AIX32_ENV
     /*
-     * The following signal action for AIX is necessary so that in case of a 
-     * crash (i.e. core is generated) we can include the user's data section 
+     * The following signal action for AIX is necessary so that in case of a
+     * crash (i.e. core is generated) we can include the user's data section
      * in the core dump. Unfortunately, by default, only a partial core is
      * generated which, in many cases, isn't too useful.
      */
     struct sigaction nsa;
-    
+
     sigemptyset(&nsa.sa_mask);
     nsa.sa_handler = SIG_DFL;
     nsa.sa_flags = SA_FULLDUMP;
@@ -5529,7 +5036,9 @@ int wmain(int argc, wchar_t **wargv)
     WSAStartup(0x0101, &WSAjunk);
 #endif /* WIN32 */
 
-    argv = MakeUtf8Cmdline(argc, wargv);
+    fs_SetProcessName(pn);
+
+    argv = fs_MakeUtf8Cmdline(argc, wargv);
 
     /* try to find volume location information */
     osi_Init();
@@ -5569,7 +5078,7 @@ int wmain(int argc, wchar_t **wargv)
     cmd_AddParm(ts, "-id", CMD_FLAG, CMD_OPTIONAL, "initial directory acl (DFS only)");
     cmd_AddParm(ts, "-if", CMD_FLAG, CMD_OPTIONAL, "initial file acl (DFS only)");
     cmd_CreateAlias(ts, "sa");
-    
+
     ts = cmd_CreateSyntax("listacl", ListACLCmd, NULL, "list access control list");
     cmd_AddParm(ts, "-path", CMD_LIST, CMD_OPTIONAL, "dir/file path");
     parm_listacl_id = ts->nParms;
@@ -5577,10 +5086,10 @@ int wmain(int argc, wchar_t **wargv)
     cmd_AddParm(ts, "-if", CMD_FLAG, CMD_OPTIONAL, "initial file acl");
     cmd_AddParm(ts, "-cmd", CMD_FLAG, CMD_OPTIONAL, "output as 'fs setacl' command");
     cmd_CreateAlias(ts, "la");
-    
+
     ts = cmd_CreateSyntax("cleanacl", CleanACLCmd, NULL, "clean up access control list");
     cmd_AddParm(ts, "-path", CMD_LIST, CMD_OPTIONAL, "dir/file path");
-    
+
     ts = cmd_CreateSyntax("copyacl", CopyACLCmd, NULL, "copy access control list");
     cmd_AddParm(ts, "-fromdir", CMD_SINGLE, 0, "source directory (or DFS file)");
     cmd_AddParm(ts, "-todir", CMD_LIST, 0, "destination directory (or DFS file)");
@@ -5588,13 +5097,13 @@ int wmain(int argc, wchar_t **wargv)
     parm_copyacl_id = ts->nParms;
     cmd_AddParm(ts, "-id", CMD_FLAG, CMD_OPTIONAL, "initial directory acl");
     cmd_AddParm(ts, "-if", CMD_FLAG, CMD_OPTIONAL, "initial file acl");
-    
+
     cmd_CreateAlias(ts, "ca");
 
     ts = cmd_CreateSyntax("flush", FlushCmd, NULL, "flush file from cache");
     cmd_AddParm(ts, "-path", CMD_LIST, CMD_OPTIONAL, "dir/file path");
     cmd_AddParm(ts, "-literal", CMD_FLAG, CMD_OPTIONAL, "literal evaluation of mountpoints and symlinks");
-    
+
 #ifndef WIN32
     ts = cmd_CreateSyntax("flushmount", FlushMountCmd, NULL,
                            "flush mount symlink from cache");
@@ -5610,7 +5119,7 @@ int wmain(int argc, wchar_t **wargv)
     cmd_AddParm(ts, "-motd", CMD_SINGLE, CMD_OPTIONAL, "message of the day");
     cmd_AddParm(ts, "-offlinemsg", CMD_SINGLE, CMD_OPTIONAL, "offline message");
     cmd_CreateAlias(ts, "sv");
-    
+
     ts = cmd_CreateSyntax("messages", MessagesCmd, NULL, "control Cache Manager messages");
     cmd_AddParm(ts, "-show", CMD_SINGLE, CMD_OPTIONAL, "[user|console|all|none]");
 
@@ -5619,21 +5128,21 @@ int wmain(int argc, wchar_t **wargv)
     cmd_AddParm(ts, "-literal", CMD_FLAG, CMD_OPTIONAL, "literal evaluation of mountpoints and symlinks");
     cmd_CreateAlias(ts, "lv");
     cmd_CreateAlias(ts, "listvol");
-    
+
     ts = cmd_CreateSyntax("listquota", ListQuotaCmd, NULL, "list volume quota");
     cmd_AddParm(ts, "-path", CMD_LIST, CMD_OPTIONAL, "dir/file path");
     cmd_CreateAlias(ts, "lq");
-    
+
     ts = cmd_CreateSyntax("diskfree", DiskFreeCmd, NULL, "show server disk space usage");
     cmd_AddParm(ts, "-path", CMD_LIST, CMD_OPTIONAL, "dir/file path");
     cmd_CreateAlias(ts, "df");
-    
+
     ts = cmd_CreateSyntax("quota", QuotaCmd, NULL, "show volume quota usage");
     cmd_AddParm(ts, "-path", CMD_LIST, CMD_OPTIONAL, "dir/file path");
-    
-    ts = cmd_CreateSyntax("lsmount", ListMountCmd, NULL, "list mount point");    
+
+    ts = cmd_CreateSyntax("lsmount", ListMountCmd, NULL, "list mount point");
     cmd_AddParm(ts, "-dir", CMD_LIST, 0, "directory");
-    
+
     ts = cmd_CreateSyntax("mkmount", MakeMountCmd, NULL, "make mount point");
     cmd_AddParm(ts, "-dir", CMD_SINGLE, 0, "directory");
     cmd_AddParm(ts, "-vol", CMD_SINGLE, 0, "volume name");
@@ -5644,35 +5153,35 @@ int wmain(int argc, wchar_t **wargv)
     /*
      *
      * defect 3069
-     * 
+     *
     cmd_AddParm(ts, "-root", CMD_FLAG, CMD_OPTIONAL, "create cellular mount point");
     */
 
-    
+
     ts = cmd_CreateSyntax("rmmount", RemoveMountCmd, NULL, "remove mount point");
     cmd_AddParm(ts, "-dir", CMD_LIST, 0, "directory");
-    
+
     ts = cmd_CreateSyntax("checkservers", CheckServersCmd, NULL, "check local cell's servers");
     cmd_AddParm(ts, "-cell", CMD_SINGLE, CMD_OPTIONAL, "cell to check");
     cmd_AddParm(ts, "-all", CMD_FLAG, CMD_OPTIONAL, "check all cells");
     cmd_AddParm(ts, "-fast", CMD_FLAG, CMD_OPTIONAL, "just list, don't check");
     cmd_AddParm(ts,"-interval",CMD_SINGLE,CMD_OPTIONAL,"seconds between probes");
-    
+
     ts = cmd_CreateSyntax("checkvolumes", CheckVolumesCmd, NULL, "check volumeID/name mappings");
     cmd_CreateAlias(ts, "checkbackups");
 
-    
+
     ts = cmd_CreateSyntax("setcachesize", SetCacheSizeCmd, NULL, "set cache size");
     cmd_AddParm(ts, "-blocks", CMD_SINGLE, CMD_OPTIONAL, "size in 1K byte blocks (0 => reset)");
     cmd_CreateAlias(ts, "cachesize");
 
     cmd_AddParm(ts, "-reset", CMD_FLAG, CMD_OPTIONAL, "reset size back to boot value");
-    
+
     ts = cmd_CreateSyntax("getcacheparms", GetCacheParmsCmd, NULL, "get cache usage info");
 
     ts = cmd_CreateSyntax("listcells", ListCellsCmd, NULL, "list configured cells");
     cmd_AddParm(ts, "-numeric", CMD_FLAG, CMD_OPTIONAL, "addresses only");
-    
+
     ts = cmd_CreateSyntax("setquota", SetQuotaCmd, NULL, "set volume quota");
     cmd_AddParm(ts, "-path", CMD_SINGLE, CMD_OPTIONAL, "dir/file path");
     cmd_AddParm(ts, "-max", CMD_SINGLE, 0, "max quota in kbytes");
@@ -5682,12 +5191,10 @@ int wmain(int argc, wchar_t **wargv)
     cmd_CreateAlias(ts, "sq");
 
     ts = cmd_CreateSyntax("newcell", NewCellCmd, NULL, "configure new cell");
-#ifndef WIN32
-    cmd_AddParm(ts, "-name", CMD_SINGLE, 0, "cell name");
-    cmd_AddParm(ts, "-servers", CMD_LIST, CMD_REQUIRED, "primary servers");
+    cmd_AddParm(ts, "-name", CMD_SINGLE, CMD_OPTIONAL, "cell name");
+    cmd_AddParm(ts, "-servers", CMD_LIST, CMD_OPTIONAL, "primary servers");
     cmd_AddParm(ts, "-linkedcell", CMD_SINGLE, CMD_OPTIONAL, "linked cell name");
 
-#ifdef FS_ENABLE_SERVER_DEBUG_PORTS
     /*
      * Turn this on only if you wish to be able to talk to a server which is listening
      * on alternative ports. This is not intended for general use and may not be
@@ -5696,8 +5203,10 @@ int wmain(int argc, wchar_t **wargv)
      */
     cmd_AddParm(ts, "-fsport", CMD_SINGLE, CMD_OPTIONAL, "cell's fileserver port");
     cmd_AddParm(ts, "-vlport", CMD_SINGLE, CMD_OPTIONAL, "cell's vldb server port");
-#endif
+    cmd_AddParm(ts, "-registry", CMD_FLAG, CMD_OPTIONAL, "add cell info to registry cellservdb");
+    cmd_AddParm(ts, "-dns",    CMD_FLAG, CMD_OPTIONAL,   "force use of dns");
 
+#ifndef WIN32
     ts = cmd_CreateSyntax("newalias", NewAliasCmd, NULL,
                          "configure new cell alias");
     cmd_AddParm(ts, "-alias", CMD_SINGLE, 0, "alias name");
@@ -5713,11 +5222,11 @@ int wmain(int argc, wchar_t **wargv)
     cmd_AddParm(ts, "-literal", CMD_FLAG, CMD_OPTIONAL, "literal evaluation of mountpoints and symlinks");
 
     ts = cmd_CreateSyntax("wscell", WSCellCmd, NULL, "list workstation's cell");
-    
+
     /*
      ts = cmd_CreateSyntax("primarycell", PrimaryCellCmd, 0, "obsolete (listed primary cell)");
      */
-    
+
 #ifndef AFS_NT40_ENV
     ts = cmd_CreateSyntax("monitor", MonitorCmd, NULL, "set cache monitor host address");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "host name or 'off'");
@@ -5726,7 +5235,7 @@ int wmain(int argc, wchar_t **wargv)
 
     ts = cmd_CreateSyntax("getcellstatus", GetCellCmd, NULL, "get cell status");
     cmd_AddParm(ts, "-cell", CMD_LIST, 0, "cell name");
-    
+
     ts = cmd_CreateSyntax("setcell", SetCellCmd, NULL, "set cell status");
     cmd_AddParm(ts, "-cell", CMD_LIST, 0, "cell name");
     cmd_AddParm(ts, "-suid", CMD_FLAG, CMD_OPTIONAL, "allow setuid programs");
@@ -5749,7 +5258,7 @@ int wmain(int argc, wchar_t **wargv)
     cmd_AddParm(ts, "-submounts", CMD_SINGLE, CMD_OPTIONAL, "allow nfs mounts to subdirs of /afs/.. ('on' or 'off')");
 #endif
 
-    ts = cmd_CreateSyntax("storebehind", StoreBehindCmd, NULL, 
+    ts = cmd_CreateSyntax("storebehind", StoreBehindCmd, NULL,
                          "store to server after file close");
     cmd_AddParm(ts, "-kbytes", CMD_SINGLE, CMD_OPTIONAL, "asynchrony for specified names");
     cmd_AddParm(ts, "-files", CMD_LIST, CMD_OPTIONAL, "specific pathnames");
@@ -5791,7 +5300,7 @@ int wmain(int argc, wchar_t **wargv)
     ts = cmd_CreateSyntax("memdump", MemDumpCmd, NULL, "dump memory allocs in debug builds");
     cmd_AddParm(ts, "-begin", CMD_FLAG, CMD_OPTIONAL, "set a memory checkpoint");
     cmd_AddParm(ts, "-end", CMD_FLAG, CMD_OPTIONAL, "dump memory allocs");
-    
+
     ts = cmd_CreateSyntax("cscpolicy", CSCPolicyCmd, NULL, "change client side caching policy for AFS shares");
     cmd_AddParm(ts, "-share", CMD_SINGLE, CMD_OPTIONAL, "AFS share");
     cmd_AddParm(ts, "-manual", CMD_FLAG, CMD_OPTIONAL, "manual caching of documents");
@@ -5826,92 +5335,18 @@ int wmain(int argc, wchar_t **wargv)
     cmd_AddParm(ts, "-path", CMD_LIST, CMD_OPTIONAL, "dir/file path");
     cmd_AddParm(ts, "-literal", CMD_FLAG, CMD_OPTIONAL, "literal evaluation of mountpoints and symlinks");
 
+    ts = cmd_CreateSyntax("chmod", ChModCmd, NULL, "set UNIX mode for object(s) in afs");
+    cmd_AddParm(ts, "-mode", CMD_SINGLE, 0, "UNIX mode bits");
+    cmd_AddParm(ts, "-path", CMD_LIST, CMD_OPTIONAL, "dir/file path");
+    cmd_AddParm(ts, "-literal", CMD_FLAG, CMD_OPTIONAL, "literal evaluation of mountpoints and symlinks");
+
     code = cmd_Dispatch(argc, argv);
 
-    if (rxInitDone) 
+    if (rxInitDone)
         rx_Finalize();
-    
-    FreeUtf8CmdLine(argc, argv);
-    
+
+    fs_FreeUtf8CmdLine(argc, argv);
+
     return code;
 }
 
-static void 
-Die(int code, char *filename)
-{ /*Die*/
-
-    if (code == EINVAL) {
-       if (filename)
-           fprintf(stderr,"%s: Invalid argument; it is possible that %s is not in AFS.\n", pn, filename);
-       else 
-            fprintf(stderr,"%s: Invalid argument.\n", pn);
-    }
-    else if (code == ENOENT) {
-       if (filename) 
-            fprintf(stderr,"%s: File '%s' doesn't exist\n", pn, filename);
-       else 
-            fprintf(stderr,"%s: no such file returned\n", pn);
-    }
-    else if (code == EROFS)  
-        fprintf(stderr,"%s: You can not change a backup or readonly volume\n", pn);
-    else if (code == EACCES || code == EPERM) {
-       if (filename) 
-            fprintf(stderr,"%s: You don't have the required access rights on '%s'\n", pn, filename);
-       else 
-            fprintf(stderr,"%s: You do not have the required rights to do this operation\n", pn);
-    }
-    else if (code == ENODEV) {
-       fprintf(stderr,"%s: AFS service may not have started.\n", pn);
-    }
-    else if (code == ESRCH) {   /* hack */
-       fprintf(stderr,"%s: Cell name not recognized.\n", pn);
-    }
-    else if (code == EPIPE) {   /* hack */
-       fprintf(stderr,"%s: Volume name or ID not recognized.\n", pn);
-    }
-    else if (code == EFBIG) {
-       fprintf(stderr,"%s: Cache size too large.\n", pn);
-    }
-    else if (code == ETIMEDOUT) {
-       if (filename)
-           fprintf(stderr,"%s:'%s': Connection timed out", pn, filename);
-       else
-           fprintf(stderr,"%s: Connection timed out", pn);
-    }
-    else if (code == EBUSY) {
-       if (filename) 
-            fprintf(stderr,"%s: All servers are busy on which '%s' resides\n", pn, filename);
-       else 
-            fprintf(stderr,"%s: All servers are busy\n", pn);
-    } 
-    else if (code == ENXIO) {
-       if (filename) 
-            fprintf(stderr,"%s: All volume instances are offline on which '%s' resides\n", pn, filename);
-       else 
-            fprintf(stderr,"%s: All volume instances are offline\n", pn);
-    } 
-    else if (code == ENOSYS) {
-       if (filename) 
-            fprintf(stderr,"%s: All servers are down on which '%s' resides\n", pn, filename);
-       else 
-            fprintf(stderr,"%s: All servers are down\n", pn);
-    } 
-    else if (code == ECHILD) {  /* hack */
-       if (filename) 
-            fprintf(stderr,"%s: Invalid owner specified for '%s'\n", pn, filename);
-       else 
-            fprintf(stderr,"%s: Invalid owner specified\n", pn);
-    } 
-    else {
-       if (filename) 
-            fprintf(stderr,"%s:'%s'", pn, filename);
-       else 
-            fprintf(stderr,"%s", pn);
-#ifdef WIN32
-       fprintf(stderr, ": code 0x%x\n", code);
-#else /* not WIN32 */
-       fprintf(stderr,": %s\n", afs_error_message(code));
-#endif /* not WIN32 */
-    }
-} /*Die*/
-