Use vsu_ExtractName() to get the RW volume name before doing the vid lookup

for addsite and remsite operations, since "vos addsite foo.bar.readonly" is
such a common thinko.

diff --git a/src/volser/vos.c b/src/volser/vos.c
index 821be0d..fb7e805 100644 (file)
--- a/src/volser/vos.c
+++ b/src/volser/vos.c
@@ -2207,9 +2207,10 @@ static AddSite(as)
 register struct cmd_syndesc *as;
 {
    afs_int32 avolid, aserver, apart,code, err;
-   char apartName[10];
+   char apartName[10], avolname[VOLSER_MAXVOLNAME+1];
 
-       avolid = vsu_GetVolumeID(as->parms[2].items->data, cstruct, &err);
+       vsu_ExtractName(avolname, as->parms[2].items->data);;
+       avolid = vsu_GetVolumeID(avolname, cstruct, &err);
        if (avolid == 0) {
            if (err) PrintError("", err);
            else fprintf(STDERR, "vos: can't find volume '%s'\n", as->parms[2].items->data);
@@ -2245,9 +2246,10 @@ register struct cmd_syndesc *as;
 { 
 
     afs_int32 avolid, aserver, apart, code, err;
-    char apartName[10];
+    char apartName[10], avolname[VOLSER_MAXVOLNAME+1];
 
-       avolid = vsu_GetVolumeID(as->parms[2].items->data, cstruct, &err);
+       vsu_ExtractName(avolname, as->parms[2].items->data);
+       avolid = vsu_GetVolumeID(avolname, cstruct, &err);
        if (avolid == 0) {
            if (err) PrintError("", err);
            else fprintf(STDERR, "vos: can't find volume '%s'\n", as->parms[2].items->data);