]> andersk Git - moira.git/blobdiff - clients/moira/attach.c
changed autocreate flag's meaning
[moira.git] / clients / moira / attach.c
index a3b850a448e6621fd6ee0fd9e40f8c8488296b4f..223a7830b6ca56a5233be4287ac48a294d2810fa 100644 (file)
@@ -2,8 +2,8 @@
   static char rcsid_module_c[] = "$Header$";
 #endif
 
-/*     This is the file attach.c for the SMS Client, which allows a nieve
- *      user to quickly and easily maintain most parts of the SMS database.
+/*     This is the file attach.c for the MOIRA Client, which allows a nieve
+ *      user to quickly and easily maintain most parts of the MOIRA database.
  *     It Contains: Functions for maintaining data used by Hesiod 
  *                   to map courses/projects/users to their file systems, 
  *                   and maintain filesys info. 
 
 #include <stdio.h>
 #include <strings.h>
-#include <sms.h>
-#include <sms_app.h>
+#include <moira.h>
+#include <moira_site.h>
 #include <menu.h>
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <ctype.h>
 
 #include "mit-copyright.h"
 #include "defs.h"
 #define NO_MACHINE      ("\\[NONE\\]") /* C will remove one of the /'s here,
                                         * and the other quotes the [ for
                                         * ingres' regexp facility. */
+#define NO_MACHINE_BAD  ("[NONE]")
 
 #define DEFAULT_TYPE     ("NFS")
 #define DEFAULT_MACHINE  DEFAULT_NONE
 #define DEFAULT_PACK     DEFAULT_NONE
-#define DEFAULT_M_POINT  DEFAULT_NONE
 #define DEFAULT_ACCESS   ("w")
 #define DEFAULT_COMMENTS DEFAULT_COMMENT
 #define DEFAULT_OWNER    (user)
 #define DEFAULT_OWNERS   (user)
 #define DEFAULT_CREATE   DEFAULT_YES
 #define DEFAULT_L_TYPE   ("PROJECT")
+#define DEFAULT_CELL    ("athena.mit.edu")
 
 /*     Function Name: SetDefaults
  *     Description: sets the default values for filesystem additions.
@@ -65,11 +69,14 @@ SetDefaults(info, name)
 char ** info;
 char * name;
 {
+    char buf[BUFSIZ];
+
     info[FS_NAME] =     Strsave(name);
     info[FS_TYPE] =     Strsave(DEFAULT_TYPE);
     info[FS_MACHINE] =  Strsave(DEFAULT_MACHINE);
     info[FS_PACK] =     Strsave(DEFAULT_PACK);
-    info[FS_M_POINT] =  Strsave(DEFAULT_M_POINT);
+    sprintf(buf, "/mit/%s", name);
+    info[FS_M_POINT] =  Strsave(buf);
     info[FS_ACCESS] =   Strsave(DEFAULT_ACCESS);
     info[FS_COMMENTS] = Strsave(DEFAULT_COMMENTS);
     info[FS_OWNER] =    Strsave(DEFAULT_OWNER);
@@ -98,23 +105,23 @@ char *name;
 
     switch (type) {
     case LABEL:
-       if ( (stat = do_sms_query("get_filesys_by_label", 1, &name,
+       if ( (stat = do_mr_query("get_filesys_by_label", 1, &name,
                                  StoreInfo, (char *)&elem)) != 0) {
-           com_err(program_name, stat, NULL);
+           com_err(program_name, stat, " in GetFSInfo");
            return(NULL);
        }
        break;
     case MACHINE:
-       if ( (stat = do_sms_query("get_filesys_by_machine", 1, &name,
+       if ( (stat = do_mr_query("get_filesys_by_machine", 1, &name,
                                  StoreInfo, (char *)&elem)) != 0) {
-           com_err(program_name, stat, NULL);
+           com_err(program_name, stat, " in GetFSInfo");
            return(NULL);
        }
        break;
     case GROUP:
-       if ( (stat = do_sms_query("get_filesys_by_group", 1, &name,
+       if ( (stat = do_mr_query("get_filesys_by_group", 1, &name,
                                  StoreInfo, (char *)&elem)) != 0) {
-           com_err(program_name, stat, NULL);
+           com_err(program_name, stat, " in GetFSInfo");
            return(NULL);
        }
        break;
@@ -122,7 +129,7 @@ char *name;
        args[ALIAS_NAME] = name;
        args[ALIAS_TYPE] = FS_ALIAS_TYPE;
        args[ALIAS_TRANS] = "*";
-       if ( (stat = do_sms_query("get_alias", 3, args, StoreInfo, 
+       if ( (stat = do_mr_query("get_alias", 3, args, StoreInfo, 
                                  (char *) &elem)) != 0) {
            com_err(program_name, stat, " in get_alias.");
            return(NULL);
@@ -177,44 +184,49 @@ char ** info;
     char print_buf[BUFSIZ];
 
     FORMFEED;
-    sprintf(print_buf,"%20s Filesystem Group: %s", 
-           " ",info[FS_NAME]);
-    Put_message(print_buf);
 
-    if (!strcmp(info[FS_TYPE], "FSGROUP")) {
+    if (!strcmp(info[FS_TYPE], "FSGROUP") || !strcmp(info[FS_TYPE], "MUL")) {
        int stat;
        struct qelem *elem = NULL;
 
-       sprintf(print_buf,"Comments; %s",info[FS_COMMENTS]);
+       if (!strcmp(info[FS_TYPE], "MUL"))
+         sprintf(print_buf,"%20s Multiple Filesystem: %s", " ", info[FS_NAME]);
+       else
+         sprintf(print_buf,"%20s Filesystem Group: %s", " ", info[FS_NAME]);
+       Put_message(print_buf);
+
+       sprintf(print_buf,"Comments: %s",info[FS_COMMENTS]);
        Put_message(print_buf);
        sprintf(print_buf, MOD_FORMAT, info[FS_MODBY], info[FS_MODTIME], 
                info[FS_MODWITH]);
        Put_message(print_buf);
        Put_message("Containing the filesystems (in order):");
-       if ((stat = do_sms_query("get_fsgroup_members", 1, &info[FS_NAME],
+       if ((stat = do_mr_query("get_fsgroup_members", 1, &info[FS_NAME],
                                 StoreInfo, (char *)&elem)) != 0) {
-           if (stat == SMS_NO_MATCH)
+           if (stat == MR_NO_MATCH)
              Put_message("    [no members]");
            else
-             com_err(program_name, stat, NULL);
+             com_err(program_name, stat, " in PrintFSInfo");
        } else {
            fsgCount = 1;
            Loop(QueueTop(elem), (void *) PrintFSGMembers);
            FreeQueue(elem);
        }
     } else {
+       sprintf(print_buf,"%20s Filesystem: %s", " ", info[FS_NAME]);
+       Put_message(print_buf);
        sprintf(print_buf,"Type: %-40s Machine: %-15s",
                info[FS_TYPE], info[FS_MACHINE]);
        Put_message(print_buf);
        sprintf(print_buf,"Default Access: %-2s Packname: %-17s Mountpoint %s ",
                info[FS_ACCESS], info[FS_PACK], info[FS_M_POINT]);
        Put_message(print_buf);
-       sprintf(print_buf,"Comments; %s",info[FS_COMMENTS]);
+       sprintf(print_buf,"Comments: %s",info[FS_COMMENTS]);
        Put_message(print_buf);
        sprintf(print_buf, "User Ownership: %-30s Group Ownership: %s",
                info[FS_OWNER], info[FS_OWNERS]);
        Put_message(print_buf);
-       sprintf(print_buf, "Auto Create: %-34s Locker Type: %s",
+       sprintf(print_buf, "Update Fileserver: %-27s Locker Type: %s",
                atoi(info[FS_CREATE]) ? "ON" : "OFF", 
                info[FS_L_TYPE]);
        Put_message(print_buf);
@@ -225,6 +237,34 @@ char ** info;
     return(info[FS_NAME]);
 }
 
+
+char *canonicalize_cell(c)
+char *c;
+{
+    struct stat stbuf;
+    char path[512];
+    int count;
+
+    sprintf(path, "/afs/%s", c);
+    if (lstat(path, &stbuf) || !stbuf.st_mode&S_IFLNK)
+      return(c);
+    count = readlink(path, path, sizeof(path));
+    if (count < 1) return(c);
+    path[count] = 0;
+    free(c);
+    return(strsave(path));
+}
+
+
+int GetAliasValue(argc, argv, retval)
+int argc;
+char **argv;
+char **retval;
+{
+    *retval = strsave(argv[2]);
+    return(MR_CONT);
+}
+
 /*     Function Name: AskFSInfo.
  *     Description: This function askes the user for information about a 
  *                   machine and saves it into a structure.
@@ -239,8 +279,8 @@ AskFSInfo(info, name)
 char ** info;
 Bool name;
 {
-    char temp_buf[BUFSIZ], *newname;
-    int fsgroup = 0;
+    char temp_buf[BUFSIZ], *newname, access_type[32];
+    int fsgroup = 0, newdefaults = 0;
 
     Put_message("");
     sprintf(temp_buf, "Changing Attributes of filesystem %s.", 
@@ -250,34 +290,131 @@ Bool name;
 
     if (name) {
        newname = Strsave(info[FS_NAME]);
-       GetValueFromUser("The new name for this filesystem",
-                        &newname);
+       if (GetValueFromUser("The new name for this filesystem",
+                            &newname) == SUB_ERROR)
+         return(NULL);
     }
 
-    GetTypeFromUser("Filesystem's Type", "filesys", &info[FS_TYPE]);
-    if (!strcmp(info[FS_TYPE], "FSGROUP") || !strcmp(info[FS_TYPE], "fsgroup"))
+    strcpy(temp_buf, info[FS_TYPE]);
+    if (GetTypeFromUser("Filesystem's Type", "filesys", &info[FS_TYPE]) ==
+       SUB_ERROR)
+      return(NULL);
+    if (!strcasecmp(info[FS_TYPE], "FSGROUP") ||
+       !strcasecmp(info[FS_TYPE], "MUL"))
       fsgroup++;
+    if (strcasecmp(info[FS_TYPE], temp_buf))
+      newdefaults++;
     if (fsgroup) {
        free(info[FS_MACHINE]);
        info[FS_MACHINE] = Strsave(NO_MACHINE);
     } else {
-       GetValueFromUser("Filesystem's Machine", &info[FS_MACHINE]);
-       strcpy(temp_buf, CanonicalizeHostname(info[FS_MACHINE]));
-       free(info[FS_MACHINE]);
-       info[FS_MACHINE] = Strsave(temp_buf);
+       if (!strcmp(info[FS_MACHINE], NO_MACHINE_BAD)) {
+           free(info[FS_MACHINE]);
+           info[FS_MACHINE] = Strsave(NO_MACHINE);
+       }
+       if (!strcasecmp(info[FS_TYPE], "AFS")) {
+           if (!name || newdefaults) {
+               free(info[FS_MACHINE]);
+               info[FS_MACHINE] = strsave(DEFAULT_CELL);
+           }
+           if (GetValueFromUser("Filesystem's Cell", &info[FS_MACHINE]) ==
+               SUB_ERROR)
+             return(NULL);
+           info[FS_MACHINE] = canonicalize_cell(info[FS_MACHINE]);
+       } else {
+           if (GetValueFromUser("Filesystem's Machine", &info[FS_MACHINE]) ==
+               SUB_ERROR)
+             return(NULL);
+           info[FS_MACHINE] = canonicalize_hostname(info[FS_MACHINE]);
+       }
     }
     if (!fsgroup) {
-       GetValueFromUser("Filesystem's Pack Name", &info[FS_PACK]);
-       GetValueFromUser("Filesystem's Mount Point", &info[FS_M_POINT]);
-       GetValueFromUser("Filesystem's Default Access", &info[FS_ACCESS]);
+       if (!strcasecmp(info[FS_TYPE], "AFS")) {
+           char *path, *args[3], *p;
+           int status, depth, i;
+           if (GetTypeFromUser("Filesystem's lockertype", "lockertype",
+                               &info[FS_L_TYPE]) == SUB_ERROR)
+             return(NULL);
+           if (!name || newdefaults) {
+               free(info[FS_PACK]);
+               lowercase(info[FS_MACHINE]);
+               sprintf(temp_buf, "%s:%s", info[FS_MACHINE], info[FS_L_TYPE]);
+               args[0] = temp_buf;
+               args[1] = "AFSPATH";
+               args[2] = "*";
+               path = "???";
+               status = do_mr_query("get_alias", 3, args,
+                                    GetAliasValue, &path);
+               if (status == MR_SUCCESS) {
+                   p = index(path, ':');
+                   if (p) {
+                       *p = 0;
+                       depth = atoi(++p);
+                   } else
+                     depth = 0;
+                   sprintf(temp_buf, "/afs/%s/%s", info[FS_MACHINE], path);
+                   if (depth >= 0) {
+                       for (p=info[FS_NAME]; *p&&(p-info[FS_NAME])<depth; p++) {
+                           if (islower(*p)) {
+                               strcat(temp_buf, "/x"); 
+                               temp_buf[strlen(temp_buf)-1] = *p;
+                           } else {
+                               sprintf(temp_buf, "/afs/%s/%s/other", info[FS_MACHINE], path);
+                               break;
+                           }
+                       }
+                   } else if (depth = -1) {
+                       if (isdigit(info[FS_NAME][0])) {
+                           strcat(temp_buf, "/");
+                           depth = strlen(temp_buf);
+                           for (p = info[FS_NAME]; *p && isdigit(*p); p++) {
+                               temp_buf[depth++] = *p;
+                               temp_buf[depth] = 0;
+                           }
+                       } else
+                         strcat(temp_buf, "/other");
+                   } else {
+                       /* no default */
+                   }
+                   strcat(temp_buf, "/");
+                   strcat(temp_buf, info[FS_NAME]);
+                   free(path);
+               } else {
+                   sprintf(temp_buf, "/afs/%s/%s/%s", info[FS_MACHINE],
+                           lowercase(info[FS_L_TYPE]), info[FS_NAME]);
+               }
+               info[FS_PACK] = strsave(temp_buf);
+           }
+       }
+       if (GetValueFromUser("Filesystem's Pack Name", &info[FS_PACK]) ==
+           SUB_ERROR)
+         return(NULL);
+       if (GetValueFromUser("Filesystem's Mount Point", &info[FS_M_POINT]) ==
+           SUB_ERROR)
+         return(NULL);
+       sprintf(access_type, "fs_access_%s", info[FS_TYPE]);
+       if (GetTypeFromUser("Filesystem's Default Access", access_type,
+                           &info[FS_ACCESS]) == SUB_ERROR)
+         return(NULL);
     }
-    GetValueFromUser("Comments about this Filesystem", &info[FS_COMMENTS]);
-    GetValueFromUser("Filesystem's owner (user)", &info[FS_OWNER]);
-    GetValueFromUser("Filesystem's owners (group)", &info[FS_OWNERS]);
+    if (GetValueFromUser("Comments about this Filesystem", &info[FS_COMMENTS])
+       == SUB_ERROR)
+      return(NULL);
+    if (GetValueFromUser("Filesystem's owner (user)", &info[FS_OWNER]) ==
+       SUB_ERROR)
+      return(NULL);
+    if (GetValueFromUser("Filesystem's owners (group)", &info[FS_OWNERS]) ==
+       SUB_ERROR)
+      return(NULL);
     if (!fsgroup)
-      GetYesNoValueFromUser("Automatically create this filesystem",
-                           &info[FS_CREATE]);
-    GetTypeFromUser("Filesystem's lockertype", "lockertype", &info[FS_L_TYPE]);
+      if (GetYesNoValueFromUser("Propagate changes to fileserver",
+                               &info[FS_CREATE]) == SUB_ERROR)
+       return(NULL);
+    if (strcasecmp(info[FS_TYPE], "AFS")) {
+       if (GetTypeFromUser("Filesystem's lockertype", "lockertype",
+                           &info[FS_L_TYPE]) == SUB_ERROR)
+         return(NULL);
+    }
 
     FreeAndClear(&info[FS_MODTIME], TRUE);
     FreeAndClear(&info[FS_MODBY], TRUE);
@@ -289,6 +426,7 @@ Bool name;
     return(info);
 }
 
+
 /* --------------- Filesystem Menu ------------- */
 
 /*     Function Name: GetFS
@@ -311,6 +449,27 @@ char **argv;
     return (DM_NORMAL);
 }
 
+/*     Function Name: GetFSM
+ *     Description: Get Filesystem information by machine.
+ *     Arguments: argc, argv - name of server in argv[1].
+ *     Returns: DM_NORMAL.
+ */
+
+/* ARGSUSED */
+int
+GetFSM(argc, argv)
+int argc;
+char **argv;
+{
+    struct qelem *top;
+
+    argv[1] = canonicalize_hostname(strsave(argv[1]));
+    top = GetFSInfo(MACHINE, argv[1]); /* get info. */
+    Loop(top, (void *) PrintFSInfo);
+    FreeQueue(top);            /* clean the queue. */
+    return (DM_NORMAL);
+}
+
 /*     Function Name: RealDeleteFS
  *     Description: Does the real deletion work.
  *     Arguments: info - array of char *'s containing all useful info.
@@ -334,7 +493,7 @@ Bool one_item;
     sprintf(temp_buf, "Are you sure that you want to delete filesystem %s",
            info[FS_NAME]);
     if(!one_item || Confirm(temp_buf)) {
-       if ( (stat = do_sms_query("delete_filesys", 1,
+       if ( (stat = do_mr_query("delete_filesys", 1,
                                  &info[FS_NAME], Scream, NULL)) != 0)
            com_err(program_name, stat, " filesystem not deleted.");
        else
@@ -378,18 +537,24 @@ char ** info;
 Bool junk;
 {
     int stat;
-    char ** args = AskFSInfo(info, TRUE);
+    char ** args;
     extern Menu nfsphys_menu;
 
-    stat = do_sms_query("update_filesys", CountArgs(args), args,
+    args = AskFSInfo(info, TRUE);
+    if (args == NULL) {
+       Put_message("Aborted.");
+       return;
+    }
+    stat = do_mr_query("update_filesys", CountArgs(args), args,
                        NullFunc, NULL);
     switch (stat) {
-    case SMS_NFS:
+    case MR_NFS:
        Put_message("That NFS filesystem is not exported.");
-       if (YesNoQuestion("Fix this now (Y/N)")) {
+       if (YesNoQuestion("Fix this now (Y/N)", TRUE) == TRUE) {
            Do_menu(&nfsphys_menu, 0, NULL);
-           if (YesNoQuestion("Retry filesystem update now (Y/N)")) {
-               if (stat = do_sms_query("update_filesys", CountArgs(args), args,
+           if (YesNoQuestion("Retry filesystem update now (Y/N)", TRUE)
+               == TRUE) {
+               if (stat = do_mr_query("update_filesys", CountArgs(args), args,
                                        NullFunc, NULL))
                    com_err(program_name, stat, " filesystem not updated");
                else
@@ -397,7 +562,7 @@ Bool junk;
            }
        }
        break;
-    case SMS_SUCCESS:
+    case MR_SUCCESS:
        break;
     default:
        com_err(program_name, stat, " in UpdateFS");
@@ -435,32 +600,36 @@ AddFS(argc, argv)
 char **argv;
 int argc;
 {
-    char *info[MAX_ARGS_SIZE], **args;
+    char *info[MAX_ARGS_SIZE], **args, buf[BUFSIZ];
     int stat;
     extern Menu nfsphys_menu;
 
     if ( !ValidName(argv[1]) )
        return(DM_NORMAL);
 
-    if ( (stat = do_sms_query("get_filesys_by_label", 1, argv + 1,
+    if ( (stat = do_mr_query("get_filesys_by_label", 1, argv + 1,
                              NullFunc, NULL)) == 0) {
        Put_message ("A Filesystem by that name already exists.");
        return(DM_NORMAL);
-    } else if (stat != SMS_NO_MATCH) {
+    } else if (stat != MR_NO_MATCH) {
        com_err(program_name, stat, " in AddFS");
        return(DM_NORMAL);
     } 
 
-    args = AskFSInfo(SetDefaults(info, argv[1]), FALSE );
+    if ((args = AskFSInfo(SetDefaults(info, argv[1]), FALSE )) == NULL) {
+       Put_message("Aborted.");
+       return(DM_NORMAL);
+    }
 
-    stat = do_sms_query("add_filesys", CountArgs(args), args, NullFunc, NULL);
+    stat = do_mr_query("add_filesys", CountArgs(args), args, NullFunc, NULL);
     switch (stat) {
-    case SMS_NFS:
+    case MR_NFS:
        Put_message("That NFS filesystem is not exported.");
-       if (YesNoQuestion("Fix this now (Y/N)")) {
+       if (YesNoQuestion("Fix this now (Y/N)", TRUE) == TRUE) {
            Do_menu(&nfsphys_menu, 0, NULL);
-           if (YesNoQuestion("Retry filesystem creation now (Y/N)")) {
-               if (stat = do_sms_query("add_filesys", CountArgs(args), args,
+           if (YesNoQuestion("Retry filesystem creation now (Y/N)", TRUE)
+               == TRUE) {
+               if (stat = do_mr_query("add_filesys", CountArgs(args), args,
                                        NullFunc, NULL))
                    com_err(program_name, stat, " in AddFS");
                else
@@ -468,12 +637,54 @@ int argc;
            }
        }
        break;
-    case SMS_SUCCESS:
+    case MR_SUCCESS:
        break;
     default:
        com_err(program_name, stat, " in AddFS");
     }
 
+    if (stat == MR_SUCCESS && !strcasecmp(info[FS_L_TYPE], "HOMEDIR")) {
+       static char *val[] = {"def_quota", NULL};
+       static char *def_quota = NULL;
+       char *argv[Q_QUOTA + 1];
+       struct qelem *top = NULL;
+
+       if (def_quota == NULL) {
+           stat = do_mr_query("get_value", CountArgs(val), val,
+                               StoreInfo, (char *) &top);
+           if (stat != MR_SUCCESS) {
+               com_err(program_name, stat, " getting default quota");
+           } else {
+               top = QueueTop(top);
+               def_quota = Strsave(((char **)top->q_data)[0]);
+               FreeQueue(top);
+           }
+       }
+       if (def_quota != NULL) {
+           sprintf(buf, "Give user %s a quota of %s on filesys %s (Y/N)",
+                   info[FS_NAME], def_quota, info[FS_NAME]);
+           if (YesNoQuestion(buf, 1) == TRUE) {
+               argv[Q_NAME] = argv[Q_FILESYS] = info[FS_NAME];
+               if (!strcmp(info[FS_TYPE], "NFS"))
+                 argv[Q_TYPE] = "USER";
+               else
+                 argv[Q_TYPE] = "ANY";
+               argv[Q_QUOTA] = def_quota;
+               if ((stat = do_mr_query("add_quota", Q_QUOTA+1, argv, Scream,
+                                        (char *) NULL)) != MR_SUCCESS) {
+                   com_err(program_name, stat, " while adding quota");
+               }
+           }
+       }
+    } else if (stat == MR_SUCCESS) {
+       if (YesNoQuestion("Assign a quota on this filesystem (Y/N)", 1)
+           == TRUE) {
+           parsed_argc = 1;
+           parsed_argv[0] = info[FS_NAME];
+           AddQuota();
+       }
+    }
+
     FreeInfo(info);
     return (DM_NORMAL);
 }
@@ -547,16 +758,16 @@ int argc;
     struct qelem *elem = NULL;
     char buf[BUFSIZ], *args[5], *bufp;
 
-    if ((stat = do_sms_query("get_fsgroup_members", 1, argv+1, StoreInfo,
+    if ((stat = do_mr_query("get_fsgroup_members", 1, argv+1, StoreInfo,
                             (char *)&elem)) != 0) {
-       if (stat != SMS_NO_MATCH)
+       if (stat != MR_NO_MATCH)
          com_err(program_name, stat, " in AddFSToGroup");
     }
     if (elem == NULL) {
        args[0] = argv[1];
        args[1] = argv[2];
        args[2] = "M";
-       stat = do_sms_query("add_filesys_to_fsgroup", 3, args, Scream, NULL);
+       stat = do_mr_query("add_filesys_to_fsgroup", 3, args, Scream, NULL);
        if (stat)
          com_err(program_name, stat, " in AddFSToGroup");
        return(DM_NORMAL);
@@ -566,7 +777,8 @@ int argc;
     Loop(elem, (void *) PrintFSGMembers);
     sprintf(buf, "%d", QueueCount(elem));
     bufp = Strsave(buf);
-    stat = GetValueFromUser("Enter number of filesystem it should follow (0 to make it first):", &bufp);
+    if (GetValueFromUser("Enter number of filesystem it should follow (0 to make it first):", &bufp) == SUB_ERROR)
+      return(DM_NORMAL);
     count = atoi(bufp);
     free(bufp);
     args[2] = SortAfter(elem, count);
@@ -574,8 +786,8 @@ int argc;
     FreeQueue(QueueTop(elem));
     args[0] = argv[1];
     args[1] = argv[2];
-    stat = do_sms_query("add_filesys_to_fsgroup", 3, args, Scream, NULL);
-    if (stat == SMS_EXISTS) {
+    stat = do_mr_query("add_filesys_to_fsgroup", 3, args, Scream, NULL);
+    if (stat == MR_EXISTS) {
        Put_message("That filesystem is already a member of the group.");
        Put_message("Use the order command if you want to change the sorting order.");
     } else if (stat)
@@ -602,7 +814,7 @@ int argc;
     sprintf(buf, "Delete filesystem %s from FS group %s", argv[2], argv[1]);
     if (!Confirm(buf))
       return(DM_NORMAL);
-    if ((stat = do_sms_query("remove_filesys_from_fsgroup", 2, argv+1,
+    if ((stat = do_mr_query("remove_filesys_from_fsgroup", 2, argv+1,
                             Scream, NULL)) != 0) {
        com_err(program_name, stat, ", not removed.");
     }
@@ -621,14 +833,16 @@ ChangeFSGroupOrder(argc, argv)
 char **argv;
 int argc;
 {
-    int stat, src, dst;
-    struct qelem *elem = NULL, *top;
+    int stat, src, dst, i;
+    struct qelem *elem = NULL, *top, *tmpelem;
     char buf[BUFSIZ], *bufp, *args[3];
 
-    if ((stat = do_sms_query("get_fsgroup_members", 1, argv+1, StoreInfo,
+    if ((stat = do_mr_query("get_fsgroup_members", 1, argv+1, StoreInfo,
                             (char *)&elem)) != 0) {
-       if (stat == SMS_NO_MATCH) {
-           sprintf(buf, "Ether %s is not a filesystem group or it has no members", argv[1]);
+       if (stat == MR_NO_MATCH) {
+           sprintf(buf,
+                   "Ether %s is not a filesystem group or it has no members",
+                   argv[1]);
            Put_message(buf);
        } else
          com_err(program_name, stat, " in ChangeFSGroupOrder");
@@ -639,8 +853,9 @@ int argc;
     Loop(top, (void *) PrintFSGMembers);
     while (1) {
        bufp = Strsave("1");
-       stat = GetValueFromUser("Enter number of the filesystem to move:",
-                               &bufp);
+       if (GetValueFromUser("Enter number of the filesystem to move:",
+                            &bufp) == SUB_ERROR)
+         return(DM_NORMAL);
        src = atoi(bufp);
        free(bufp);
        if (src < 0) {
@@ -650,30 +865,45 @@ int argc;
            Put_message("Aborted.");
            return(DM_NORMAL);
        }
-       for (elem = top; src-- > 1 && elem->q_forw; elem = elem->q_forw);
-       if (src > 1) {
+       for (elem = top, i = src; i-- > 1 && elem->q_forw; elem = elem->q_forw);
+       if (i > 0) {
            Put_message("You entered a number that is too high");
            continue;
        }
        break;
     }
-    bufp = Strsave("0");
-    stat = GetValueFromUser("Enter number of filesystem it should follow (0 to make it first):", &bufp);
-    dst = atoi(bufp);
-    free(bufp);
-    if (src == dst || src == dst + 1) {
-       Put_message("That has no effect on the sorting order!");
-       return(DM_NORMAL);
+    while (1) {
+       bufp = Strsave("0");
+       if (GetValueFromUser("Enter number of filesystem it should follow (0 to make it first):", &bufp) == SUB_ERROR)
+         return(DM_NORMAL);
+       dst = atoi(bufp);
+       free(bufp);
+       if (src == dst || src == dst + 1) {
+           Put_message("That has no effect on the sorting order!");
+           return(DM_NORMAL);
+       }
+       if (dst < 0) {
+           Put_message("You must enter a non-negative number.");
+           continue;
+       }
+       for (tmpelem = top, i = dst;
+            i-- > 1 && tmpelem->q_forw;
+            tmpelem = tmpelem->q_forw);
+       if (i > 0) {
+           Put_message("You entered a number that is too high");
+           continue;
+       }
+       break;
     }
     args[2] = SortAfter(top, dst);
     args[0] = argv[1];
     args[1] = ((char **)elem->q_data)[0];
-    if ((stat = do_sms_query("remove_filesys_from_fsgroup", 2, args,
+    if ((stat = do_mr_query("remove_filesys_from_fsgroup", 2, args,
                             Scream, NULL)) != 0) {
        com_err(program_name, stat, " in ChangeFSGroupOrder");
        return(DM_NORMAL);
     }
-    if ((stat = do_sms_query("add_filesys_to_fsgroup", 3, args,
+    if ((stat = do_mr_query("add_filesys_to_fsgroup", 3, args,
                             Scream, NULL)) != 0) {
        com_err(program_name, stat, " in ChangeFSGroupOrder");
     }
@@ -738,7 +968,7 @@ char **argv;
  * print out values, free memory used and then exit.
  */
 
-    if ( (stat = do_sms_query("get_alias", 3, args, StoreInfo, 
+    if ( (stat = do_mr_query("get_alias", 3, args, StoreInfo, 
                              (char *)&elem)) == 0) {
        top = elem = QueueTop(elem);
        while (elem != NULL) {
@@ -751,16 +981,18 @@ char **argv;
        FreeQueue(top);
        return(DM_NORMAL);
     }
-    else if ( stat != SMS_NO_MATCH) {
+    else if ( stat != MR_NO_MATCH) {
        com_err(program_name, stat, " in CreateFSAlias.");
         return(DM_NORMAL);
     }
 
-    args[ALIAS_TRANS]= args[ALIAS_END] = NULL; /* set to NULL initially. */
-    GetValueFromUser("Which filesystem will this alias point to?",
-                    &args[ALIAS_TRANS]);
+    args[ALIAS_TRANS]= Strsave("");
+    args[ALIAS_END] = NULL;
+    if (GetValueFromUser("Which filesystem will this alias point to?",
+                        &args[ALIAS_TRANS]) == SUB_ERROR)
+      return(DM_NORMAL);
 
-    if ( (stat = do_sms_query("add_alias", 3, args, NullFunc, NULL)) != 0)
+    if ( (stat = do_mr_query("add_alias", 3, args, NullFunc, NULL)) != 0)
        com_err(program_name, stat, " in CreateFSAlias.");
 
     FreeInfo(args);
@@ -791,7 +1023,7 @@ Bool one_item;
            "Are you sure that you want to delete the filesystem alias %s",
            info[ALIAS_NAME]);
     if(!one_item || Confirm(temp_buf)) {
-       if ( (stat = do_sms_query("delete_alias", CountArgs(info),
+       if ( (stat = do_mr_query("delete_alias", CountArgs(info),
                                  info, Scream, NULL)) != 0 )
            com_err(program_name, stat, " filesystem alias not deleted.");
        else
This page took 0.063814 seconds and 4 git commands to generate.