vos: Return true when GetServerAndPart finds a site
[openafs.git] / src / volser / vos.c
index 8b08d37..075ca07 100644 (file)
@@ -80,6 +80,7 @@ enum {
     COMMONPARM_OFFSET_ENCRYPT   = 29,
     COMMONPARM_OFFSET_NORESOLVE = 30,
     COMMONPARM_OFFSET_CONFIG    = 31,
+    COMMONPARM_OFFSET_RXGK      = 32,
 };
 
 #define COMMONPARMS \
@@ -98,6 +99,8 @@ cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_NORESOLVE, \
     "-noresolve", CMD_FLAG, CMD_OPTIONAL, "don't resolve addresses"); \
 cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_CONFIG, \
     "-config", CMD_SINGLE, CMD_OPTIONAL, "config location"); \
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_RXGK, \
+    "-rxgk", CMD_SINGLE, CMD_OPTIONAL, "rxgk security level to use"); \
 
 #define ERROR_EXIT(code) do { \
     error = (code); \
@@ -234,6 +237,7 @@ GetServer(char *aname)
     afs_uint32 addr; /* in network byte order */
     afs_int32 code;
     char hostname[MAXHOSTCHARS];
+    afs_uint32 **addr_list;
     int i;
 
     addr = GetServerNoresolve(aname);
@@ -245,10 +249,11 @@ GetServer(char *aname)
     }
 
     th = gethostbyname(aname);
-    if (th != NULL) {
-       for (i=0; i < th->h_length; i++) {
-           if (!rx_IsLoopbackAddr(ntohl(*(afs_uint32 *)th->h_addr_list[i]))) {
-               memcpy(&addr, th->h_addr_list[i], sizeof(addr));
+    if (th != NULL && th->h_addrtype == AF_INET) {
+       addr_list = (afs_uint32 **)th->h_addr_list;
+       for(i = 0; addr_list[i] != NULL; i++) {
+           if (!rx_IsLoopbackAddr(ntohl(*addr_list[i]))) {
+               memcpy(&addr, addr_list[i], sizeof(addr));
                return addr;
            }
        }
@@ -263,10 +268,11 @@ GetServer(char *aname)
        code = gethostname(hostname, MAXHOSTCHARS);
        if (code == 0) {
            th = gethostbyname(hostname);
-           if (th != NULL) {
-               for (i=0; i < th->h_length; i++) {
-                   if (!rx_IsLoopbackAddr(ntohl(*(afs_uint32 *)th->h_addr_list[i]))) {
-                       memcpy(&addr, th->h_addr_list[i], sizeof(addr));
+           if (th != NULL && th->h_addrtype == AF_INET) {
+               addr_list = (afs_uint32 **)th->h_addr_list;
+               for (i=0; addr_list[i] != NULL; i++) {
+                   if (!rx_IsLoopbackAddr(ntohl(*addr_list[i]))) {
+                       memcpy(&addr, addr_list[i], sizeof(addr));
                        return addr;
                    }
                }
@@ -1397,25 +1403,32 @@ XDisplayVolumes2(afs_uint32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
 
 }                              /*XDisplayVolumes2 */
 
-
-/* set <server> and <part> to the correct values depending on
- * <voltype> and <entry> */
-static void
+/**
+ * Retrieve the sites (i.e., server id, partition id) from a vldb entry for a
+ * given volume type.  May be called repeatedly to get each read-only site.
+ *
+ * @param[in] entry       vldb entry from a previous vldb query
+ * @param[in] voltype     type of volume of interest (rw, ro, bk)
+ * @param[out] server     vldb entry server number for voltype
+ * @param[out] part       vldb part id for voltype
+ * @param[inout] previdx  cursor; should be initialized to -1
+ *                        before first call on a given entry.
+ *
+ * @returns true when a volume site has been found
+ */
+static int
 GetServerAndPart(struct nvldbentry *entry, int voltype, afs_uint32 *server,
                 afs_int32 *part, int *previdx)
 {
     int i, istart, vtype;
 
-    *server = -1;
-    *part = -1;
-
-    /* Doesn't check for non-existance of backup volume */
+    /* Doesn't check for non-existence of backup volume */
     if ((voltype == RWVOL) || (voltype == BACKVOL)) {
-       vtype = ITSRWVOL;
-       istart = 0;             /* seach the entire entry */
+       vtype = VLSF_RWVOL;
+       istart = 0;             /* search the entire entry */
     } else {
-       vtype = ITSROVOL;
-       /* Seach from beginning of entry or pick up where we left off */
+       vtype = VLSF_ROVOL;
+       /* Search from beginning of entry or pick up where we left off */
        istart = ((*previdx < 0) ? 0 : *previdx + 1);
     }
 
@@ -1424,13 +1437,13 @@ GetServerAndPart(struct nvldbentry *entry, int voltype, afs_uint32 *server,
            *server = entry->serverNumber[i];
            *part = entry->serverPartition[i];
            *previdx = i;
-           return;
+           return 1;
        }
     }
 
-    /* Didn't find any, return -1 */
+    /* Didn't find any more, reset index. */
     *previdx = -1;
-    return;
+    return 0;
 }
 
 static void
@@ -1646,17 +1659,14 @@ ExamineVolume(struct cmd_syndesc *as, void *arock)
         * It its a BK vol, get the RW entry (even if VLDB may say the BK doen't exist).
         * If its a RO vol, get the next RO entry.
         */
-       GetServerAndPart(&entry, ((voltype == ROVOL) ? ROVOL : RWVOL),
-                        &aserver, &apart, &previdx);
-       if (previdx == -1) {    /* searched all entries */
-           if (!foundentry) {
-               fprintf(STDERR, "Volume %s does not exist in VLDB\n\n",
-                       as->parms[0].items->data);
-               error = ENOENT;
-           }
+       foundentry = GetServerAndPart(&entry, ((voltype == ROVOL) ? ROVOL : RWVOL),
+                                     &aserver, &apart, &previdx);
+       if (!foundentry) {      /* searched all entries */
+           fprintf(STDERR, "Volume %s does not exist in VLDB\n\n",
+                   as->parms[0].items->data);
+           error = ENOENT;
            break;
        }
-       foundentry = 1;
 
        /* Get information about the volume from the server */
        if (verbose) {
@@ -1674,7 +1684,7 @@ ExamineVolume(struct cmd_syndesc *as, void *arock)
        if (code) {
            error = code;
            if (code == ENODEV) {
-               if ((voltype == BACKVOL) && !(entry.flags & BACK_EXISTS)) {
+               if ((voltype == BACKVOL) && !(entry.flags & VLF_BACKEXISTS)) {
                    /* The VLDB says there is no backup volume and its not on disk */
                    fprintf(STDERR, "Volume %s does not exist\n",
                            as->parms[0].items->data);
@@ -1699,7 +1709,7 @@ ExamineVolume(struct cmd_syndesc *as, void *arock)
            } else
                VolumeStats_int(pntr, &entry, aserver, apart, voltype);
 
-           if ((voltype == BACKVOL) && !(entry.flags & BACK_EXISTS)) {
+           if ((voltype == BACKVOL) && !(entry.flags & VLF_BACKEXISTS)) {
                /* The VLDB says there is no backup volume yet we found one on disk */
                fprintf(STDERR, "Volume %s does not exist in VLDB\n",
                        as->parms[0].items->data);
@@ -1755,6 +1765,7 @@ SetFields(struct cmd_syndesc *as, void *arock)
     afs_int32 apart;
     int previdx = -1;
     int have_field = 0;
+    int found;
 
     volid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);  /* -id */
     if (volid == 0) {
@@ -1775,8 +1786,8 @@ SetFields(struct cmd_syndesc *as, void *arock)
     }
     MapHostToNetwork(&entry);
 
-    GetServerAndPart(&entry, RWVOL, &aserver, &apart, &previdx);
-    if (previdx == -1) {
+    found = GetServerAndPart(&entry, RWVOL, &aserver, &apart, &previdx);
+    if (!found) {
        fprintf(STDERR, "Volume %s does not exist in VLDB\n\n",
                as->parms[0].items->data);
        return (ENOENT);
@@ -2065,32 +2076,6 @@ CreateVolume(struct cmd_syndesc *as, void *arock)
     return 0;
 }
 
-#if 0
-static afs_int32
-DeleteAll(struct nvldbentry *entry)
-{
-    int i;
-    afs_int32 error, code, curserver, curpart;
-    afs_uint32 volid;
-
-    MapHostToNetwork(entry);
-    error = 0;
-    for (i = 0; i < entry->nServers; i++) {
-       curserver = entry->serverNumber[i];
-       curpart = entry->serverPartition[i];
-       if (entry->serverFlags[i] & ITSROVOL) {
-           volid = entry->volumeId[ROVOL];
-       } else {
-           volid = entry->volumeId[RWVOL];
-       }
-       code = UV_DeleteVolume(curserver, curpart, volid);
-       if (code && !error)
-           error = code;
-    }
-    return error;
-}
-#endif
-
 static int
 DeleteVolume(struct cmd_syndesc *as, void *arock)
 {
@@ -2101,6 +2086,11 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
     char pname[10];
     afs_int32 idx, j;
 
+    if (as->parms[1].items && !as->parms[0].items) {
+       fprintf(STDERR, "vos: The -partition option requires the -server option.\n");
+       return EINVAL;
+    }
+
     if (as->parms[0].items) {
        server = GetServer(as->parms[0].items->data);
        if (!server) {
@@ -2155,9 +2145,9 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
            return (code);
        }
 
-       if (((volid == entry.volumeId[RWVOL]) && (entry.flags & RW_EXISTS))
+       if (((volid == entry.volumeId[RWVOL]) && (entry.flags & VLF_RWEXISTS))
            || ((volid == entry.volumeId[BACKVOL])
-               && (entry.flags & BACK_EXISTS))) {
+               && (entry.flags & VLF_BACKEXISTS))) {
            idx = Lp_GetRwIndex(&entry);
            if ((idx == -1) || (server && (server != entry.serverNumber[idx]))
                || ((partition != -1)
@@ -2167,9 +2157,9 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
                return ENOENT;
            }
        } else if ((volid == entry.volumeId[ROVOL])
-                  && (entry.flags & RO_EXISTS)) {
+                  && (entry.flags & VLF_ROEXISTS)) {
            for (idx = -1, j = 0; j < entry.nServers; j++) {
-               if (entry.serverFlags[j] != ITSROVOL)
+               if (!(entry.serverFlags[j] & VLSF_ROVOL))
                    continue;
 
                if (((server == 0) || (server == entry.serverNumber[j]))
@@ -2749,18 +2739,6 @@ CloneVolume(struct cmd_syndesc *as, void *arock)
                volname, VOLSER_OLDMAXVOLNAME - 1);
            return E2BIG;
        }
-#if 0
-       /*
-        * In order that you be able to make clones of RO or BK, this
-        * check must be omitted.
-        */
-       if (!VolNameOK(volname)) {
-           fprintf(STDERR,
-               "Illegal volume name %s, should not end in .readonly or .backup\n",
-               volname);
-           return EINVAL;
-       }
-#endif
        if (IsNumeric(volname)) {
            fprintf(STDERR,
                "Illegal volume name %s, should not be a number\n",
@@ -2842,7 +2820,7 @@ BackupVolume(struct cmd_syndesc *as, void *arock)
 
     /* is there a backup volume already? */
 
-    if (entry.flags & BACK_EXISTS) {
+    if (entry.flags & VLF_BACKEXISTS) {
        /* yep, where is it? */
 
        buvolid = entry.volumeId[BACKVOL];
@@ -2891,8 +2869,10 @@ ReleaseVolume(struct cmd_syndesc *as, void *arock)
 
     if (as->parms[1].items) /* -force */
        flags |= (REL_COMPLETE | REL_FULLDUMPS);
-    if (as->parms[2].items) /* -stayonline */
-       flags |= REL_STAYUP;
+    if (as->parms[2].items) { /* -stayonline */
+       fprintf(STDERR, "vos: -stayonline not supported\n");
+       return EINVAL;
+    }
     if (as->parms[3].items) /* -force-reclone */
         flags |= REL_COMPLETE;
 
@@ -3842,6 +3822,12 @@ SyncVldb(struct cmd_syndesc *as, void *arock)
     afs_uint32 tserver;
 
     tserver = 0;
+
+    if (as->parms[1].items && !as->parms[0].items) {
+       fprintf(STDERR, "vos: The -partition option requires the -server option.\n");
+       exit(1);
+    }
+
     if (as->parms[0].items) {
        tserver = GetServer(as->parms[0].items->data);
        if (!tserver) {
@@ -3868,12 +3854,6 @@ SyncVldb(struct cmd_syndesc *as, void *arock)
            exit(1);
        }
        flags = 1;
-
-       if (!tserver) {
-           fprintf(STDERR,
-                   "The -partition option requires a -server option\n");
-           exit(1);
-       }
     }
 
     if (as->parms[3].items) {
@@ -4125,33 +4105,32 @@ VolserStatus(struct cmd_syndesc *as, void *arock)
        }
        if (pntr->iflags) {
            fprintf(STDOUT, "attachFlags:  ");
-           switch (pntr->iflags) {
-           case ITOffline:
+           if ((pntr->iflags & ITOffline) != 0) {
                fprintf(STDOUT, "offline ");
-               break;
-           case ITBusy:
+           }
+           if ((pntr->iflags & ITBusy) != 0) {
                fprintf(STDOUT, "busy ");
-               break;
-           case ITReadOnly:
+           }
+           if ((pntr->iflags & ITReadOnly) != 0) {
                fprintf(STDOUT, "readonly ");
-               break;
-           case ITCreate:
+           }
+           if ((pntr->iflags & ITCreate) != 0) {
                fprintf(STDOUT, "create ");
-               break;
-           case ITCreateVolID:
+           }
+           if ((pntr->iflags & ITCreateVolID) != 0) {
                fprintf(STDOUT, "create volid ");
-               break;
            }
            fprintf(STDOUT, "\n");
        }
        if (pntr->vflags) {
            fprintf(STDOUT, "volumeStatus: ");
-           switch (pntr->vflags) {
-           case VTDeleteOnSalvage:
+           if ((pntr->vflags & VTDeleteOnSalvage) != 0) {
                fprintf(STDOUT, "deleteOnSalvage ");
-           case VTOutOfService:
+           }
+           if ((pntr->vflags & VTOutOfService) != 0) {
                fprintf(STDOUT, "outOfService ");
-           case VTDeleted:
+           }
+           if ((pntr->vflags & VTDeleted) != 0) {
                fprintf(STDOUT, "deleted ");
            }
            fprintf(STDOUT, "\n");
@@ -4183,25 +4162,39 @@ VolserStatus(struct cmd_syndesc *as, void *arock)
 static int
 RenameVolume(struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 code1, code2, code;
+    afs_int32 code;
     struct nvldbentry entry;
+    struct nvldbentry entry2;
 
-    code1 = VLDB_GetEntryByName(as->parms[0].items->data, &entry);
-    if (code1) {
+    /* Get the entry of the volume to be renamed (-oldname), by name or id. */
+    code = VLDB_GetEntryByName(as->parms[0].items->data, &entry);
+    if (code) {
        fprintf(STDERR, "vos: Could not find entry for volume %s\n",
                as->parms[0].items->data);
+       PrintError("", code);
        exit(1);
     }
-    code2 = VLDB_GetEntryByName(as->parms[1].items->data, &entry);
-    if ((!code1) && (!code2)) {        /*the newname already exists */
-       fprintf(STDERR, "vos: volume %s already exists\n",
+
+    /*
+     * Verify the new name is available before attempting to rename.
+     * Allow renaming of the same volume in order to complete a
+     * previously interrupted rename.
+     */
+    code = VLDB_GetEntryByName(as->parms[1].items->data, &entry2);
+    if (code != 0 && code != VL_NOENT) {
+       fprintf(STDERR, "vos: Could not check entry for volume %s\n",
                as->parms[1].items->data);
+       PrintError("", code);
        exit(1);
     }
-
-    if (code1 && code2) {
-       fprintf(STDERR, "vos: Could not find entry for volume %s or %s\n",
-               as->parms[0].items->data, as->parms[1].items->data);
+    if (code == 0 && entry.volumeId[RWVOL] != entry2.volumeId[RWVOL]) {
+       fprintf(STDERR, "vos: Cannot rename volume %s (%lu) to %s;"
+                       " volume %s (%lu) already exists\n",
+                       as->parms[0].items->data,
+                       (unsigned long)entry.volumeId[RWVOL],
+                       as->parms[1].items->data,
+                       as->parms[1].items->data,
+                       (unsigned long)entry2.volumeId[RWVOL]);
        exit(1);
     }
     if (!VolNameOK(as->parms[0].items->data)) {
@@ -4259,8 +4252,8 @@ GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part, afs_int32 *
     if (volid == rentry->volumeId[ROVOL]) {
        *voltype = ROVOL;
        for (i = 0; i < rentry->nServers; i++) {
-           if ((index == -1) && (rentry->serverFlags[i] & ITSROVOL)
-               && !(rentry->serverFlags[i] & RO_DONTUSE))
+           if ((index == -1) && (rentry->serverFlags[i] & VLSF_ROVOL)
+               && !(rentry->serverFlags[i] & VLSF_DONTUSE))
                index = i;
        }
        if (index == -1) {
@@ -4503,9 +4496,9 @@ static int CompareVldbEntry(char *p1, char *p2)
     pos2 = -1;
 
     for(i = 0; i < arg1->nServers; i++)
-       if(arg1->serverFlags[i] & ITSRWVOL) pos1 = i;
+       if(arg1->serverFlags[i] & VLSF_RWVOL) pos1 = i;
     for(i = 0; i < arg2->nServers; i++)
-       if(arg2->serverFlags[i] & ITSRWVOL) pos2 = i;
+       if(arg2->serverFlags[i] & VLSF_RWVOL) pos2 = i;
     if(pos1 == -1 || pos2 == -1){
        pos1 = 0;
        pos2 = 0;
@@ -4542,7 +4535,7 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
 
     apart = 0;
 
-    attributes.Mask = 0;
+    memset(&attributes, 0, sizeof(attributes));
     lock = (as->parms[3].items ? 1 : 0);       /* -lock   flag */
     quiet = (as->parms[4].items ? 1 : 0);      /* -quit   flag */
     sort = (as->parms[5].items ? 0 : 1);       /* -nosort flag */
@@ -4719,6 +4712,7 @@ BackSys(struct cmd_syndesc *as, void *arock)
 #ifndef HAVE_POSIX_REGEX
     char *ccode;
 #endif
+    int found;
 
     memset(&attributes, 0, sizeof(struct VldbListByAttributes));
     attributes.Mask = 0;
@@ -4963,7 +4957,7 @@ BackSys(struct cmd_syndesc *as, void *arock)
            continue;
        }
 
-       if (!(vllist->flags & RW_EXISTS)) {
+       if (!(vllist->flags & VLF_RWEXISTS)) {
            if (verbose) {
                fprintf(STDOUT,
                        "Omitting to backup %s since RW volume does not exist \n",
@@ -4976,8 +4970,8 @@ BackSys(struct cmd_syndesc *as, void *arock)
 
        avolid = vllist->volumeId[RWVOL];
        MapHostToNetwork(vllist);
-       GetServerAndPart(vllist, RWVOL, &aserver1, &apart1, &previdx);
-       if (aserver1 == -1 || apart1 == -1) {
+       found = GetServerAndPart(vllist, RWVOL, &aserver1, &apart1, &previdx);
+       if (!found) {
            fprintf(STDOUT, "could not backup %s, invalid VLDB entry\n",
                    vllist->name);
            totalFail++;
@@ -5045,7 +5039,7 @@ UnlockVLDB(struct cmd_syndesc *as, void *arock)
 
     apart = -1;
     totalE = 0;
-    attributes.Mask = 0;
+    memset(&attributes, 0, sizeof(attributes));
 
     if (as->parms[0].items) {  /* server specified */
        aserver = GetServer(as->parms[0].items->data);
@@ -5064,14 +5058,17 @@ UnlockVLDB(struct cmd_syndesc *as, void *arock)
                    as->parms[1].items->data);
            exit(1);
        }
-       if (!IsPartValid(apart, aserver, &code)) {      /*check for validity of the partition */
-           if (code)
-               PrintError("", code);
-           else
-               fprintf(STDERR,
-                       "vos : partition %s does not exist on the server\n",
-                       as->parms[1].items->data);
-           exit(1);
+       if (aserver) {
+           /* Check for validity of the partition if a server was given. */
+           if (!IsPartValid(apart, aserver, &code)) {
+               if (code)
+                   PrintError("", code);
+               else
+                   fprintf(STDERR,
+                           "vos : partition %s does not exist on the server\n",
+                           as->parms[1].items->data);
+               exit(1);
+           }
        }
        attributes.partition = apart;
        attributes.Mask |= VLLIST_PARTITION;
@@ -5244,6 +5241,7 @@ ChangeAddr(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 ip1, ip2, vcode;
     int remove = 0;
+    int force = 0;
 
     if (noresolve)
        ip1 = GetServerNoresolve(as->parms[0].items->data);
@@ -5261,6 +5259,10 @@ ChangeAddr(struct cmd_syndesc *as, void *arock)
        return (EINVAL);
     }
 
+    if (as->parms[3].items) {
+       force = 1;
+    }
+
     if (as->parms[1].items) {
        if (noresolve)
            ip2 = GetServerNoresolve(as->parms[1].items->data);
@@ -5280,6 +5282,42 @@ ChangeAddr(struct cmd_syndesc *as, void *arock)
        ip1 = 0xffffffff;
     }
 
+    if (!remove && !force) {
+       afs_int32 m_nentries;
+       bulkaddrs m_addrs;
+       afs_int32 m_uniq = 0;
+       afsUUID m_uuid;
+       ListAddrByAttributes m_attrs;
+       struct uuid_fmtbuf buffer;
+
+       memset(&m_attrs, 0, sizeof(m_attrs));
+       memset(&m_uuid, 0, sizeof(m_uuid));
+       memset(&m_addrs, 0, sizeof(m_addrs));
+
+       m_attrs.Mask = VLADDR_IPADDR;
+       m_attrs.ipaddr = ntohl(ip1);    /* -oldaddr */
+
+       vcode =
+           ubik_VL_GetAddrsU(cstruct, UBIK_CALL_NEW, &m_attrs, &m_uuid,
+                             &m_uniq, &m_nentries, &m_addrs);
+       xdr_free((xdrproc_t) xdr_bulkaddrs, &m_addrs);
+       switch (vcode) {
+       case 0:         /* mh entry detected */
+           afsUUID_to_string(&m_uuid, &buffer);
+           fprintf(STDERR, "vos: Refusing to change address in multi-homed server entry.\n");
+           fprintf(STDERR, "     -oldaddr address is registered to file server UUID %s\n",
+                           buffer.buffer);
+           fprintf(STDERR, "     Please restart the file server or use vos setaddrs.\n");
+           return EINVAL;
+       case VL_NOENT:
+           break;
+       default:
+           fprintf(STDERR, "vos: could not list the server addresses\n");
+           PrintError("", vcode);
+           return vcode;
+       }
+    }
+
     vcode = ubik_VL_ChangeAddr(cstruct, UBIK_CALL_NEW, ntohl(ip1), ntohl(ip2));
     if (vcode) {
        char hoststr1[16], hoststr2[16];
@@ -5316,23 +5354,22 @@ print_addrs(const bulkaddrs * addrs, afsUUID * m_uuid, int nentries,
            int print)
 {
     int i;
-    afs_uint32 *addrp;
-    char buf[1024];
+    afs_uint32 addr;
+    struct uuid_fmtbuf buf;
 
     if (print) {
-       afsUUID_to_string(m_uuid, buf, sizeof(buf));
-       printf("UUID: %s\n", buf);
+       afsUUID_to_string(m_uuid, &buf);
+       printf("UUID: %s\n", buf.buffer);
     }
 
     /* print out the list of all the server */
-    addrp = (afs_uint32 *) addrs->bulkaddrs_val;
-    for (i = 0; i < nentries; i++, addrp++) {
-       *addrp = htonl(*addrp);
+    for (i = 0; i < addrs->bulkaddrs_len; i++) {
+       addr = htonl(addrs->bulkaddrs_val[i]);
        if (noresolve) {
            char hoststr[16];
-           printf("%s\n", afs_inet_ntoa_r(*addrp, hoststr));
+           printf("%s\n", afs_inet_ntoa_r(addr, hoststr));
        } else {
-           printf("%s\n", hostutil_GetNameByINet(*addrp));
+           printf("%s\n", hostutil_GetNameByINet(addr));
        }
     }
 
@@ -5515,6 +5552,68 @@ SetAddrs(struct cmd_syndesc *as, void *arock)
 }
 
 static int
+RemoveAddrs(struct cmd_syndesc *as, void *arock)
+{
+    afs_int32 code;
+    ListAddrByAttributes attrs;
+    afsUUID uuid;
+    afs_int32 uniq = 0;
+    afs_int32 nentries = 0;
+    bulkaddrs addrs;
+    afs_uint32 ip1;
+    afs_uint32 ip2;
+
+    memset(&attrs, 0, sizeof(ListAddrByAttributes));
+    memset(&addrs, 0, sizeof(bulkaddrs));
+    memset(&uuid, 0, sizeof(afsUUID));
+    attrs.Mask = VLADDR_UUID;
+
+    if (as->parms[0].items) {  /* -uuid */
+       if (afsUUID_from_string(as->parms[0].items->data, &attrs.uuid) < 0) {
+           fprintf(STDERR, "vos: invalid UUID '%s'\n",
+                   as->parms[0].items->data);
+           return EINVAL;
+       }
+    }
+
+    code =
+       ubik_VL_GetAddrsU(cstruct, UBIK_CALL_NEW, &attrs, &uuid, &uniq,
+                         &nentries, &addrs);
+    if (code == VL_NOENT) {
+       fprintf(STDERR, "vos: UUID not found\n");
+       goto out;
+    }
+    if (code != 0) {
+       fprintf(STDERR, "vos: could not list the server addresses\n");
+       PrintError("", code);
+       goto out;
+    }
+    if (addrs.bulkaddrs_len == 0) {
+       fprintf(STDERR, "vos: no addresses found for UUID\n");
+       goto out;
+    }
+
+    ip2 = addrs.bulkaddrs_val[0]; /* network byte order */
+    ip1 = 0xffffffff;             /* indicates removal mode */
+
+    if (verbose) {
+       printf("vos: Removing UUID with hosts:\n");
+       print_addrs(&addrs, &uuid, nentries, 1);
+    }
+
+    code = ubik_VL_ChangeAddr(cstruct, UBIK_CALL_NEW, ip1, ip2);
+    if (code) {
+       fprintf(STDERR, "Could not remove server entry from the VLDB.\n");
+       PrintError("", code);
+    }
+
+  out:
+    xdr_free((xdrproc_t) xdr_bulkaddrs, &addrs);
+    return code;
+}
+
+
+static int
 LockEntry(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
@@ -5606,12 +5705,12 @@ ConvertRO(struct cmd_syndesc *as, void *arock)
 
     MapHostToNetwork(&entry);
     for (i = 0; i < entry.nServers; i++) {
-       if (entry.serverFlags[i] & ITSRWVOL) {
+       if (entry.serverFlags[i] & VLSF_RWVOL) {
            rwserver = entry.serverNumber[i];
            rwpartition = entry.serverPartition[i];
            if (roserver)
                break;
-       } else if ((entry.serverFlags[i] & ITSROVOL) && !roserver) {
+       } else if ((entry.serverFlags[i] & VLSF_ROVOL) && !roserver) {
            same = VLDB_IsSameAddrs(server, entry.serverNumber[i], &code);
            if (code) {
                fprintf(STDERR,
@@ -5823,6 +5922,7 @@ static int
 MyBeforeProc(struct cmd_syndesc *as, void *arock)
 {
     char *tcell;
+    char *rxgk_seclevel_str = NULL;
     afs_int32 code;
     int secFlags;
 
@@ -5853,6 +5953,23 @@ MyBeforeProc(struct cmd_syndesc *as, void *arock)
     if (as->parms[COMMONPARM_OFFSET_CONFIG].items)   /* -config flag set */
        confdir = as->parms[COMMONPARM_OFFSET_CONFIG].items->data;
 
+    if (cmd_OptionAsString(as, COMMONPARM_OFFSET_RXGK, &rxgk_seclevel_str) == 0) {
+       if (strcmp(rxgk_seclevel_str, "clear") == 0)
+           secFlags |= AFSCONF_SECOPTS_ALWAYSCLEAR;
+       else if (strcmp(rxgk_seclevel_str, "auth") == 0)
+           secFlags |= AFSCONF_SECOPTS_NEVERENCRYPT;
+       else if (strcmp(rxgk_seclevel_str, "crypt") == 0) {
+           /* don't need to set any flags; this is the default for rxgk */
+       } else {
+           fprintf(STDERR, "Invalid argument to -rxgk: %s\n", rxgk_seclevel_str);
+           exit(1);
+       }
+       secFlags |= AFSCONF_SECOPTS_RXGK;
+
+       free(rxgk_seclevel_str);
+       rxgk_seclevel_str = NULL;
+    }
+
     if ((code = vsu_ClientInit(confdir, tcell, secFlags, UV_SetSecurity,
                               &cstruct))) {
        fprintf(STDERR, "could not initialize VLDB library (code=%lu) \n",
@@ -5916,9 +6033,6 @@ main(int argc, char **argv)
                "initial quota (KB)");
     cmd_AddParm(ts, "-id", CMD_SINGLE, CMD_OPTIONAL, "volume ID");
     cmd_AddParm(ts, "-roid", CMD_SINGLE, CMD_OPTIONAL, "readonly volume ID");
-#ifdef notdef
-    cmd_AddParm(ts, "-minquota", CMD_SINGLE, CMD_OPTIONAL, "");
-#endif
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("remove", DeleteVolume, NULL, 0, "delete a volume");
@@ -6010,7 +6124,8 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-force", CMD_FLAG, CMD_OPTIONAL,
                "force a complete release and full dumps");
     cmd_AddParmAlias(ts, 1, "-f"); /* original force option */
-    cmd_AddParm(ts, "-stayonline", CMD_FLAG, CMD_OPTIONAL,
+    /* keep this reserved */
+    cmd_AddParm(ts, "-stayonline", CMD_FLAG, CMD_OPTIONAL|CMD_HIDDEN,
                "release to cloned temp vol, then clone back to repsite RO");
     cmd_AddParm(ts, "-force-reclone", CMD_FLAG, CMD_OPTIONAL,
                "force a reclone and complete release with incremental dumps");
@@ -6228,6 +6343,8 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-newaddr", CMD_SINGLE, CMD_OPTIONAL, "new IP address");
     cmd_AddParm(ts, "-remove", CMD_FLAG, CMD_OPTIONAL,
                "remove the IP address from the VLDB");
+    cmd_AddParm(ts, "-force", CMD_FLAG, CMD_OPTIONAL,
+               "allow multi-homed server entry change (not recommended)");
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("listaddrs", ListAddrs, NULL, 0,
@@ -6264,11 +6381,16 @@ main(int argc, char **argv)
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("setaddrs", SetAddrs, NULL, 0,
-                         "set the list of IP address for a given UUID in the VLDB");
+                         "set the list of IP addresses for a given UUID in the VLDB");
     cmd_AddParm(ts, "-uuid", CMD_SINGLE, 0, "uuid of server");
     cmd_AddParm(ts, "-host", CMD_LIST, 0, "address of host");
+    COMMONPARMS;
 
+    ts = cmd_CreateSyntax("remaddrs", RemoveAddrs, NULL, 0,
+                         "remove the list of IP addresses for a given UUID in the VLDB");
+    cmd_AddParm(ts, "-uuid", CMD_SINGLE, 0, "uuid of server");
     COMMONPARMS;
+
     code = cmd_Dispatch(argc, argv);
     if (rxInitDone) {
        /* Shut down the ubik_client and rx connections */