vos: do not mix memory allocation methods
[openafs.git] / src / volser / vos.c
index 94282f4..c1bd3b1 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * 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
@@ -10,6 +10,8 @@
 #include <afsconfig.h>
 #include <afs/param.h>
 
+#include <roken.h>
+
 #ifdef IGNORE_SOME_GCC_WARNINGS
 # pragma GCC diagnostic warning "-Wimplicit-function-declaration"
 #endif
@@ -75,7 +77,7 @@
 /* Local Prototypes */
 int PrintDiagnostics(char *astring, afs_int32 acode);
 int GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part,
-                  afs_int32 *voltype, struct nvldbentry *rentry);
+                  afs_int32 *voltype, struct uvldbentry *rentry);
 
 struct tqElem {
     afs_uint32 volid;
@@ -95,7 +97,10 @@ cmd_AddParm(ts, "-verbose", CMD_FLAG, CMD_OPTIONAL, "verbose");\
 cmd_AddParm(ts, "-encrypt", CMD_FLAG, CMD_OPTIONAL, "encrypt commands");\
 cmd_AddParm(ts, "-noresolve", CMD_FLAG, CMD_OPTIONAL, "don't resolve addresses"); \
 
-#define ERROR_EXIT(code) {error=(code); goto error_exit;}
+#define ERROR_EXIT(code) do { \
+    error = (code); \
+    goto error_exit; \
+} while (0)
 
 int rxInitDone = 0;
 struct rx_connection *tconn;
@@ -165,8 +170,7 @@ FileExists(char *filename)
 static int
 VolNameOK(char *name)
 {
-    int total;
-
+    size_t total;
 
     total = strlen(name);
     if (!strcmp(&name[total - 9], ".readonly")) {
@@ -182,7 +186,8 @@ VolNameOK(char *name)
 static int
 IsNumeric(char *name)
 {
-    int result, len, i;
+    int result, i;
+    size_t len;
     char *ptr;
 
     result = 1;
@@ -224,9 +229,9 @@ GetServerNoresolve(char *aname)
 afs_uint32
 GetServer(char *aname)
 {
-    register struct hostent *th;
+    struct hostent *th;
     afs_uint32 addr; /* in network byte order */
-    register afs_int32 code;
+    afs_int32 code;
     char hostname[MAXHOSTCHARS];
 
     if ((addr = GetServerNoresolve(aname)) == 0) {
@@ -285,10 +290,10 @@ IsPartValid(afs_int32 partId, afs_uint32 server, afs_int32 *code)
 
 
 
- /*sends the contents of file associated with <fd> and <blksize>  to Rx Stream 
+ /*sends the contents of file associated with <fd> and <blksize>  to Rx Stream
   * associated  with <call> */
-int 
-SendFile(usd_handle_t ufd, register struct rx_call *call, long blksize)
+int
+SendFile(usd_handle_t ufd, struct rx_call *call, long blksize)
 {
     char *buffer = (char *)0;
     afs_int32 error = 0;
@@ -343,9 +348,9 @@ WriteData(struct rx_call *call, void *rock)
     long blksize;
     afs_int32 error, code;
     int ufdIsOpen = 0;
-    afs_hyper_t filesize, currOffset; 
-    afs_uint32 buffer;         
-    afs_uint32 got;            
+    afs_hyper_t filesize, currOffset;
+    afs_uint32 buffer;
+    afs_uint32 got;
 
     error = 0;
 
@@ -917,11 +922,11 @@ XDisplayFormat2(volintXInfo *a_xInfoP, afs_uint32 a_servID, afs_int32 a_partID,
                fprintf(STDOUT, "serv\t\t%s\t%s\n", address, hostname);
                fprintf(STDOUT, "part\t\t%s\n", pname);
                 fprintf(STDOUT, "status\t\tOK\n");
-               fprintf(STDOUT, "backupID\t%lu\n", 
+               fprintf(STDOUT, "backupID\t%lu\n",
                        afs_printable_uint32_lu(a_xInfoP->backupID));
-               fprintf(STDOUT, "parentID\t%lu\n", 
+               fprintf(STDOUT, "parentID\t%lu\n",
                        afs_printable_uint32_lu(a_xInfoP->parentID));
-               fprintf(STDOUT, "cloneID\t\t%lu\n", 
+               fprintf(STDOUT, "cloneID\t\t%lu\n",
                        afs_printable_uint32_lu(a_xInfoP->cloneID));
                fprintf(STDOUT, "inUse\t\t%s\n", a_xInfoP->inUse ? "Y" : "N");
                switch (a_xInfoP->type) {
@@ -939,30 +944,30 @@ XDisplayFormat2(volintXInfo *a_xInfoP, afs_uint32 a_servID, afs_int32 a_partID,
                        break;
                }
                t = a_xInfoP->creationDate;
-               fprintf(STDOUT, "creationDate\t%-9lu\t%s", 
+               fprintf(STDOUT, "creationDate\t%-9lu\t%s",
                        afs_printable_uint32_lu(a_xInfoP->creationDate),
                        ctime(&t));
 
                t = a_xInfoP->accessDate;
-               fprintf(STDOUT, "accessDate\t%-9lu\t%s", 
+               fprintf(STDOUT, "accessDate\t%-9lu\t%s",
                        afs_printable_uint32_lu(a_xInfoP->accessDate),
                        ctime(&t));
 
                t = a_xInfoP->updateDate;
-               fprintf(STDOUT, "updateDate\t%-9lu\t%s", 
+               fprintf(STDOUT, "updateDate\t%-9lu\t%s",
                        afs_printable_uint32_lu(a_xInfoP->updateDate),
                        ctime(&t));
 
                t = a_xInfoP->backupDate;
-               fprintf(STDOUT, "backupDate\t%-9lu\t%s", 
+               fprintf(STDOUT, "backupDate\t%-9lu\t%s",
                        afs_printable_uint32_lu(a_xInfoP->backupDate),
                        ctime(&t));
 
                t = a_xInfoP->copyDate;
-               fprintf(STDOUT, "copyDate\t%-9lu\t%s", 
+               fprintf(STDOUT, "copyDate\t%-9lu\t%s",
                        afs_printable_uint32_lu(a_xInfoP->copyDate),
                        ctime(&t));
-               
+
                fprintf(STDOUT, "diskused\t%u\n", a_xInfoP->size);
                fprintf(STDOUT, "maxquota\t%u\n", a_xInfoP->maxquota);
 
@@ -1067,7 +1072,7 @@ DisplayFormat2(long server, long partition, volintInfo *pntr)
     if (pntr->status == VOK)
         fprintf(STDOUT, "name\t\t%s\n", pntr->name);
 
-    fprintf(STDOUT, "id\t\t%lu\n", 
+    fprintf(STDOUT, "id\t\t%lu\n",
            afs_printable_uint32_lu(pntr->volid));
     fprintf(STDOUT, "serv\t\t%s\t%s\n", address, hostname);
     fprintf(STDOUT, "part\t\t%s\n", pname);
@@ -1082,11 +1087,11 @@ DisplayFormat2(long server, long partition, volintInfo *pntr)
        fprintf(STDOUT, "status\t\tUNATTACHABLE\n");
        return;
     }
-    fprintf(STDOUT, "backupID\t%lu\n", 
+    fprintf(STDOUT, "backupID\t%lu\n",
            afs_printable_uint32_lu(pntr->backupID));
-    fprintf(STDOUT, "parentID\t%lu\n", 
+    fprintf(STDOUT, "parentID\t%lu\n",
            afs_printable_uint32_lu(pntr->parentID));
-    fprintf(STDOUT, "cloneID\t\t%lu\n", 
+    fprintf(STDOUT, "cloneID\t\t%lu\n",
            afs_printable_uint32_lu(pntr->cloneID));
     fprintf(STDOUT, "inUse\t\t%s\n", pntr->inUse ? "Y" : "N");
     fprintf(STDOUT, "needsSalvaged\t%s\n", pntr->needsSalvaged ? "Y" : "N");
@@ -1107,43 +1112,43 @@ DisplayFormat2(long server, long partition, volintInfo *pntr)
        break;
     }
     t = pntr->creationDate;
-    fprintf(STDOUT, "creationDate\t%-9lu\t%s", 
+    fprintf(STDOUT, "creationDate\t%-9lu\t%s",
            afs_printable_uint32_lu(pntr->creationDate),
            ctime(&t));
 
     t = pntr->accessDate;
-    fprintf(STDOUT, "accessDate\t%-9lu\t%s", 
+    fprintf(STDOUT, "accessDate\t%-9lu\t%s",
            afs_printable_uint32_lu(pntr->accessDate),
            ctime(&t));
 
     t = pntr->updateDate;
-    fprintf(STDOUT, "updateDate\t%-9lu\t%s", 
+    fprintf(STDOUT, "updateDate\t%-9lu\t%s",
            afs_printable_uint32_lu(pntr->updateDate),
            ctime(&t));
 
     t = pntr->backupDate;
-    fprintf(STDOUT, "backupDate\t%-9lu\t%s", 
+    fprintf(STDOUT, "backupDate\t%-9lu\t%s",
            afs_printable_uint32_lu(pntr->backupDate),
            ctime(&t));
 
     t = pntr->copyDate;
-    fprintf(STDOUT, "copyDate\t%-9lu\t%s", 
+    fprintf(STDOUT, "copyDate\t%-9lu\t%s",
            afs_printable_uint32_lu(pntr->copyDate),
            ctime(&t));
 
-    fprintf(STDOUT, "flags\t\t%#lx\t(Optional)\n", 
+    fprintf(STDOUT, "flags\t\t%#lx\t(Optional)\n",
            afs_printable_uint32_lu(pntr->flags));
     fprintf(STDOUT, "diskused\t%u\n", pntr->size);
     fprintf(STDOUT, "maxquota\t%u\n", pntr->maxquota);
-    fprintf(STDOUT, "minquota\t%lu\t(Optional)\n", 
+    fprintf(STDOUT, "minquota\t%lu\t(Optional)\n",
            afs_printable_uint32_lu(pntr->spare0));
     fprintf(STDOUT, "filecount\t%u\n", pntr->filecount);
     fprintf(STDOUT, "dayUse\t\t%u\n", pntr->dayUse);
     fprintf(STDOUT, "weekUse\t\t%lu\t(Optional)\n",
            afs_printable_uint32_lu(pntr->spare1));
-    fprintf(STDOUT, "spare2\t\t%lu\t(Optional)\n", 
+    fprintf(STDOUT, "spare2\t\t%lu\t(Optional)\n",
            afs_printable_uint32_lu(pntr->spare2));
-    fprintf(STDOUT, "spare3\t\t%lu\t(Optional)\n", 
+    fprintf(STDOUT, "spare3\t\t%lu\t(Optional)\n",
            afs_printable_uint32_lu(pntr->spare3));
     return;
 }
@@ -1374,10 +1379,10 @@ XDisplayVolumes2(afs_uint32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
 }                              /*XDisplayVolumes2 */
 
 
-/* set <server> and <part> to the correct values depending on 
+/* set <server> and <part> to the correct values depending on
  * <voltype> and <entry> */
 static void
-GetServerAndPart(struct nvldbentry *entry, int voltype, afs_uint32 *server,
+GetServerAndPart(struct uvldbentry *entry, int voltype, afs_uint32 *server,
                 afs_int32 *part, int *previdx)
 {
     int i, istart, vtype;
@@ -1395,9 +1400,10 @@ GetServerAndPart(struct nvldbentry *entry, int voltype, afs_uint32 *server,
        istart = ((*previdx < 0) ? 0 : *previdx + 1);
     }
 
-    for (i = istart; i < entry->nServers; i++) {
+    for (i = istart; i < entry->nServers && i < NMAXNSERVERS; i++) {
        if (entry->serverFlags[i] & vtype) {
-           *server = entry->serverNumber[i];
+            /* *uuid = entry->serverNumber[i]; */
+           *server = entry->serverUnique[i];
            *part = entry->serverPartition[i];
            *previdx = i;
            return;
@@ -1436,9 +1442,9 @@ PrintLocked(afs_int32 aflags)
 }
 
 static void
-PostVolumeStats(struct nvldbentry *entry)
+PostVolumeStats(struct uvldbentry *entry)
 {
-    SubEnumerateEntry(entry);
+    SubEnumerateEntryU(entry);
     /* Check for VLOP_ALLOPERS */
     PrintLocked(entry->flags);
     return;
@@ -1468,7 +1474,7 @@ PostVolumeStats(struct nvldbentry *entry)
  *------------------------------------------------------------------------*/
 
 static void
-XVolumeStats(volintXInfo *a_xInfoP, struct nvldbentry *a_entryP,
+XVolumeStats(volintXInfo *a_xInfoP, struct uvldbentry *a_entryP,
             afs_int32 a_srvID, afs_int32 a_partID, int a_volType)
 {                              /*XVolumeStats */
 
@@ -1488,7 +1494,7 @@ XVolumeStats(volintXInfo *a_xInfoP, struct nvldbentry *a_entryP,
 }                              /*XVolumeStats */
 
 static void
-VolumeStats_int(volintInfo *pntr, struct nvldbentry *entry, afs_uint32 server,
+VolumeStats_int(volintInfo *pntr, struct uvldbentry *entry, afs_uint32 server,
             afs_int32 part, int voltype)
 {
     int totalOK, totalNotOK, totalBusy;
@@ -1500,14 +1506,14 @@ VolumeStats_int(volintInfo *pntr, struct nvldbentry *entry, afs_uint32 server,
 
 /* command to forcibly remove a volume */
 static int
-NukeVolume(register struct cmd_syndesc *as)
+NukeVolume(struct cmd_syndesc *as)
 {
-    register afs_int32 code;
+    afs_int32 code;
     afs_uint32 volID;
     afs_int32  err;
     afs_int32 partID;
     afs_uint32 server;
-    register char *tp;
+    char *tp;
 
     server = GetServer(tp = as->parms[0].items->data);
     if (!server) {
@@ -1566,9 +1572,9 @@ NukeVolume(register struct cmd_syndesc *as)
  *------------------------------------------------------------------------
  */
 static int
-ExamineVolume(register struct cmd_syndesc *as, void *arock)
+ExamineVolume(struct cmd_syndesc *as, void *arock)
 {
-    struct nvldbentry entry;
+    struct uvldbentry entry;
     afs_int32 vcode = 0;
     volintInfo *pntr = (volintInfo *) 0;
     volintXInfo *xInfoP = (volintXInfo *) 0;
@@ -1597,7 +1603,7 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
                (unsigned long)volid);
        fflush(STDOUT);
     }
-    vcode = VLDB_GetEntryByID(volid, -1, &entry);
+    vcode = VLDB_GetEntryByIDU(volid, -1, &entry);
     if (vcode) {
        fprintf(STDERR,
                "Could not fetch the entry for volume number %lu from VLDB \n",
@@ -1606,7 +1612,7 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
     }
     if (verbose)
        fprintf(STDOUT, "done\n");
-    MapHostToNetwork(&entry);
+    MapHostToNetworkU(&entry);
 
     if (entry.volumeId[RWVOL] == volid)
        voltype = RWVOL;
@@ -1668,7 +1674,7 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
                XVolumeStats(xInfoP, &entry, aserver, apart, voltype);
            else if (as->parms[2].items) {
                DisplayFormat2(aserver, apart, pntr);
-               EnumerateEntry(&entry);
+               EnumerateEntryU(&entry);
                isSubEnum = 1;
            } else
                VolumeStats_int(pntr, &entry, aserver, apart, voltype);
@@ -1719,9 +1725,9 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
  *------------------------------------------------------------------------
  */
 static int
-SetFields(register struct cmd_syndesc *as, void *arock)
+SetFields(struct cmd_syndesc *as, void *arock)
 {
-    struct nvldbentry entry;
+    struct uvldbentry entry;
     volintInfo info;
     afs_uint32 volid;
     afs_int32 code, err;
@@ -1739,14 +1745,14 @@ SetFields(register struct cmd_syndesc *as, void *arock)
        return -1;
     }
 
-    code = VLDB_GetEntryByID(volid, RWVOL, &entry);
+    code = VLDB_GetEntryByIDU(volid, RWVOL, &entry);
     if (code) {
        fprintf(STDERR,
                "Could not fetch the entry for volume number %lu from VLDB \n",
                (unsigned long)volid);
        return (code);
     }
-    MapHostToNetwork(&entry);
+    MapHostToNetworkU(&entry);
 
     GetServerAndPart(&entry, RWVOL, &aserver, &apart, &previdx);
     if (previdx == -1) {
@@ -1803,7 +1809,7 @@ SetFields(register struct cmd_syndesc *as, void *arock)
  *------------------------------------------------------------------------
  */
 static int
-volOnline(register struct cmd_syndesc *as, void *arock)
+volOnline(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 server;
     afs_int32 partition;
@@ -1864,7 +1870,7 @@ volOnline(register struct cmd_syndesc *as, void *arock)
  *------------------------------------------------------------------------
  */
 static int
-volOffline(register struct cmd_syndesc *as, void *arock)
+volOffline(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 server;
     afs_int32 partition;
@@ -1916,14 +1922,14 @@ volOffline(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-CreateVolume(register struct cmd_syndesc *as, void *arock)
+CreateVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 pnum;
     char part[10];
     afs_uint32 volid = 0, rovolid = 0, bkvolid = 0;
     afs_uint32 *arovolid;
     afs_int32 code;
-    struct nvldbentry entry;
+    struct uvldbentry entry;
     afs_int32 vcode;
     afs_int32 quota;
 
@@ -1968,7 +1974,7 @@ CreateVolume(register struct cmd_syndesc *as, void *arock)
                as->parms[2].items->data);
        return EINVAL;
     }
-    vcode = VLDB_GetEntryByName(as->parms[2].items->data, &entry);
+    vcode = VLDB_GetEntryByNameU(as->parms[2].items->data, &entry);
     if (!vcode) {
        fprintf(STDERR, "Volume %s already exists\n",
                as->parms[2].items->data);
@@ -2032,16 +2038,16 @@ CreateVolume(register struct cmd_syndesc *as, void *arock)
 
 #if 0
 static afs_int32
-DeleteAll(struct nvldbentry *entry)
+DeleteAll(struct uvldbentry *entry)
 {
     int i;
     afs_int32 error, code, curserver, curpart;
     afs_uint32 volid;
 
-    MapHostToNetwork(entry);
+    MapHostToNetworkU(entry);
     error = 0;
     for (i = 0; i < entry->nServers; i++) {
-       curserver = entry->serverNumber[i];
+       curserver = entry->serverUnique[i];
        curpart = entry->serverPartition[i];
        if (entry->serverFlags[i] & ITSROVOL) {
            volid = entry->volumeId[ROVOL];
@@ -2109,9 +2115,9 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
      * them in from the VLDB entry.
      */
     if ((partition == -1) || !server) {
-       struct nvldbentry entry;
+       struct uvldbentry entry;
 
-       code = VLDB_GetEntryByID(volid, -1, &entry);
+       code = VLDB_GetEntryByIDU(volid, -1, &entry);
        if (code) {
            fprintf(STDERR,
                    "Could not fetch the entry for volume %lu from VLDB\n",
@@ -2123,8 +2129,8 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
        if (((volid == entry.volumeId[RWVOL]) && (entry.flags & RW_EXISTS))
            || ((volid == entry.volumeId[BACKVOL])
                && (entry.flags & BACK_EXISTS))) {
-           idx = Lp_GetRwIndex(&entry);
-           if ((idx == -1) || (server && (server != entry.serverNumber[idx]))
+           idx = Lp_GetRwIndexU(&entry);
+           if ((idx == -1) || (server && (server != entry.serverUnique[idx]))
                || ((partition != -1)
                    && (partition != entry.serverPartition[idx]))) {
                fprintf(STDERR, "VLDB: Volume '%s' no match\n",
@@ -2137,7 +2143,7 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
                if (entry.serverFlags[j] != ITSROVOL)
                    continue;
 
-               if (((server == 0) || (server == entry.serverNumber[j]))
+               if (((server == 0) || (server == entry.serverUnique[j]))
                    && ((partition == -1)
                        || (partition == entry.serverPartition[j]))) {
                    if (idx != -1) {
@@ -2160,7 +2166,7 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
            return ENOENT;
        }
 
-       server = htonl(entry.serverNumber[idx]);
+       server = htonl(entry.serverUnique[idx]);
        partition = entry.serverPartition[idx];
     }
 
@@ -2179,7 +2185,7 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
 
 #define TESTM  0               /* set for move space tests, clear for production */
 static int
-MoveVolume(register struct cmd_syndesc *as, void *arock)
+MoveVolume(struct cmd_syndesc *as, void *arock)
 {
 
     afs_uint32 volid;
@@ -2308,13 +2314,13 @@ MoveVolume(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-CopyVolume(register struct cmd_syndesc *as, void *arock)
+CopyVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 volid;
     afs_uint32 fromserver, toserver;
     afs_int32 frompart, topart, code, err, flags;
     char fromPartName[10], toPartName[10], *tovolume;
-    struct nvldbentry entry;
+    struct uvldbentry entry;
     struct diskPartition64 partition;  /* for space check */
     volintInfo *p;
 
@@ -2359,7 +2365,7 @@ CopyVolume(register struct cmd_syndesc *as, void *arock)
                tovolume);
        return EINVAL;
     }
-    code = VLDB_GetEntryByName(tovolume, &entry);
+    code = VLDB_GetEntryByNameU(tovolume, &entry);
     if (!code) {
        fprintf(STDERR, "Volume %s already exists\n", tovolume);
        PrintDiagnostics("copy", code);
@@ -2456,7 +2462,7 @@ CopyVolume(register struct cmd_syndesc *as, void *arock)
 
 
 static int
-ShadowVolume(register struct cmd_syndesc *as, void *arock)
+ShadowVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 volid, tovolid;
     afs_uint32 fromserver, toserver;
@@ -2653,14 +2659,14 @@ ShadowVolume(register struct cmd_syndesc *as, void *arock)
 
 
 static int
-CloneVolume(register struct cmd_syndesc *as, void *arock)
+CloneVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 volid, cloneid;
     afs_uint32 server;
     afs_int32 part, voltype;
     char partName[10], *volname;
     afs_int32 code, err, flags;
-    struct nvldbentry entry;
+    struct uvldbentry entry;
 
     volid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
     if (volid == 0) {
@@ -2715,7 +2721,7 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
            return E2BIG;
        }
 #if 0
-       /* 
+       /*
         * In order that you be able to make clones of RO or BK, this
         * check must be omitted.
         */
@@ -2752,7 +2758,7 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
     if (as->parms[6].items) flags |= RV_RDONLY;
 
 
-    code = 
+    code =
        UV_CloneVolume(server, part, volid, cloneid, volname, flags);
 
     if (code) {
@@ -2768,17 +2774,17 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
 
 
 static int
-BackupVolume(register struct cmd_syndesc *as, void *arock)
+BackupVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_uint32 aserver;
     afs_int32 apart, vtype, code, err;
-    struct nvldbentry entry;
+    struct uvldbentry entry;
 
     afs_uint32 buvolid;
     afs_uint32 buserver;
     afs_int32 bupart, butype;
-    struct nvldbentry buentry;
+    struct uvldbentry buentry;
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
     if (avolid == 0) {
@@ -2840,10 +2846,10 @@ BackupVolume(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ReleaseVolume(register struct cmd_syndesc *as, void *arock)
+ReleaseVolume(struct cmd_syndesc *as, void *arock)
 {
 
-    struct nvldbentry entry;
+    struct uvldbentry entry;
     afs_uint32 avolid;
     afs_uint32 aserver;
     afs_int32 apart, vtype, code, err;
@@ -2887,13 +2893,13 @@ ReleaseVolume(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-DumpVolumeCmd(register struct cmd_syndesc *as, void *arock)
+DumpVolumeCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_uint32 aserver;
     afs_int32 apart, voltype, fromdate = 0, code, err, i, flags;
     char filename[MAXPATHLEN];
-    struct nvldbentry entry;
+    struct uvldbentry entry;
 
     rx_SetRxDeadTime(60 * 10);
     for (i = 0; i < MAXSERVERS; i++) {
@@ -2989,7 +2995,7 @@ retry_dump:
 #define TS_NEW 3
 
 static int
-RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
+RestoreVolumeCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid, aparentid;
     afs_uint32 aserver;
@@ -2998,12 +3004,9 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
     afs_int32 acreation = 0, alastupdate = 0;
     int restoreflags = 0;
     int readonly = 0, offline = 0, voltype = RWVOL;
-    char prompt;
     char afilename[MAXPATHLEN], avolname[VOLSER_MAXVOLNAME + 1], apartName[10];
     char volname[VOLSER_MAXVOLNAME + 1];
-    struct nvldbentry entry;
-
-    prompt = 'n';
+    struct uvldbentry entry;
 
     aparentid = 0;
     if (as->parms[4].items) {
@@ -3125,7 +3128,7 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
     /* Check if volume exists or not */
 
     vsu_ExtractName(volname, avolname);
-    vcode = VLDB_GetEntryByName(volname, &entry);
+    vcode = VLDB_GetEntryByNameU(volname, &entry);
     if (vcode) {               /* no volume - do a full restore */
        restoreflags = RV_FULLRST;
        if ((aoverwrite == INC) || (aoverwrite == ABORT))
@@ -3133,8 +3136,8 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
                    "Volume does not exist; Will perform a full restore\n");
     }
 
-    else if ((!readonly && Lp_GetRwIndex(&entry) == -1)        /* RW volume does not exist - do a full */
-            ||(readonly && !Lp_ROMatch(0, 0, &entry))) {       /* RO volume does not exist - do a full */
+    else if ((!readonly && Lp_GetRwIndexU(&entry) == -1)       /* RW volume does not exist - do a full */
+            ||(readonly && !Lp_ROMatchU(0, 0, &entry))) {      /* RO volume does not exist - do a full */
        restoreflags = RV_FULLRST;
        if ((aoverwrite == INC) || (aoverwrite == ABORT))
            fprintf(STDERR,
@@ -3153,7 +3156,7 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
     else {                     /* volume exists - do we do a full incremental or abort */
        afs_uint32 Oserver;
        afs_int32 Opart, Otype, vol_elsewhere = 0;
-       struct nvldbentry Oentry;
+       struct uvldbentry Oentry;
        int c, dc;
 
        if (avolid == 0) {
@@ -3266,7 +3269,7 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
     if (as->parms[10].items) {
        restoreflags |= RV_NODEL;
     }
-    
+
 
     code =
        UV_RestoreVolume2(aserver, apart, avolid, aparentid,
@@ -3287,7 +3290,7 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-LockReleaseCmd(register struct cmd_syndesc *as, void *arock)
+LockReleaseCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_int32 code, err;
@@ -3313,7 +3316,7 @@ LockReleaseCmd(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-AddSite(register struct cmd_syndesc *as, void *arock)
+AddSite(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_uint32 aserver;
@@ -3376,7 +3379,7 @@ AddSite(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-RemoveSite(register struct cmd_syndesc *as, void *arock)
+RemoveSite(struct cmd_syndesc *as, void *arock)
 {
 
     afs_uint32 avolid;
@@ -3429,7 +3432,7 @@ RemoveSite(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ChangeLocation(register struct cmd_syndesc *as, void *arock)
+ChangeLocation(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_uint32 aserver;
@@ -3478,7 +3481,7 @@ ChangeLocation(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ListPartitions(register struct cmd_syndesc *as, void *arock)
+ListPartitions(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 aserver;
     afs_int32 code;
@@ -3637,7 +3640,7 @@ XCompareVolID(const void *a_obj1P, const void *a_obj2P)
  *------------------------------------------------------------------------*/
 
 static int
-ListVolumes(register struct cmd_syndesc *as, void *arock)
+ListVolumes(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart, int32list, fast;
     afs_uint32 aserver;
@@ -3792,7 +3795,7 @@ ListVolumes(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-SyncVldb(register struct cmd_syndesc *as, void *arock)
+SyncVldb(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 pnum = 0, code;  /* part name */
     char part[10];
@@ -3874,7 +3877,7 @@ SyncVldb(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-SyncServer(register struct cmd_syndesc *as, void *arock)
+SyncServer(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 pnum, code;      /* part name */
     char part[10];
@@ -3930,19 +3933,19 @@ SyncServer(register struct cmd_syndesc *as, void *arock)
 static int
 VolumeInfoCmd(char *name)
 {
-    struct nvldbentry entry;
+    struct uvldbentry entry;
     afs_int32 vcode;
 
     /* The vlserver will handle names with the .readonly
      * and .backup extension as well as volume ids.
      */
-    vcode = VLDB_GetEntryByName(name, &entry);
+    vcode = VLDB_GetEntryByNameU(name, &entry);
     if (vcode) {
        PrintError("", vcode);
        exit(1);
     }
-    MapHostToNetwork(&entry);
-    EnumerateEntry(&entry);
+    MapHostToNetworkU(&entry);
+    EnumerateEntryU(&entry);
 
     /* Defect #3027: grubby check to handle locked volume.
      * If VLOP_ALLOPERS is set, the entry is locked.
@@ -3954,9 +3957,9 @@ VolumeInfoCmd(char *name)
 }
 
 static int
-VolumeZap(register struct cmd_syndesc *as, void *arock)
+VolumeZap(struct cmd_syndesc *as, void *arock)
 {
-    struct nvldbentry entry;
+    struct uvldbentry entry;
     afs_uint32 volid, zapbackupid = 0, backupid = 0;
     afs_int32 code, server, part, err;
 
@@ -3999,7 +4002,7 @@ VolumeZap(register struct cmd_syndesc *as, void *arock)
                    as->parms[1].items->data);
        exit(1);
     }
-    code = VLDB_GetEntryByID(volid, -1, &entry);
+    code = VLDB_GetEntryByIDU(volid, -1, &entry);
     if (!code) {
        if (volid == entry.volumeId[RWVOL])
            backupid = entry.volumeId[BACKVOL];
@@ -4040,7 +4043,7 @@ VolumeZap(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-VolserStatus(register struct cmd_syndesc *as, void *arock)
+VolserStatus(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 server;
     afs_int32 code;
@@ -4121,10 +4124,12 @@ VolserStatus(register struct cmd_syndesc *as, void *arock)
        fprintf(STDOUT, "volume: %lu  partition: %s  procedure: %s\n",
                (unsigned long)pntr->volid, pname, pntr->lastProcName);
        if (pntr->callValid) {
-           fprintf(STDOUT,
-                   "packetRead: %lu  lastReceiveTime: %d  packetSend: %lu  lastSendTime: %d\n",
-                   (unsigned long)pntr->readNext, pntr->lastReceiveTime,
-                   (unsigned long)pntr->transmitNext, pntr->lastSendTime);
+           t = pntr->lastReceiveTime;
+           fprintf(STDOUT, "packetRead: %lu  lastReceiveTime: %s",
+                   (unsigned long)pntr->readNext, ctime(&t));
+           t = pntr->lastSendTime;
+           fprintf(STDOUT, "packetSend: %lu  lastSendTime: %s",
+                   (unsigned long)pntr->transmitNext, ctime(&t));
        }
        pntr++;
        fprintf(STDOUT, "--------------------------------------\n");
@@ -4136,18 +4141,18 @@ VolserStatus(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-RenameVolume(register struct cmd_syndesc *as, void *arock)
+RenameVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 code1, code2, code;
-    struct nvldbentry entry;
+    struct uvldbentry entry;
 
-    code1 = VLDB_GetEntryByName(as->parms[0].items->data, &entry);
+    code1 = VLDB_GetEntryByNameU(as->parms[0].items->data, &entry);
     if (code1) {
        fprintf(STDERR, "vos: Could not find entry for volume %s\n",
                as->parms[0].items->data);
        exit(1);
     }
-    code2 = VLDB_GetEntryByName(as->parms[1].items->data, &entry);
+    code2 = VLDB_GetEntryByNameU(as->parms[1].items->data, &entry);
     if ((!code1) && (!code2)) {        /*the newname already exists */
        fprintf(STDERR, "vos: volume %s already exists\n",
                as->parms[1].items->data);
@@ -4182,10 +4187,10 @@ RenameVolume(register struct cmd_syndesc *as, void *arock)
                as->parms[1].items->data);
        exit(1);
     }
-    MapHostToNetwork(&entry);
+    MapHostToNetworkU(&entry);
     code =
-       UV_RenameVolume(&entry, as->parms[0].items->data,
-                       as->parms[1].items->data);
+       UV_RenameVolumeU(&entry, as->parms[0].items->data,
+                        as->parms[1].items->data);
     if (code) {
        PrintDiagnostics("rename", code);
        exit(1);
@@ -4197,12 +4202,12 @@ RenameVolume(register struct cmd_syndesc *as, void *arock)
 
 int
 GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part, afs_int32 *voltype,
-              struct nvldbentry *rentry)
+              struct uvldbentry *rentry)
 {
     afs_int32 vcode;
     int i, index = -1;
 
-    vcode = VLDB_GetEntryByID(volid, -1, rentry);
+    vcode = VLDB_GetEntryByIDU(volid, -1, rentry);
     if (vcode) {
        fprintf(STDERR,
                "Could not fetch the entry for volume %lu from VLDB \n",
@@ -4210,7 +4215,7 @@ GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part, afs_int32 *
        PrintError("", vcode);
        return (vcode);
     }
-    MapHostToNetwork(rentry);
+    MapHostToNetworkU(rentry);
     if (volid == rentry->volumeId[ROVOL]) {
        *voltype = ROVOL;
        for (i = 0; i < rentry->nServers; i++) {
@@ -4225,12 +4230,12 @@ GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part, afs_int32 *
            return -1;
        }
 
-       *server = rentry->serverNumber[index];
+       *server = rentry->serverUnique[index];
        *part = rentry->serverPartition[index];
        return 0;
     }
 
-    index = Lp_GetRwIndex(rentry);
+    index = Lp_GetRwIndexU(rentry);
     if (index == -1) {
        fprintf(STDERR,
                "RW Volume is not found in VLDB entry for volume %lu\n",
@@ -4239,13 +4244,13 @@ GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part, afs_int32 *
     }
     if (volid == rentry->volumeId[RWVOL]) {
        *voltype = RWVOL;
-       *server = rentry->serverNumber[index];
+       *server = rentry->serverUnique[index];
        *part = rentry->serverPartition[index];
        return 0;
     }
     if (volid == rentry->volumeId[BACKVOL]) {
        *voltype = BACKVOL;
-       *server = rentry->serverNumber[index];
+       *server = rentry->serverUnique[index];
        *part = rentry->serverPartition[index];
        return 0;
     }
@@ -4256,14 +4261,14 @@ GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part, afs_int32 *
 }
 
 static int
-DeleteEntry(register struct cmd_syndesc *as, void *arock)
+DeleteEntry(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart = 0;
     afs_uint32 avolid;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
-    nbulkentries arrayEntries;
-    register struct nvldbentry *vllist;
+    ubulkentries arrayEntries;
+    struct uvldbentry *vllist;
     struct cmd_item *itp;
     afs_int32 nentries;
     int j;
@@ -4374,7 +4379,7 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
 
     /* Get all the VLDB entries on a server and/or partition */
     memset(&arrayEntries, 0, sizeof(arrayEntries));
-    vcode = VLDB_ListAttributes(&attributes, &nentries, &arrayEntries);
+    vcode = VLDB_ListAttributesU(&attributes, &nentries, &arrayEntries);
     if (vcode) {
        fprintf(STDERR, "Could not access the VLDB for attributes\n");
        PrintError("", vcode);
@@ -4383,7 +4388,7 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
 
     /* Process each entry */
     for (j = 0; j < nentries; j++) {
-       vllist = &arrayEntries.nbulkentries_val[j];
+       vllist = &arrayEntries.ubulkentries_val[j];
        if (seenprefix) {
            /* It only deletes the RW volumes */
            if (strncmp(vllist->name, prefix, strlen(prefix))) {
@@ -4425,8 +4430,8 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
     fprintf(STDOUT,
            "Total VLDB entries deleted: %lu; failed to delete: %lu\n",
            (unsigned long)totalBack, (unsigned long)totalFail);
-    if (arrayEntries.nbulkentries_val)
-       free(arrayEntries.nbulkentries_val);
+
+    xdr_free((xdrproc_t) xdr_ubulkentries, &arrayEntries);
     return 0;
 }
 
@@ -4434,24 +4439,24 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
 static int
 CompareVldbEntryByName(const void *p1, const void *p2)
 {
-    struct nvldbentry *arg1, *arg2;
+    struct uvldbentry *arg1, *arg2;
 
-    arg1 = (struct nvldbentry *)p1;
-    arg2 = (struct nvldbentry *)p2;
+    arg1 = (struct uvldbentry *)p1;
+    arg2 = (struct uvldbentry *)p2;
     return (strcmp(arg1->name, arg2->name));
 }
 
 /*
 static int CompareVldbEntry(char *p1, char *p2)
 {
-    struct nvldbentry *arg1,*arg2;
+    struct uvldbentry *arg1,*arg2;
     int i;
     int pos1, pos2;
     char comp1[100],comp2[100];
     char temp1[20],temp2[20];
 
-    arg1 = (struct nvldbentry *)p1;
-    arg2 = (struct nvldbentry *)p2;
+    arg1 = (struct uvldbentry *)p1;
+    arg2 = (struct uvldbentry *)p2;
     pos1 = -1;
     pos2 = -1;
 
@@ -4463,8 +4468,8 @@ static int CompareVldbEntry(char *p1, char *p2)
        pos1 = 0;
        pos2 = 0;
     }
-    sprintf(comp1,"%10u",arg1->serverNumber[pos1]);
-    sprintf(comp2,"%10u",arg2->serverNumber[pos2]);
+    sprintf(comp1,"%10u",arg1->serverUnique[pos1]);
+    sprintf(comp2,"%10u",arg2->serverUnique[pos2]);
     sprintf(temp1,"%10u",arg1->serverPartition[pos1]);
     sprintf(temp2,"%10u",arg2->serverPartition[pos2]);
     strcat(comp1,temp1);
@@ -4484,8 +4489,8 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
     afs_int32 code;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
-    nbulkentries arrayEntries;
-    struct nvldbentry *vllist, *tarray = 0, *ttarray;
+    ubulkentries arrayEntries;
+    struct uvldbentry *vllist, *tarray = 0, *ttarray;
     afs_int32 centries, nentries = 0;
     afs_int32 tarraysize = 0;
     afs_int32 parraysize;
@@ -4565,12 +4570,12 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
        nextindex = -1;
 
        vcode =
-           VLDB_ListAttributesN2(&attributes, 0, thisindex, &centries,
+           VLDB_ListAttributesN2U(&attributes, 0, thisindex, &centries,
                                  &arrayEntries, &nextindex);
        if (vcode == RXGEN_OPCODE) {
            /* Vlserver not running with ListAttributesN2. Fall back */
            vcode =
-               VLDB_ListAttributes(&attributes, &centries, &arrayEntries);
+               VLDB_ListAttributesU(&attributes, &centries, &arrayEntries);
            nextindex = -1;
        }
        if (vcode) {
@@ -4583,9 +4588,9 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
        /* We don't sort, so just print the entries now */
        if (!sort) {
            for (j = 0; j < centries; j++) {    /* process each entry */
-               vllist = &arrayEntries.nbulkentries_val[j];
-               MapHostToNetwork(vllist);
-               EnumerateEntry(vllist);
+               vllist = &arrayEntries.ubulkentries_val[j];
+               MapHostToNetworkU(vllist);
+               EnumerateEntryU(vllist);
 
                PrintLocked(vllist->flags);
            }
@@ -4596,15 +4601,20 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
         */
        else if (centries > 0) {
            if (!tarray) {
-               /* steal away the first bulk entries array */
-               tarray = (struct nvldbentry *)arrayEntries.nbulkentries_val;
-               tarraysize = centries * sizeof(struct nvldbentry);
-               arrayEntries.nbulkentries_val = 0;
+               /* malloc the first bulk entries array */
+                tarraysize = centries * sizeof(struct uvldbentry);
+                tarray = malloc(tarraysize);
+               if (!tarray) {
+                   fprintf(STDERR,
+                           "Could not allocate enough space for the VLDB entries\n");
+                   goto bypass;
+               }
+                memcpy((char*)tarray, arrayEntries.ubulkentries_val, tarraysize);
            } else {
                /* Grow the tarray to keep the extra entries */
-               parraysize = (centries * sizeof(struct nvldbentry));
+               parraysize = (centries * sizeof(struct uvldbentry));
                ttarray =
-                   (struct nvldbentry *)realloc(tarray,
+                   (struct uvldbentry *)realloc(tarray,
                                                 tarraysize + parraysize);
                if (!ttarray) {
                    fprintf(STDERR,
@@ -4615,25 +4625,22 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
 
                /* Copy them in */
                memcpy(((char *)tarray) + tarraysize,
-                      (char *)arrayEntries.nbulkentries_val, parraysize);
+                      (char *)arrayEntries.ubulkentries_val, parraysize);
                tarraysize += parraysize;
            }
        }
 
        /* Free the bulk array */
-       if (arrayEntries.nbulkentries_val) {
-           free(arrayEntries.nbulkentries_val);
-           arrayEntries.nbulkentries_val = 0;
-       }
+        xdr_free((xdrproc_t) xdr_ubulkentries, &arrayEntries);
     }
 
     /* Here is where we now sort all the entries and print them */
     if (sort && (nentries > 0)) {
-       qsort((char *)tarray, nentries, sizeof(struct nvldbentry),
+       qsort((char *)tarray, nentries, sizeof(struct uvldbentry),
              CompareVldbEntryByName);
        for (vllist = tarray, j = 0; j < nentries; j++, vllist++) {
-           MapHostToNetwork(vllist);
-           EnumerateEntry(vllist);
+           MapHostToNetworkU(vllist);
+           EnumerateEntryU(vllist);
 
            PrintLocked(vllist->flags);
        }
@@ -4648,7 +4655,7 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
 }
 
 static int
-BackSys(register struct cmd_syndesc *as, void *arock)
+BackSys(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_int32 apart = 0;
@@ -4656,8 +4663,8 @@ BackSys(register struct cmd_syndesc *as, void *arock)
     afs_int32 code, apart1;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
-    nbulkentries arrayEntries;
-    register struct nvldbentry *vllist;
+    ubulkentries arrayEntries;
+    struct uvldbentry *vllist;
     afs_int32 nentries;
     int j;
     char pname[10];
@@ -4762,7 +4769,7 @@ BackSys(register struct cmd_syndesc *as, void *arock)
     }
 
     memset(&arrayEntries, 0, sizeof(arrayEntries));    /* initialize to hint the stub to alloc space */
-    vcode = VLDB_ListAttributes(&attributes, &nentries, &arrayEntries);
+    vcode = VLDB_ListAttributesU(&attributes, &nentries, &arrayEntries);
     if (vcode) {
        fprintf(STDERR, "Could not access the VLDB for attributes\n");
        PrintError("", vcode);
@@ -4816,7 +4823,7 @@ BackSys(register struct cmd_syndesc *as, void *arock)
     }
 
     for (j = 0; j < nentries; j++) {   /* process each vldb entry */
-       vllist = &arrayEntries.nbulkentries_val[j];
+       vllist = &arrayEntries.ubulkentries_val[j];
 
        if (seenprefix) {
            for (ti = as->parms[0].items; ti; ti = ti->next) {
@@ -4928,7 +4935,7 @@ BackSys(register struct cmd_syndesc *as, void *arock)
        }
 
        avolid = vllist->volumeId[RWVOL];
-       MapHostToNetwork(vllist);
+       MapHostToNetworkU(vllist);
        GetServerAndPart(vllist, RWVOL, &aserver1, &apart1, &previdx);
        if (aserver1 == -1 || apart1 == -1) {
            fprintf(STDOUT, "could not backup %s, invalid VLDB entry\n",
@@ -4976,21 +4983,20 @@ BackSys(register struct cmd_syndesc *as, void *arock)
     fprintf(STDOUT, "Total volumes backed up: %lu; failed to backup: %lu\n",
            (unsigned long)totalBack, (unsigned long)totalFail);
     fflush(STDOUT);
-    if (arrayEntries.nbulkentries_val)
-       free(arrayEntries.nbulkentries_val);
+    xdr_free((xdrproc_t) xdr_ubulkentries, &arrayEntries);
     return 0;
 }
 
 static int
-UnlockVLDB(register struct cmd_syndesc *as, void *arock)
+UnlockVLDB(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart;
     afs_uint32 aserver = 0;
     afs_int32 code;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
-    nbulkentries arrayEntries;
-    register struct nvldbentry *vllist;
+    ubulkentries arrayEntries;
+    struct uvldbentry *vllist;
     afs_int32 nentries;
     int j;
     afs_uint32 volid;
@@ -5033,18 +5039,18 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
     attributes.flag = VLOP_ALLOPERS;
     attributes.Mask |= VLLIST_FLAG;
     memset(&arrayEntries, 0, sizeof(arrayEntries));    /*initialize to hint the stub  to alloc space */
-    vcode = VLDB_ListAttributes(&attributes, &nentries, &arrayEntries);
+    vcode = VLDB_ListAttributesU(&attributes, &nentries, &arrayEntries);
     if (vcode) {
        fprintf(STDERR, "Could not access the VLDB for attributes\n");
        PrintError("", vcode);
        exit(1);
     }
     for (j = 0; j < nentries; j++) {   /* process each entry */
-       vllist = &arrayEntries.nbulkentries_val[j];
+       vllist = &arrayEntries.ubulkentries_val[j];
        volid = vllist->volumeId[RWVOL];
        vcode =
            ubik_VL_ReleaseLock(cstruct, 0, volid, -1,
-                               LOCKREL_OPCODE | LOCKREL_AFSID | 
+                               LOCKREL_OPCODE | LOCKREL_AFSID |
                                LOCKREL_TIMESTAMP);
        if (vcode) {
            fprintf(STDERR, "Could not unlock entry for volume %s\n",
@@ -5078,16 +5084,15 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
        }
     }
 
-    if (arrayEntries.nbulkentries_val)
-       free(arrayEntries.nbulkentries_val);
+    xdr_free((xdrproc_t) xdr_ubulkentries, &arrayEntries);
     return 0;
 }
 
 static char *
 PrintInt64Size(afs_uint64 in)
 {
-    register afs_uint32 hi, lo;
-    register char * units;
+    afs_uint32 hi, lo;
+    char * units;
     static char output[16];
 
     SplitInt64(in,hi,lo);
@@ -5112,7 +5117,7 @@ PrintInt64Size(afs_uint64 in)
 }
 
 static int
-PartitionInfo(register struct cmd_syndesc *as, void *arock)
+PartitionInfo(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart;
     afs_uint32 aserver;
@@ -5188,14 +5193,14 @@ PartitionInfo(register struct cmd_syndesc *as, void *arock)
                PrintInt64Size(sumFree));
         fprintf(STDOUT,
                 "%s on %d partitions\n",
-                PrintInt64Size(sumStorage), 
+                PrintInt64Size(sumStorage),
                 sumPartitions);
     }
     return 0;
 }
 
 static int
-ChangeAddr(register struct cmd_syndesc *as, void *arock)
+ChangeAddr(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 ip1, ip2, vcode;
     int remove = 0;
@@ -5227,7 +5232,7 @@ ChangeAddr(register struct cmd_syndesc *as, void *arock)
        }
     } else {
        /* Play a trick here. If we are removing an address, ip1 will be -1
-        * and ip2 will be the original address. This switch prevents an 
+        * and ip2 will be the original address. This switch prevents an
         * older revision vlserver from removing the IP address.
         */
        remove = 1;
@@ -5288,7 +5293,7 @@ print_addrs(const bulkaddrs * addrs, afsUUID * m_uuid, int nentries,
     /* print out the list of all the server */
     addrp = (afs_uint32 *) addrs->bulkaddrs_val;
     for (i = 0; i < nentries; i++, addrp++) {
-       /* If it is a multihomed address, then we will need to 
+       /* If it is a multihomed address, then we will need to
         * get the addresses for this multihomed server from
         * the vlserver and print them.
         */
@@ -5354,7 +5359,7 @@ print_addrs(const bulkaddrs * addrs, afsUUID * m_uuid, int nentries,
 }
 
 static int
-ListAddrs(register struct cmd_syndesc *as, void *arock)
+ListAddrs(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 vcode, m_uniq=0;
     afs_int32 i, printuuid = 0;
@@ -5373,7 +5378,7 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
     if (as->parms[0].items) {
        /* -uuid */
         if (afsUUID_from_string(as->parms[0].items->data, &askuuid) < 0) {
-           fprintf(STDERR, "vos: invalid UUID '%s'\n", 
+           fprintf(STDERR, "vos: invalid UUID '%s'\n",
                    as->parms[0].items->data);
            exit(-1);
        }
@@ -5459,7 +5464,7 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
 
 
 static int
-SetAddrs(register struct cmd_syndesc *as, void *arock)
+SetAddrs(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 vcode;
     bulkaddrs m_addrs;
@@ -5523,7 +5528,7 @@ SetAddrs(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-LockEntry(register struct cmd_syndesc *as, void *arock)
+LockEntry(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_int32 vcode, err;
@@ -5550,13 +5555,13 @@ LockEntry(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ConvertRO(register struct cmd_syndesc *as, void *arock)
+ConvertRO(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 partition = -1;
     afs_uint32 volid;
     afs_uint32 server;
     afs_int32 code, i, same;
-    struct nvldbentry entry, storeEntry;
+    struct uvldbentry entry, storeEntry;
     afs_int32 vcode;
     afs_int32 rwindex = 0;
     afs_uint32 rwserver = 0;
@@ -5566,7 +5571,7 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
     afs_int32 ropartition = 0;
     int force = 0;
     struct rx_connection *aconn;
-    char c, dc;
+    int c, dc;
 
     server = GetServer(as->parms[0].items->data);
     if (!server) {
@@ -5601,7 +5606,7 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
     if (as->parms[3].items)
        force = 1;
 
-    vcode = VLDB_GetEntryByID(volid, -1, &entry);
+    vcode = VLDB_GetEntryByIDU(volid, -1, &entry);
     if (vcode) {
        fprintf(STDERR,
                "Could not fetch the entry for volume %lu from VLDB\n",
@@ -5615,15 +5620,15 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
     if (volid != entry.volumeId[ROVOL])
        volid = entry.volumeId[ROVOL];
 
-    MapHostToNetwork(&entry);
+    MapHostToNetworkU(&entry);
     for (i = 0; i < entry.nServers; i++) {
        if (entry.serverFlags[i] & ITSRWVOL) {
            rwindex = i;
-           rwserver = entry.serverNumber[i];
+           rwserver = entry.serverUnique[i];
            rwpartition = entry.serverPartition[i];
        }
        if (entry.serverFlags[i] & ITSROVOL) {
-           same = VLDB_IsSameAddrs(server, entry.serverNumber[i], &code);
+           same = VLDB_IsSameAddrs(server, entry.serverUnique[i], &code);
            if (code) {
                fprintf(STDERR,
                        "Failed to get info about server's %d address(es) from vlserver (err=%d); aborting call!\n",
@@ -5632,7 +5637,7 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
            }
            if (same) {
                roindex = i;
-               roserver = entry.serverNumber[i];
+               roserver = entry.serverUnique[i];
                ropartition = entry.serverPartition[i];
                break;
            }
@@ -5682,11 +5687,11 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
     if (rwserver) {
        (entry.nServers)--;
        if (rwindex != entry.nServers) {
-           entry.serverNumber[rwindex] = entry.serverNumber[entry.nServers];
+           entry.serverUnique[rwindex] = entry.serverUnique[entry.nServers];
            entry.serverPartition[rwindex] =
                entry.serverPartition[entry.nServers];
            entry.serverFlags[rwindex] = entry.serverFlags[entry.nServers];
-           entry.serverNumber[entry.nServers] = 0;
+           entry.serverUnique[entry.nServers] = 0;
            entry.serverPartition[entry.nServers] = 0;
            entry.serverFlags[entry.nServers] = 0;
        }
@@ -5698,9 +5703,9 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
                entry.flags |= RO_EXISTS;
        }
     }
-    MapNetworkToHost(&entry, &storeEntry);
+    MapNetworkToHostU(&entry, &storeEntry);
     code =
-       VLDB_ReplaceEntry(entry.volumeId[RWVOL], RWVOL, &storeEntry,
+       VLDB_ReplaceEntryU(entry.volumeId[RWVOL], RWVOL, &storeEntry,
                          (LOCKREL_OPCODE | LOCKREL_AFSID |
                           LOCKREL_TIMESTAMP));
     if (code) {
@@ -5716,12 +5721,12 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-Sizes(register struct cmd_syndesc *as, void *arock)
+Sizes(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_uint32 aserver;
     afs_int32 apart, voltype, fromdate = 0, code, err, i;
-    struct nvldbentry entry;
+    struct uvldbentry entry;
     volintSize vol_size;
 
     rx_SetRxDeadTime(60 * 10);
@@ -5780,11 +5785,7 @@ Sizes(register struct cmd_syndesc *as, void *arock)
     fprintf(STDOUT, "Volume: %s\n", as->parms[0].items->data);
 
     if (as->parms[3].items) {  /* do the dump estimate */
-#ifdef AFS_64BIT_ENV
        vol_size.dump_size = 0;
-#else
-   FillInt64(vol_size.dump_size,0, 1);
-#endif
        code = UV_GetSize(avolid, aserver, apart, fromdate, &vol_size);
        if (code) {
            PrintDiagnostics("size", code);
@@ -5802,7 +5803,7 @@ Sizes(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-EndTrans(register struct cmd_syndesc *as, void *arock)
+EndTrans(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 server;
     afs_int32 code, tid, rcode;
@@ -5876,9 +5877,9 @@ win32_enableCrypt(void)
 static int
 MyBeforeProc(struct cmd_syndesc *as, void *arock)
 {
-    register char *tcell;
-    register afs_int32 code;
-    register afs_int32 sauth;
+    char *tcell;
+    afs_int32 code;
+    afs_int32 sauth;
 
     /* Initialize the ubik_client connection */
     rx_SetRxDeadTime(90);
@@ -5928,14 +5929,14 @@ osi_audit(void)
 int
 main(int argc, char **argv)
 {
-    register afs_int32 code;
+    afs_int32 code;
 
-    register struct cmd_syndesc *ts;
+    struct cmd_syndesc *ts;
 
 #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.
      */