From: Garry Zacheiss <zacheiss@mit.edu>
Date: Tue, 7 May 2002 21:01:27 +0000 (+0000)
Subject: Use vsu_ExtractName() to get the RW volume name before doing the vid lookup
X-Git-Tag: openafs-devel_1_3_3~134
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=bac8c825478e309c225494f031ba9c6fd62817ee;p=packages%2Fo%2Fopenafs.git

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 821be0d02..fb7e8057e 100644
--- 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);