vos: volume lock not released on convertROtoRW error
[openafs.git] / src / volser / vos.c
index ec86de6..fb8a638 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
 #include <afsconfig.h>
 #include <afs/param.h>
 
+#include <roken.h>
+
 #ifdef IGNORE_SOME_GCC_WARNINGS
 # pragma GCC diagnostic warning "-Wimplicit-function-declaration"
 #endif
 
-#include <sys/types.h>
-#include <string.h>
-#ifdef HAVE_STDINT_H
-# include <stdint.h>
-#endif
 #ifdef AFS_NT40_ENV
-#include <fcntl.h>
-#include <io.h>
-#include <winsock2.h>
 #include <WINNT/afsreg.h>
-#else
-#include <sys/time.h>
-#include <sys/file.h>
-#include <netdb.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
 #endif
-#include <sys/stat.h>
+
 #ifdef AFS_AIX_ENV
 #include <sys/statfs.h>
 #endif
-#include <errno.h>
 
 #include <lock.h>
 #include <afs/stds.h>
@@ -60,9 +47,6 @@
 #include "dump.h"
 #include "lockdata.h"
 
-#ifdef AFS_AIX32_ENV
-#include <signal.h>
-#endif
 #include "volser_internal.h"
 #include "volser_prototypes.h"
 #include "vsutils_prototypes.h"
@@ -94,12 +78,14 @@ cmd_AddParm(ts, "-localauth",CMD_FLAG,CMD_OPTIONAL,"use server tickets");\
 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"); \
+cmd_AddParm(ts, "-config", CMD_SINGLE, CMD_OPTIONAL, "config location"); \
 
-#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;
-afs_uint32 tserver;
 extern struct ubik_client *cstruct;
 const char *confdir;
 
@@ -147,7 +133,7 @@ FileExists(char *filename)
 {
     usd_handle_t ufd;
     int code;
-    afs_hyper_t size;
+    afs_int64 size;
 
     code = usd_Open(filename, USD_OPEN_RDONLY, 0, &ufd);
     if (code) {
@@ -165,7 +151,7 @@ FileExists(char *filename)
 static int
 VolNameOK(char *name)
 {
-    int total;
+    size_t total;
 
 
     total = strlen(name);
@@ -182,7 +168,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 +211,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) {
@@ -236,7 +223,7 @@ GetServer(char *aname)
        memcpy(&addr, th->h_addr, sizeof(addr));
     }
 
-    if (addr == htonl(0x7f000001)) {   /* local host */
+    if (rx_IsLoopbackAddr(ntohl(addr))) {      /* local host */
        code = gethostname(hostname, MAXHOSTCHARS);
        if (code)
            return 0;
@@ -285,10 +272,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 +330,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_int64 currOffset;
+    afs_uint32 buffer;
+    afs_uint32 got;
 
     error = 0;
 
@@ -366,19 +353,15 @@ WriteData(struct rx_call *call, void *rock)
            goto wfail;
        }
        /* test if we have a valid dump */
-       hset64(filesize, 0, 0);
-       USD_SEEK(ufd, filesize, SEEK_END, &currOffset);
-       hset64(filesize, hgethi(currOffset), hgetlo(currOffset)-sizeof(afs_uint32));
-       USD_SEEK(ufd, filesize, SEEK_SET, &currOffset);
+       USD_SEEK(ufd, 0, SEEK_END, &currOffset);
+       USD_SEEK(ufd, currOffset - sizeof(afs_uint32), SEEK_SET, &currOffset);
        USD_READ(ufd, (char *)&buffer, sizeof(afs_uint32), &got);
        if ((got != sizeof(afs_uint32)) || (ntohl(buffer) != DUMPENDMAGIC)) {
            fprintf(STDERR, "Signature missing from end of file '%s'\n", filename);
            error = VOLSERBADOP;
            goto wfail;
        }
-       /* rewind, we are done */
-       hset64(filesize, 0, 0);
-       USD_SEEK(ufd, filesize, SEEK_SET, &currOffset);
+       USD_SEEK(ufd, 0, SEEK_SET, &currOffset);
     }
     code = SendFile(ufd, call, blksize);
     if (code) {
@@ -450,7 +433,7 @@ DumpFunction(struct rx_call *call, void *rock)
     char *filename = (char *)rock;
     usd_handle_t ufd;          /* default is to stdout */
     afs_int32 error = 0, code;
-    afs_hyper_t size;
+    afs_int64 size;
     long blksize;
     int ufdIsOpen = 0;
 
@@ -464,7 +447,7 @@ DumpFunction(struct rx_call *call, void *rock)
            usd_Open(filename, USD_OPEN_CREATE | USD_OPEN_RDWR, 0666, &ufd);
        if (code == 0) {
            ufdIsOpen = 1;
-           hzero(size);
+           size = 0;
            code = USD_IOCTL(ufd, USD_IOCTL_SETSIZE, &size);
        }
        if (code == 0) {
@@ -917,11 +900,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 +922,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 +1050,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 +1065,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 +1090,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,7 +1357,7 @@ 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,
@@ -1491,7 +1474,9 @@ static void
 VolumeStats_int(volintInfo *pntr, struct nvldbentry *entry, afs_uint32 server,
             afs_int32 part, int voltype)
 {
-    int totalOK, totalNotOK, totalBusy;
+    int totalOK = 0;
+    int totalNotOK = 0;
+    int totalBusy = 0;
 
     DisplayFormat(pntr, server, part, &totalOK, &totalNotOK, &totalBusy, 0, 1,
                  1);
@@ -1500,14 +1485,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,7 +1551,7 @@ 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;
     afs_int32 vcode = 0;
@@ -1579,7 +1564,7 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
     afs_int32 apart;
     int previdx = -1;
     int wantExtendedInfo;      /*Do we want extended vol info? */
-
+    int isSubEnum=0;           /* Keep track whether sub enumerate called. */
     wantExtendedInfo = (as->parms[1].items ? 1 : 0);   /* -extended */
 
     volid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);  /* -id */
@@ -1669,6 +1654,7 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
            else if (as->parms[2].items) {
                DisplayFormat2(aserver, apart, pntr);
                EnumerateEntry(&entry);
+               isSubEnum = 1;
            } else
                VolumeStats_int(pntr, &entry, aserver, apart, voltype);
 
@@ -1690,7 +1676,9 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
        fprintf(STDERR, "Dump only information from VLDB\n\n");
        fprintf(STDOUT, "%s \n", entry.name);   /* PostVolumeStats doesn't print name */
     }
-    PostVolumeStats(&entry);
+
+    if (!isSubEnum)
+       PostVolumeStats(&entry);
 
     return (error);
 }
@@ -1716,7 +1704,7 @@ 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;
     volintInfo info;
@@ -1800,7 +1788,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;
@@ -1861,7 +1849,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;
@@ -1895,7 +1883,7 @@ volOffline(register struct cmd_syndesc *as, void *arock)
 
     transflag = (as->parms[4].items ? ITBusy : ITOffline);
     sleeptime = (as->parms[3].items ? atol(as->parms[3].items->data) : 0);
-    transdone = (sleeptime ? 0 /*online */ : VTOutOfService);
+    transdone = ((sleeptime || as->parms[4].items) ? 0 /*online */ : VTOutOfService);
     if (as->parms[4].items && !as->parms[3].items) {
        fprintf(STDERR, "-sleep option must be used with -busy flag\n");
        return -1;
@@ -1913,7 +1901,7 @@ 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];
@@ -1923,6 +1911,7 @@ CreateVolume(register struct cmd_syndesc *as, void *arock)
     struct nvldbentry entry;
     afs_int32 vcode;
     afs_int32 quota;
+    afs_uint32 tserver;
 
     arovolid = &rovolid;
 
@@ -2176,7 +2165,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;
@@ -2305,7 +2294,7 @@ 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;
@@ -2453,7 +2442,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;
@@ -2650,7 +2639,7 @@ 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;
@@ -2712,7 +2701,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.
         */
@@ -2746,10 +2735,15 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
 
     flags = 0;
     if (as->parms[5].items) flags |= RV_OFFLINE;
+    if (as->parms[6].items && as->parms[7].items) {
+       fprintf(STDERR, "vos: cannot specify that a volume be -readwrite and -readonly\n");
+       return EINVAL;
+    }
     if (as->parms[6].items) flags |= RV_RDONLY;
+    if (as->parms[7].items) flags |= RV_RWONLY;
 
 
-    code = 
+    code =
        UV_CloneVolume(server, part, volid, cloneid, volname, flags);
 
     if (code) {
@@ -2765,7 +2759,7 @@ 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;
@@ -2837,7 +2831,7 @@ 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;
@@ -2845,9 +2839,12 @@ ReleaseVolume(register struct cmd_syndesc *as, void *arock)
     afs_uint32 aserver;
     afs_int32 apart, vtype, code, err;
     int force = 0;
+    int stayUp = 0;
 
     if (as->parms[1].items)
        force = 1;
+    if (as->parms[2].items)
+       stayUp = 1;
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
     if (avolid == 0) {
        if (err)
@@ -2873,7 +2870,8 @@ ReleaseVolume(register struct cmd_syndesc *as, void *arock)
        return E2BIG;
     }
 
-    code = UV_ReleaseVolume(avolid, aserver, apart, force);
+    code = UV_ReleaseVolume(avolid, aserver, apart, force, stayUp);
+
     if (code) {
        PrintDiagnostics("release", code);
        return code;
@@ -2884,7 +2882,7 @@ 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;
@@ -2898,8 +2896,8 @@ DumpVolumeCmd(register struct cmd_syndesc *as, void *arock)
        if (rxConn == 0)
            break;
        rx_SetConnDeadTime(rxConn, rx_connDeadTime);
-       if (rxConn->service)
-           rxConn->service->connDeadTime = rx_connDeadTime;
+       if (rx_ServiceOf(rxConn))
+           rx_ServiceOf(rxConn)->connDeadTime = rx_connDeadTime;
     }
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
@@ -2986,7 +2984,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;
@@ -2995,13 +2993,10 @@ 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';
-
     aparentid = 0;
     if (as->parms[4].items) {
        avolid = vsu_GetVolumeID(as->parms[4].items->data, cstruct, &err);
@@ -3263,7 +3258,7 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
     if (as->parms[10].items) {
        restoreflags |= RV_NODEL;
     }
-    
+
 
     code =
        UV_RestoreVolume2(aserver, apart, avolid, aparentid,
@@ -3284,7 +3279,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;
@@ -3310,7 +3305,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;
@@ -3373,7 +3368,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;
@@ -3426,7 +3421,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;
@@ -3465,7 +3460,7 @@ ChangeLocation(register struct cmd_syndesc *as, void *arock)
     }
     code = UV_ChangeLocation(aserver, apart, avolid);
     if (code) {
-       PrintDiagnostics("addsite", code);
+       PrintDiagnostics("changeloc", code);
        exit(1);
     }
     MapPartIdIntoName(apart, apartName);
@@ -3475,7 +3470,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;
@@ -3634,7 +3629,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;
@@ -3733,8 +3728,6 @@ ListVolumes(register struct cmd_syndesc *as, void *arock)
                                   &pntr, &count);
            if (code) {
                PrintDiagnostics("listvol", code);
-               if (pntr)
-                   free(pntr);
                exit(1);
            }
            if (wantExtendedInfo) {
@@ -3789,12 +3782,13 @@ 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];
     int flags = 0;
     char *volname = 0;
+    afs_uint32 tserver;
 
     tserver = 0;
     if (as->parms[0].items) {
@@ -3871,10 +3865,11 @@ 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];
+    afs_uint32 tserver;
 
     int flags = 0;
 
@@ -3951,7 +3946,7 @@ VolumeInfoCmd(char *name)
 }
 
 static int
-VolumeZap(register struct cmd_syndesc *as, void *arock)
+VolumeZap(struct cmd_syndesc *as, void *arock)
 {
     struct nvldbentry entry;
     afs_uint32 volid, zapbackupid = 0, backupid = 0;
@@ -4037,7 +4032,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;
@@ -4068,9 +4063,11 @@ VolserStatus(register struct cmd_syndesc *as, void *arock)
     for (i = 0; i < count; i++) {
        /*print out the relevant info */
        fprintf(STDOUT, "--------------------------------------\n");
-       t = pntr->time;
+       t = pntr->creationTime;
        fprintf(STDOUT, "transaction: %lu  created: %s",
                (unsigned long)pntr->tid, ctime(&t));
+       t = pntr->time;
+       fprintf(STDOUT, "lastActiveTime: %s", ctime(&t));
        if (pntr->returnCode) {
            fprintf(STDOUT, "returnCode: %lu\n",
                    (unsigned long)pntr->returnCode);
@@ -4116,10 +4113,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");
@@ -4131,7 +4130,7 @@ 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;
@@ -4251,14 +4250,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;
+    struct nvldbentry *vllist;
     struct cmd_item *itp;
     afs_int32 nentries;
     int j;
@@ -4282,7 +4281,8 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
                            itp->data);
                continue;
            }
-           if (as->parms[4].items) {   /* -noexecute */
+           if (as->parms[4].items || as->parms[5].items) {
+               /* -noexecute (hidden) or -dryrun */
                fprintf(STDOUT, "Would have deleted VLDB entry for %s \n",
                        itp->data);
                fflush(STDOUT);
@@ -4392,7 +4392,8 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
            }
        }
 
-       if (as->parms[4].items) {       /* -noexecute */
+       if (as->parms[4].items || as->parms[5].items) {
+           /* -noexecute (hidden) or -dryrun */
            fprintf(STDOUT, "Would have deleted VLDB entry for %s \n",
                    vllist->name);
            fflush(STDOUT);
@@ -4403,7 +4404,7 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
        avolid = vllist->volumeId[RWVOL];
        vcode = ubik_VL_DeleteEntry(cstruct, 0, avolid, RWVOL);
        if (vcode) {
-           fprintf(STDOUT, "Could not delete VDLB entry for  %s\n",
+           fprintf(STDOUT, "Could not delete VLDB entry for  %s\n",
                    vllist->name);
            totalFail++;
            PrintError("", vcode);
@@ -4420,8 +4421,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_nbulkentries, &arrayEntries);
     return 0;
 }
 
@@ -4591,10 +4592,15 @@ 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 nvldbentry);
+                tarray = malloc(tarraysize);
+               if (!tarray) {
+                   fprintf(STDERR,
+                           "Could not allocate enough space for the VLDB entries\n");
+                   goto bypass;
+               }
+                memcpy((char*)tarray, arrayEntries.nbulkentries_val, tarraysize);
            } else {
                /* Grow the tarray to keep the extra entries */
                parraysize = (centries * sizeof(struct nvldbentry));
@@ -4616,10 +4622,7 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
        }
 
        /* Free the bulk array */
-       if (arrayEntries.nbulkentries_val) {
-           free(arrayEntries.nbulkentries_val);
-           arrayEntries.nbulkentries_val = 0;
-       }
+        xdr_free((xdrproc_t) xdr_nbulkentries, &arrayEntries);
     }
 
     /* Here is where we now sort all the entries and print them */
@@ -4643,7 +4646,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;
@@ -4652,7 +4655,7 @@ BackSys(register struct cmd_syndesc *as, void *arock)
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
     nbulkentries arrayEntries;
-    register struct nvldbentry *vllist;
+    struct nvldbentry *vllist;
     afs_int32 nentries;
     int j;
     char pname[10];
@@ -4971,13 +4974,12 @@ 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_nbulkentries, &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;
@@ -4985,7 +4987,7 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
     nbulkentries arrayEntries;
-    register struct nvldbentry *vllist;
+    struct nvldbentry *vllist;
     afs_int32 nentries;
     int j;
     afs_uint32 volid;
@@ -5039,7 +5041,7 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
        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",
@@ -5073,16 +5075,15 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
        }
     }
 
-    if (arrayEntries.nbulkentries_val)
-       free(arrayEntries.nbulkentries_val);
+    xdr_free((xdrproc_t) xdr_nbulkentries, &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);
@@ -5107,7 +5108,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;
@@ -5183,14 +5184,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;
@@ -5222,7 +5223,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;
@@ -5265,14 +5266,8 @@ static void
 print_addrs(const bulkaddrs * addrs, afsUUID * m_uuid, int nentries,
            int print)
 {
-    afs_int32 vcode, m_uniq=0;
-    afs_int32 i, j;
+    int i;
     afs_uint32 *addrp;
-    bulkaddrs m_addrs;
-    ListAddrByAttributes m_attrs;
-    afs_int32 m_nentries;
-    afs_uint32 *m_addrp;
-    afs_int32 base, index;
     char buf[1024];
 
     if (print) {
@@ -5283,56 +5278,6 @@ 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 
-        * get the addresses for this multihomed server from
-        * the vlserver and print them.
-        */
-       if (((*addrp & 0xff000000) == 0xff000000) && ((*addrp) & 0xffff)) {
-           /* Get the list of multihomed fileservers */
-           base = (*addrp >> 16) & 0xff;
-           index = (*addrp) & 0xffff;
-
-           if ((base >= 0) && (base <= VL_MAX_ADDREXTBLKS) && (index >= 1)
-               && (index <= VL_MHSRV_PERBLK)) {
-               m_attrs.Mask = VLADDR_INDEX;
-               m_attrs.index = (base * VL_MHSRV_PERBLK) + index;
-               m_nentries = 0;
-               m_addrs.bulkaddrs_val = 0;
-               m_addrs.bulkaddrs_len = 0;
-               vcode =
-                   ubik_VL_GetAddrsU(cstruct, 0, &m_attrs, m_uuid,
-                                     &m_uniq, &m_nentries,
-                                     &m_addrs);
-               if (vcode) {
-                   fprintf(STDERR,
-                           "vos: could not list the multi-homed server addresses\n");
-                   PrintError("", vcode);
-               }
-
-               /* Print the list */
-               m_addrp = (afs_uint32 *) m_addrs.bulkaddrs_val;
-               for (j = 0; j < m_nentries; j++, m_addrp++) {
-                   *m_addrp = htonl(*m_addrp);
-                   if (noresolve) {
-                       char hoststr[16];
-                       printf("%s ", afs_inet_ntoa_r(*m_addrp, hoststr));
-                   } else {
-                       printf("%s ", hostutil_GetNameByINet(*m_addrp));
-                   }
-               }
-               if (j == 0) {
-                   printf("<unknown>\n");
-               } else {
-                   printf("\n");
-               }
-
-               continue;
-           }
-       }
-
-       /* Otherwise, it is a non-multihomed entry and contains
-        * the IP address of the server - print it.
-        */
        *addrp = htonl(*addrp);
        if (noresolve) {
            char hoststr[16];
@@ -5349,7 +5294,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;
@@ -5360,22 +5305,23 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
     afsUUID m_uuid, askuuid;
     afs_int32 m_nentries;
 
-    memset(&m_attrs, 0, sizeof(struct ListAddrByAttributes));
-    m_attrs.Mask = VLADDR_INDEX;
+    if (as->parms[0].items && as->parms[1].items) {
+        fprintf(STDERR, "vos: Can't use the -uuid and -host flags together\n");
+        exit(-1);
+    }
 
-    memset(&m_addrs, 0, sizeof(bulkaddrs));
+    memset(&m_attrs, 0, sizeof(struct ListAddrByAttributes));
     memset(&askuuid, 0, sizeof(afsUUID));
     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);
        }
        m_attrs.Mask = VLADDR_UUID;
        m_attrs.uuid = askuuid;
-    }
-    if (as->parms[1].items) {
+    } else if (as->parms[1].items) {
        /* -host */
        struct hostent *he;
        afs_uint32 saddr;
@@ -5388,13 +5334,17 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
        memcpy(&saddr, he->h_addr, 4);
        m_attrs.Mask = VLADDR_IPADDR;
        m_attrs.ipaddr = ntohl(saddr);
+    } else {
+        /* by index */
+        m_attrs.Mask = VLADDR_INDEX;
     }
+
     if (as->parms[2].items) {
        printuuid = 1;
     }
 
-    m_addrs.bulkaddrs_val = 0;
-    m_addrs.bulkaddrs_len = 0;
+    memset(&m_addrs, 0, sizeof(bulkaddrs));
+    memset(&vlcb, 0, sizeof(struct VLCallBack));
 
     vcode =
        ubik_VL_GetAddrs(cstruct, UBIK_CALL_NEW, 0, 0, &vlcb, &nentries,
@@ -5402,59 +5352,57 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
     if (vcode) {
        fprintf(STDERR, "vos: could not list the server addresses\n");
        PrintError("", vcode);
-       return (vcode);
-    }
-
-    m_nentries = 0;
-    m_addrs.bulkaddrs_val = 0;
-    m_addrs.bulkaddrs_len = 0;
-    i = 1;
-    while (1) {
-       m_attrs.index = i;
-
-       vcode =
-           ubik_VL_GetAddrsU(cstruct, UBIK_CALL_NEW, &m_attrs, &m_uuid,
-                             &m_uniq, &m_nentries, &m_addrs);
-
-       if (vcode == VL_NOENT) {
-           if (m_attrs.Mask == VLADDR_UUID) {
-               fprintf(STDERR, "vos: no entry for UUID '%s' found in VLDB\n",
-                       as->parms[0].items->data);
-               exit(-1);
-           } else if (m_attrs.Mask == VLADDR_IPADDR) {
-               fprintf(STDERR, "vos: no entry for host '%s' [0x%08x] found in VLDB\n",
-                       as->parms[1].items->data, m_attrs.ipaddr);
-               exit(-1);
-           } else {
-               i++;
-               nentries++;
-               continue;
-           }
-       }
-
-       if (vcode == VL_INDEXERANGE) {
-           break;
-       }
-
-       if (vcode) {
-           fprintf(STDERR, "vos: could not list the server addresses\n");
-           PrintError("", vcode);
-           return (vcode);
-       }
-
-       print_addrs(&m_addrs, &m_uuid, m_nentries, printuuid);
-       i++;
+       goto out;
+    }
+
+    for (i = 1, m_nentries = 0; nentries; i++) {
+        m_attrs.index = i;
+
+        xdr_free((xdrproc_t)xdr_bulkaddrs, &m_addrs); /* reset addr list */
+        vcode =
+            ubik_VL_GetAddrsU(cstruct, UBIK_CALL_NEW, &m_attrs, &m_uuid,
+                              &m_uniq, &m_nentries, &m_addrs);
+        switch (vcode) {
+        case 0: /* success */
+            print_addrs(&m_addrs, &m_uuid, m_nentries, printuuid);
+            nentries--;
+            break;
+
+        case VL_NOENT:
+            if (m_attrs.Mask == VLADDR_UUID) {
+                fprintf(STDERR, "vos: no entry for UUID '%s' found in VLDB\n",
+                        as->parms[0].items->data);
+                exit(-1);
+            } else if (m_attrs.Mask == VLADDR_IPADDR) {
+                fprintf(STDERR, "vos: no entry for host '%s' [0x%08x] found in VLDB\n",
+                        as->parms[1].items->data, m_attrs.ipaddr);
+                exit(-1);
+            }
+            continue;
+
+        case VL_INDEXERANGE:
+            vcode = 0; /* not an error, just means we're done */
+            goto out;
+
+        default: /* any other error */
+            fprintf(STDERR, "vos: could not list the server addresses\n");
+            PrintError("", vcode);
+            goto out;
+        }
 
-       if ((as->parms[1].items) || (as->parms[0].items) || (i > nentries))
-           break;
+        /* if -uuid or -host, only list one response */
+        if (as->parms[1].items || as->parms[0].items)
+            break;
     }
 
-    return 0;
+out:
+    xdr_free((xdrproc_t)xdr_bulkaddrs, &m_addrs);
+    return vcode;
 }
 
 
 static int
-SetAddrs(register struct cmd_syndesc *as, void *arock)
+SetAddrs(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 vcode;
     bulkaddrs m_addrs;
@@ -5502,13 +5450,13 @@ SetAddrs(register struct cmd_syndesc *as, void *arock)
     if (vcode) {
        if (vcode == VL_MULTIPADDR) {
            fprintf(STDERR, "vos: VL_RegisterAddrs rpc failed; The IP address exists on a different server; repair it\n");
-           PrintError("", vcode);
-           return vcode;
        } else if (vcode == RXGEN_OPCODE) {
            fprintf(STDERR, "vlserver doesn't support VL_RegisterAddrs rpc; ignored\n");
-           PrintError("", vcode);
-           return vcode;
+       } else {
+           fprintf(STDERR, "vos: VL_RegisterAddrs rpc failed\n");
        }
+       PrintError("", vcode);
+       return vcode;
     }
     if (verbose) {
        fprintf(STDOUT, "vos: Changed UUID with addresses:\n");
@@ -5518,7 +5466,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;
@@ -5545,7 +5493,7 @@ 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;
@@ -5561,7 +5509,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) {
@@ -5590,7 +5538,7 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
            PrintError("", code);
        else
            fprintf(STDERR, "Unknown volume ID or name '%s'\n",
-                   as->parms[0].items->data);
+                   as->parms[2].items->data);
        return -1;
     }
     if (as->parms[3].items)
@@ -5601,7 +5549,7 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
        fprintf(STDERR,
                "Could not fetch the entry for volume %lu from VLDB\n",
                (unsigned long)volid);
-       PrintError("convertROtoRW", code);
+       PrintError("convertROtoRW ", vcode);
        return vcode;
     }
 
@@ -5616,8 +5564,9 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
            rwindex = i;
            rwserver = entry.serverNumber[i];
            rwpartition = entry.serverPartition[i];
-       }
-       if (entry.serverFlags[i] & ITSROVOL) {
+           if (roserver)
+               break;
+       } else if ((entry.serverFlags[i] & ITSROVOL) && !roserver) {
            same = VLDB_IsSameAddrs(server, entry.serverNumber[i], &code);
            if (code) {
                fprintf(STDERR,
@@ -5629,7 +5578,8 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
                roindex = i;
                roserver = entry.serverNumber[i];
                ropartition = entry.serverPartition[i];
-               break;
+               if (rwserver)
+                    break;
            }
        }
     }
@@ -5662,6 +5612,13 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
     vcode =
        ubik_VL_SetLock(cstruct, 0, entry.volumeId[RWVOL], RWVOL,
                  VLOP_MOVE);
+    if (vcode) {
+       fprintf(STDERR,
+               "Unable to lock volume %lu, code %d\n",
+               (unsigned long)entry.volumeId[RWVOL],vcode);
+       PrintError("", vcode);
+       return -1;
+    }
     aconn = UV_Bind(server, AFSCONF_VOLUMEPORT);
     code = AFSVolConvertROtoRWvolume(aconn, partition, volid);
     if (code) {
@@ -5669,7 +5626,7 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
                "Converting RO volume %lu to RW volume failed with code %d\n",
                (unsigned long)volid, code);
        PrintError("convertROtoRW ", code);
-       return -1;
+       goto error_exit;
     }
     entry.serverFlags[roindex] = ITSRWVOL;
     entry.flags |= RW_EXISTS;
@@ -5703,15 +5660,20 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
                "Warning: volume converted, but vldb update failed with code %d!\n",
                code);
     }
+
+  error_exit:
     vcode = UV_LockRelease(entry.volumeId[RWVOL]);
     if (vcode) {
-       PrintDiagnostics("unlock", vcode);
+       fprintf(STDERR,
+               "Unable to unlock volume %lu, code %d\n",
+               (unsigned long)entry.volumeId[RWVOL],vcode);
+       PrintError("", vcode);
     }
     return code;
 }
 
 static int
-Sizes(register struct cmd_syndesc *as, void *arock)
+Sizes(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_uint32 aserver;
@@ -5725,8 +5687,8 @@ Sizes(register struct cmd_syndesc *as, void *arock)
        if (rxConn == 0)
            break;
        rx_SetConnDeadTime(rxConn, rx_connDeadTime);
-       if (rxConn->service)
-           rxConn->service->connDeadTime = rx_connDeadTime;
+       if (rx_ServiceOf(rxConn))
+           rx_ServiceOf(rxConn)->connDeadTime = rx_connDeadTime;
     }
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
@@ -5775,11 +5737,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);
@@ -5797,7 +5755,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;
@@ -5871,29 +5829,39 @@ 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;
+    int secFlags;
 
     /* Initialize the ubik_client connection */
     rx_SetRxDeadTime(90);
-    cstruct = (struct ubik_client *)0;
+    cstruct = NULL;
+    secFlags = AFSCONF_SECOPTS_FALLBACK_NULL;
 
-    sauth = 0;
     tcell = NULL;
     if (as->parms[12].items)   /* if -cell specified */
        tcell = as->parms[12].items->data;
-    if (as->parms[14].items)   /* -serverauth specified */
-       sauth = 1;
+
+    if (as->parms[13].items)
+       secFlags |= AFSCONF_SECOPTS_NOAUTH;
+
+    if (as->parms[14].items) { /* -localauth specified */
+       secFlags |= AFSCONF_SECOPTS_LOCALAUTH;
+       confdir = AFSDIR_SERVER_ETC_DIRPATH;
+    }
+
     if (as->parms[16].items     /* -encrypt specified */
 #ifdef AFS_NT40_ENV
         || win32_enableCrypt()
 #endif /* AFS_NT40_ENV */
          )
-       vsu_SetCrypt(1);
-    if ((code =
-        vsu_ClientInit((as->parms[13].items != 0), confdir, tcell, sauth,
-                       &cstruct, UV_SetSecurity))) {
+       secFlags |= AFSCONF_SECOPTS_ALWAYSENCRYPT;
+
+    if (as->parms[18].items)   /* -config flag set */
+       confdir = as->parms[18].items->data;
+
+    if ((code = vsu_ClientInit(confdir, tcell, secFlags, UV_SetSecurity,
+                              &cstruct))) {
        fprintf(STDERR, "could not initialize VLDB library (code=%lu) \n",
                (unsigned long)code);
        exit(1);
@@ -5907,6 +5875,7 @@ MyBeforeProc(struct cmd_syndesc *as, void *arock)
        noresolve = 1;
     else
        noresolve = 0;
+
     return 0;
 }
 
@@ -5923,14 +5892,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.
      */
@@ -6039,12 +6008,16 @@ main(int argc, char **argv)
                "leave clone volume offline");
     cmd_AddParm(ts, "-readonly", CMD_FLAG, CMD_OPTIONAL,
                "make clone volume read-only, not readwrite");
+    cmd_AddParm(ts, "-readwrite", CMD_FLAG, CMD_OPTIONAL,
+               "make clone volume readwrite, not read-only");
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("release", ReleaseVolume, NULL, "release a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     cmd_AddParm(ts, "-force", CMD_FLAG, CMD_OPTIONAL,
                "force a complete release");
+    cmd_AddParm(ts, "-stayonline", CMD_FLAG, CMD_OPTIONAL,
+               "release to cloned temp vol, then clone back to repsite RO");
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("dump", DumpVolumeCmd, NULL, "dump a volume");
@@ -6134,14 +6107,14 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
     cmd_AddParm(ts, "-volume", CMD_SINGLE, CMD_OPTIONAL, "volume name or ID");
-    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "report without updating");
+    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "list what would be done, don't do it");
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("syncserv", SyncServer, NULL,
                          "synchronize server with VLDB");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
-    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "report without updating");
+    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "list what would be done, don't do it");
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("examine", ExamineVolume, NULL,
@@ -6218,7 +6191,7 @@ main(int argc, char **argv)
                "exclude common prefix volumes");
     cmd_AddParm(ts, "-xprefix", CMD_LIST, CMD_OPTIONAL,
                "negative prefix on volume(s)");
-    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "no action");
+    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "list what would be done, don't do it");
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("delentry", DeleteEntry, NULL,
@@ -6228,8 +6201,9 @@ main(int argc, char **argv)
                "prefix of the volume whose VLDB entry is to be deleted");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
-    cmd_AddParm(ts, "-noexecute", CMD_FLAG, CMD_OPTIONAL,
-               "no execute");
+    cmd_AddParm(ts, "-noexecute", CMD_FLAG, CMD_OPTIONAL|CMD_HIDDEN, "");
+    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL,
+               "list what would be done, don't do it");
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("partinfo", PartitionInfo, NULL,