]> andersk Git - moira.git/blobdiff - clients/moira/attach.c
Code style cleanup. (No functional changes)
[moira.git] / clients / moira / attach.c
index c3ed591f1fd57b57746c766525446dc5e46cf295..d1a7e2e1619c4643aeea349ed05f0182ca2806f8 100644 (file)
@@ -4,10 +4,10 @@
 
 /*     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. 
- *     
+ *     It Contains: Functions for maintaining data used by Hesiod
+ *                   to map courses/projects/users to their file systems,
+ *                   and maintain filesys info.
+ *
  *     Created:        5/4/88
  *     By:             Chris D. Peterson
  *
@@ -17,7 +17,7 @@
  *     
  *     Copyright 1988 by the Massachusetts Institute of Technology.
  *
- *     For further information on copyright and distribution 
+ *     For further information on copyright and distribution
  *     see the file mit-copyright.h
  */
 
@@ -43,7 +43,6 @@
 #define ALIAS        3
 
 #define NO_MACHINE      ("[NONE]")
-#define NO_MACHINE_BAD  ("[NONE]")
 
 #define DEFAULT_TYPE     ("AFS")
 #define DEFAULT_MACHINE  DEFAULT_NONE
 
 /*     Function Name: SetDefaults
  *     Description: sets the default values for filesystem additions.
- *     Arguments: info - an array of char pointers to recieve defaults. 
+ *     Arguments: info - an array of char pointers to recieve defaults.
  *     Returns: char ** (this array, now filled).
  */
 
-static char ** 
-SetDefaults(info, name)
-char ** info;
-char * name;
+static char **SetDefaults(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);
-    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);
-    info[FS_OWNERS] =   Strsave(DEFAULT_OWNERS);
-    info[FS_CREATE] =   Strsave(DEFAULT_CREATE);
-    info[FS_L_TYPE] =   Strsave(DEFAULT_L_TYPE);
-    info[FS_MODTIME] = info[FS_MODBY] = info[FS_MODWITH] = info[FS_END] = NULL;
-    return(info);
+  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);
+  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);
+  info[FS_OWNERS] =   Strsave(DEFAULT_OWNERS);
+  info[FS_CREATE] =   Strsave(DEFAULT_CREATE);
+  info[FS_L_TYPE] =   Strsave(DEFAULT_L_TYPE);
+  info[FS_MODTIME] = info[FS_MODBY] = info[FS_MODWITH] = info[FS_END] = NULL;
+  return info;
 }
 
 /*     Function Name: GetFSInfo
@@ -92,49 +88,51 @@ char * name;
  *     Returns: a pointer to the first element in the queue.
  */
 
-static struct qelem *
-GetFSInfo(type, name)
-int type;
-char *name;
+static struct qelem *GetFSInfo(int type, char *name)
 {
-    int stat;
-    struct qelem * elem = NULL;
-    char * args[5];
+  int stat;
+  struct qelem *elem = NULL;
+  char *args[5];
 
-    switch (type) {
+  switch (type)
+    {
     case LABEL:
-       if ( (stat = do_mr_query("get_filesys_by_label", 1, &name,
-                                 StoreInfo, (char *)&elem)) != 0) {
-           com_err(program_name, stat, " in GetFSInfo");
-           return(NULL);
+      if ((stat = do_mr_query("get_filesys_by_label", 1, &name,
+                             StoreInfo, (char *)&elem)))
+       {
+         com_err(program_name, stat, " in GetFSInfo");
+         return NULL;
        }
-       break;
+      break;
     case MACHINE:
-       if ( (stat = do_mr_query("get_filesys_by_machine", 1, &name,
-                                 StoreInfo, (char *)&elem)) != 0) {
-           com_err(program_name, stat, " in GetFSInfo");
-           return(NULL);
+      if ((stat = do_mr_query("get_filesys_by_machine", 1, &name,
+                             StoreInfo, (char *)&elem)))
+       {
+         com_err(program_name, stat, " in GetFSInfo");
+         return NULL;
        }
-       break;
+      break;
     case GROUP:
-       if ( (stat = do_mr_query("get_filesys_by_group", 1, &name,
-                                 StoreInfo, (char *)&elem)) != 0) {
-           com_err(program_name, stat, " in GetFSInfo");
-           return(NULL);
+      if ((stat = do_mr_query("get_filesys_by_group", 1, &name,
+                             StoreInfo, (char *)&elem)))
+       {
+         com_err(program_name, stat, " in GetFSInfo");
+         return NULL;
        }
-       break;
+      break;
     case ALIAS:
-       args[ALIAS_NAME] = name;
-       args[ALIAS_TYPE] = FS_ALIAS_TYPE;
-       args[ALIAS_TRANS] = "*";
-       if ( (stat = do_mr_query("get_alias", 3, args, StoreInfo, 
-                                 (char *) &elem)) != 0) {
-           com_err(program_name, stat, " in get_alias.");
-           return(NULL);
+      args[ALIAS_NAME] = name;
+      args[ALIAS_TYPE] = FS_ALIAS_TYPE;
+      args[ALIAS_TRANS] = "*";
+      if ((stat = do_mr_query("get_alias", 3, args, StoreInfo,
+                             (char *) &elem)))
+       {
+         com_err(program_name, stat, " in get_alias.");
+         return NULL;
        }
     }
 
-    return(QueueTop(elem));
+  return QueueTop(elem);
 }
 
 /*     Function Name: PrintFSAlias
@@ -143,29 +141,26 @@ char *name;
  *     Returns: the name of the filesys - used be QueryLoop().
  */
 
-static char *
-PrintFSAlias(info)
-char ** info;
+static char *PrintFSAlias(char **info)
 {
-    char buf[BUFSIZ];
+  char buf[BUFSIZ];
 
-    sprintf(buf,"Alias: %-25s Filesystem: %s",info[ALIAS_NAME], 
-           info[ALIAS_TRANS]);
-    Put_message(buf);
-    return(info[ALIAS_NAME]);
+  sprintf(buf, "Alias: %-25s Filesystem: %s", info[ALIAS_NAME],
+         info[ALIAS_TRANS]);
+  Put_message(buf);
+  return info[ALIAS_NAME];
 }
 
 static int fsgCount = 1;
 
-static char *
-PrintFSGMembers(info)
-char ** info;
+static char *PrintFSGMembers(char **info)
 {
-    char print_buf[BUFSIZ];
+  char print_buf[BUFSIZ];
 
-    sprintf(print_buf, "  %d. Filesystem: %-32s (sort key: %s)", fsgCount++, info[0], info[1]);
-    Put_message(print_buf);
-    return(info[0]);
+  sprintf(print_buf, "  %d. Filesystem: %-32s (sort key: %s)",
+         fsgCount++, info[0], info[1]);
+  Put_message(print_buf);
+  return info[0];
 }
 
 
@@ -175,96 +170,96 @@ char ** info;
  *     Returns: none.
  */
 
-static char *
-PrintFSInfo(info)
-char ** info;
+static char *PrintFSInfo(char **info)
 {
-    char print_buf[BUFSIZ];
-
-    FORMFEED;
-
-    if (!strcmp(info[FS_TYPE], "FSGROUP") || !strcmp(info[FS_TYPE], "MUL")) {
-       int stat;
-       struct qelem *elem = NULL;
-
-       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_mr_query("get_fsgroup_members", 1, &info[FS_NAME],
-                                StoreInfo, (char *)&elem)) != 0) {
-           if (stat == MR_NO_MATCH)
-             Put_message("    [no members]");
-           else
-             com_err(program_name, stat, " in PrintFSInfo");
-       } else {
-           fsgCount = 1;
-           Loop(QueueTop(elem), (void *) PrintFSGMembers);
-           FreeQueue(elem);
+  char print_buf[BUFSIZ];
+
+  FORMFEED;
+
+  if (!strcmp(info[FS_TYPE], "FSGROUP") || !strcmp(info[FS_TYPE], "MUL"))
+    {
+      int stat;
+      struct qelem *elem = NULL;
+
+      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_mr_query("get_fsgroup_members", 1, &info[FS_NAME],
+                             StoreInfo, (char *)&elem)))
+       {
+         if (stat == MR_NO_MATCH)
+           Put_message("    [no members]");
+         else
+           com_err(program_name, stat, " in PrintFSInfo");
+       }
+      else
+       {
+         fsgCount = 1;
+         Loop(QueueTop(elem), 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]);
-       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, "Update Fileserver: %-27s Locker Type: %s",
-               atoi(info[FS_CREATE]) ? "ON" : "OFF", 
-               info[FS_L_TYPE]);
-       Put_message(print_buf);
-       sprintf(print_buf, MOD_FORMAT, info[FS_MODBY], info[FS_MODTIME], 
-               info[FS_MODWITH]);
-       Put_message(print_buf);
     }
-    return(info[FS_NAME]);
+  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]);
+      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, "Update Fileserver: %-27s Locker Type: %s",
+             atoi(info[FS_CREATE]) ? "ON" : "OFF", info[FS_L_TYPE]);
+      Put_message(print_buf);
+      sprintf(print_buf, MOD_FORMAT, info[FS_MODBY], info[FS_MODTIME],
+             info[FS_MODWITH]);
+      Put_message(print_buf);
+    }
+  return info[FS_NAME];
 }
 
 
-char *canonicalize_cell(c)
-char *c;
+char *canonicalize_cell(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));
+  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;
+int GetAliasValue(int argc, char **argv, char **retval)
 {
-    *retval = strsave(argv[2]);
-    return(MR_CONT);
+  *retval = strsave(argv[2]);
+  return MR_CONT;
 }
 
 /*     Function Name: AskFSInfo.
- *     Description: This function askes the user for information about a 
+ *     Description: This function askes the user for information about a
  *                   machine and saves it into a structure.
  *     Arguments: info - a pointer the the structure to put the
  *                             info into.
@@ -272,152 +267,178 @@ char **retval;
  *     Returns: none.
  */
 
-static char **
-AskFSInfo(info, name)
-char ** info;
-Bool name;
+static char **AskFSInfo(char **info, Bool name)
 {
-    char temp_buf[BUFSIZ], *newname, access_type[32];
-    int fsgroup = 0, newdefaults = 0;
-
-    Put_message("");
-    sprintf(temp_buf, "Changing Attributes of filesystem %s.", 
-           info[FS_NAME]);
-    Put_message(temp_buf);
-    Put_message("");
-
-    if (name) {
-       newname = Strsave(info[FS_NAME]);
-       if (GetValueFromUser("The new name for this filesystem",
-                            &newname) == SUB_ERROR)
-         return(NULL);
+  char temp_buf[BUFSIZ], *newname, access_type[32];
+  int fsgroup = 0, newdefaults = 0;
+
+  Put_message("");
+  sprintf(temp_buf, "Changing Attributes of filesystem %s.", info[FS_NAME]);
+  Put_message(temp_buf);
+  Put_message("");
+
+  if (name)
+    {
+      newname = Strsave(info[FS_NAME]);
+      if (GetValueFromUser("The new name for this filesystem",
+                          &newname) == SUB_ERROR)
+       return NULL;
     }
 
-    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 {
-       if (!strcasecmp(info[FS_TYPE], "AFS")) {
-           if (!name || newdefaults) {
-               free(info[FS_MACHINE]);
-               info[FS_MACHINE] = strsave(DEFAULT_CELL);
+  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
+    {
+      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]) ==
+         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]);
+           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) {
-       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 = strchr(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;
+  if (!fsgroup)
+    {
+      if (!strcasecmp(info[FS_TYPE], "AFS"))
+       {
+         char *path, *args[3], *p;
+         int status, depth;
+         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 = strchr(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 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 {
+                       }
+                     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]);
+                 strcat(temp_buf, "/");
+                 strcat(temp_buf, info[FS_NAME]);
+                 free(path);
                }
-               info[FS_PACK] = strsave(temp_buf);
+             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);
+      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;
     }
-    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)
-      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);
+  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)
+    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);
-    FreeAndClear(&info[FS_MODWITH], TRUE);
+  FreeAndClear(&info[FS_MODTIME], TRUE);
+  FreeAndClear(&info[FS_MODBY], TRUE);
+  FreeAndClear(&info[FS_MODWITH], TRUE);
 
-    if (name)                  /* slide the newname into the #2 slot. */
-       SlipInNewName(info, newname);
+  if (name)                    /* slide the newname into the #2 slot. */
+    SlipInNewName(info, newname);
 
-    return(info);
+  return info;
 }
 
 
@@ -429,18 +450,14 @@ Bool name;
  *     Returns: DM_NORMAL.
  */
 
-/* ARGSUSED */
-int
-GetFS(argc, argv)
-int argc;
-char **argv;
+int GetFS(int argc, char **argv)
 {
-    struct qelem *top;
+  struct qelem *top;
 
-    top = GetFSInfo(LABEL, argv[1]); /* get info. */
-    Loop(top, (void *) PrintFSInfo);
-    FreeQueue(top);            /* clean the queue. */
-    return (DM_NORMAL);
+  top = GetFSInfo(LABEL, argv[1]); /* get info. */
+  Loop(top, PrintFSInfo);
+  FreeQueue(top);              /* clean the queue. */
+  return DM_NORMAL;
 }
 
 /*     Function Name: GetFSM
@@ -449,52 +466,46 @@ char **argv;
  *     Returns: DM_NORMAL.
  */
 
-/* ARGSUSED */
-int
-GetFSM(argc, argv)
-int argc;
-char **argv;
+int GetFSM(int argc, char **argv)
 {
-    struct qelem *top;
+  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);
+  argv[1] = canonicalize_hostname(strsave(argv[1]));
+  top = GetFSInfo(MACHINE, argv[1]); /* get info. */
+  Loop(top, 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.
- *                 one_item - a Boolean that is true if only one item 
+ *                 one_item - a Boolean that is true if only one item
  *                              in queue that dumped us here.
  *     Returns: none.
  */
 
-void
-RealDeleteFS(info, one_item)
-char ** info;
-Bool one_item;
+void RealDeleteFS(char **info, Bool one_item)
 {
-    int stat;
-    char temp_buf[BUFSIZ];
-
-/* 
- * Deletetions are  performed if the user hits 'y' on a list of multiple 
- * filesystem, or if the user confirms on a unique alias.
- */
-    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_mr_query("delete_filesys", 1,
-                                 &info[FS_NAME], Scream, NULL)) != 0)
-           com_err(program_name, stat, " filesystem not deleted.");
-       else
-           Put_message("Filesystem deleted.");
+  int stat;
+  char temp_buf[BUFSIZ];
+
+  /*
+   * Deletions are performed if the user hits 'y' on a list of multiple
+   * filesystem, or if the user confirms on a unique alias.
+   */
+  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_mr_query("delete_filesys", 1,
+                             &info[FS_NAME], Scream, NULL)))
+       com_err(program_name, stat, " filesystem not deleted.");
+      else
+       Put_message("Filesystem deleted.");
     }
-    else 
-       Put_message("Filesystem not deleted.");
+  else
+    Put_message("Filesystem not deleted.");
 }
 
 /*     Function Name: DeleteFS
@@ -503,63 +514,56 @@ Bool one_item;
  *     Returns: none.
  */
 
-/* ARGSUSED */
-int
-DeleteFS(argc, argv)
-int argc;
-char **argv;
+int DeleteFS(int argc, char **argv)
 {
-    struct qelem *elem = GetFSInfo(LABEL, argv[1]);
-    QueryLoop(elem, PrintFSInfo, RealDeleteFS, "Delete the Filesystem");
+  struct qelem *elem = GetFSInfo(LABEL, argv[1]);
+  QueryLoop(elem, PrintFSInfo, RealDeleteFS, "Delete the Filesystem");
 
-    FreeQueue(elem);
-    return (DM_NORMAL);
+  FreeQueue(elem);
+  return DM_NORMAL;
 }
 
 /*     Function Name: RealChangeFS
  *     Description: performs the actual change to the filesys.
- *     Arguments: info - the information 
+ *     Arguments: info - the information
  *                 junk - an unused boolean.
  *     Returns: none.
  */
 
-/* ARGSUSED. */
-static void
-RealChangeFS(info, junk)
-char ** info;
-Bool junk;
+static void RealChangeFS(char **info, Bool junk)
 {
-    int stat;
-    char ** args;
-    extern Menu nfsphys_menu;
-
-    args = AskFSInfo(info, TRUE);
-    if (args == NULL) {
-       Put_message("Aborted.");
-       return;
+  int stat;
+  char **args;
+  extern Menu nfsphys_menu;
+
+  args = AskFSInfo(info, TRUE);
+  if (!args)
+    {
+      Put_message("Aborted.");
+      return;
     }
-    stat = do_mr_query("update_filesys", CountArgs(args), args,
-                       NullFunc, NULL);
-    switch (stat) {
+  stat = do_mr_query("update_filesys", CountArgs(args), args, NullFunc, NULL);
+  switch (stat)
+    {
     case MR_NFS:
-       Put_message("That NFS filesystem is not exported.");
-       if (YesNoQuestion("Fix this now (Y/N)", TRUE) == TRUE) {
-           Do_menu(&nfsphys_menu, 0, NULL);
-           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
-                   Put_message("filesystem sucessfully updated.");
+      Put_message("That NFS filesystem is not exported.");
+      if (YesNoQuestion("Fix this now (Y/N)", TRUE) == TRUE)
+       {
+         Do_menu(&nfsphys_menu, 0, NULL);
+         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
+               Put_message("filesystem sucessfully updated.");
            }
        }
-       break;
+      break;
     case MR_SUCCESS:
-       break;
+      break;
     default:
-       com_err(program_name, stat, " in UpdateFS");
+      com_err(program_name, stat, " in UpdateFS");
     }
 }
 
@@ -569,17 +573,13 @@ Bool junk;
  *     Returns: DM_NORMAL.
  */
 
-/* ARGSUSED */
-int
-ChangeFS(argc, argv)
-char **argv;
-int argc;
+int ChangeFS(int argc, char **argv)
 {
-    struct qelem *elem = GetFSInfo(LABEL, argv[1]);
-    QueryLoop(elem, NullPrint, RealChangeFS, "Update the Filesystem");
+  struct qelem *elem = GetFSInfo(LABEL, argv[1]);
+  QueryLoop(elem, NullPrint, RealChangeFS, "Update the Filesystem");
 
-    FreeQueue(elem);
-    return (DM_NORMAL);
+  FreeQueue(elem);
+  return DM_NORMAL;
 }
 
 /*     Function Name: AddFS
@@ -588,152 +588,157 @@ int argc;
  *     Returns: DM_NORMAL.
  */
 
-/* ARGSUSED */
-int
-AddFS(argc, argv)
-char **argv;
-int argc;
+int AddFS(int argc, char **argv)
 {
-    char *info[MAX_ARGS_SIZE], **args, buf[BUFSIZ];
-    int stat;
-    extern Menu nfsphys_menu;
-
-    if ( !ValidName(argv[1]) )
-       return(DM_NORMAL);
-
-    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 != MR_NO_MATCH) {
-       com_err(program_name, stat, " in AddFS");
-       return(DM_NORMAL);
-    } 
-
-    if ((args = AskFSInfo(SetDefaults(info, argv[1]), FALSE )) == NULL) {
-       Put_message("Aborted.");
-       return(DM_NORMAL);
+  char *info[MAX_ARGS_SIZE], **args, buf[BUFSIZ];
+  int stat;
+  extern Menu nfsphys_menu;
+
+  if (!ValidName(argv[1]))
+    return DM_NORMAL;
+
+  if (!(stat = do_mr_query("get_filesys_by_label", 1, argv + 1,
+                             NullFunc, NULL)))
+    {
+      Put_message ("A Filesystem by that name already exists.");
+      return DM_NORMAL;
+    }
+  else if (stat != MR_NO_MATCH)
+    {
+      com_err(program_name, stat, " in AddFS");
+      return DM_NORMAL;
+    }
+
+  if (!(args = AskFSInfo(SetDefaults(info, argv[1]), FALSE)))
+    {
+      Put_message("Aborted.");
+      return DM_NORMAL;
     }
 
-    stat = do_mr_query("add_filesys", CountArgs(args), args, NullFunc, NULL);
-    switch (stat) {
+  stat = do_mr_query("add_filesys", CountArgs(args), args, NullFunc, NULL);
+  switch (stat)
+    {
     case MR_NFS:
-       Put_message("That NFS filesystem is not exported.");
-       if (YesNoQuestion("Fix this now (Y/N)", TRUE) == TRUE) {
-           Do_menu(&nfsphys_menu, 0, NULL);
-           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
-                   Put_message("Created.");
-           }
+      Put_message("That NFS filesystem is not exported.");
+      if (YesNoQuestion("Fix this now (Y/N)", TRUE) == TRUE)
+       {
+         Do_menu(&nfsphys_menu, 0, NULL);
+         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
+             Put_message("Created.");
+         }
        }
-       break;
+      break;
     case MR_SUCCESS:
-       break;
+      break;
     default:
-       com_err(program_name, stat, " in AddFS");
+      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 (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)
+       {
+         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");
-               }
+      if (def_quota)
+       {
+         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,
+                                     NULL)))
+               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();
+    }
+  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);
+  FreeInfo(info);
+  return DM_NORMAL;
 }
 
 /*     Function Name: SortAfter
- *     Description: choose a sortkey to cause an item to be added after 
+ *     Description: choose a sortkey to cause an item to be added after
  *             the count element in the queue
  *     Arguments: queue of filesys names & sortkeys, queue count pointer
  *     Returns: sort key to use.
  */
 
-/* ARGSUSED */
-char *
-SortAfter(elem, count)
-struct qelem *elem;
-int count;
+char *SortAfter(struct qelem *elem, int count)
 {
-    char *prev, *next, prevnext, *key, keybuf[9];
-
-    /* first find the two keys we need to insert between */
-    prev = "A";
-    for (; count > 0; count--) {
-       prev = ((char **)elem->q_data)[1];
-       if (elem->q_forw)
-         elem = elem->q_forw;
-       else
-         break;
-    }
-    if (count > 0)
-      next = "Z";
-    else
-      next = ((char **)elem->q_data)[1];
-
-    /* now copy the matching characters */
-    for (key = keybuf; *prev && *prev == *next; next++) {
-       *key++ = *prev++;
+  char *prev, *next, prevnext, *key, keybuf[9];
+
+  /* first find the two keys we need to insert between */
+  prev = "A";
+  for (; count > 0; count--)
+    {
+      prev = ((char **)elem->q_data)[1];
+      if (elem->q_forw)
+       elem = elem->q_forw;
+      else
+       break;
     }
-
-    /* and set the last character */
-    if (*prev == 0)
-      *prev = prevnext = 'A';
-    else
-      prevnext = prev[1];
-    if (prevnext == 0)
-      prevnext = 'A';
-    if (*next == 0)
-      *next = 'Z';
-    if (*next - *prev > 1) {
-       *key++ = (*next + *prev)/2;
-    } else {
-       *key++ = *prev;
-       *key++ = (prevnext + 'Z')/2;
+  if (count > 0)
+    next = "Z";
+  else
+    next = ((char **)elem->q_data)[1];
+
+  /* now copy the matching characters */
+  for (key = keybuf; *prev && *prev == *next; next++)
+    *key++ = *prev++;
+
+  /* and set the last character */
+  if (*prev == 0)
+    *prev = prevnext = 'A';
+  else
+    prevnext = prev[1];
+  if (prevnext == 0)
+    prevnext = 'A';
+  if (*next == 0)
+    *next = 'Z';
+  if (*next - *prev > 1)
+    *key++ = (*next + *prev) / 2;
+  else
+    {
+      *key++ = *prev;
+      *key++ = (prevnext + 'Z') / 2;
     }
-    *key = 0;
-    return(Strsave(keybuf));
+  *key = 0;
+  return Strsave(keybuf);
 }
 
 /*     Function Name: AddFSToGroup
@@ -742,51 +747,53 @@ int count;
  *     Returns: DM_NORMAL.
  */
 
-/* ARGSUSED */
-int
-AddFSToGroup(argc, argv)
-char **argv;
-int argc;
+int AddFSToGroup(int argc, char **argv)
 {
-    int stat, count;
-    struct qelem *elem = NULL;
-    char buf[BUFSIZ], *args[5], *bufp;
-
-    if ((stat = do_mr_query("get_fsgroup_members", 1, argv+1, StoreInfo,
-                            (char *)&elem)) != 0) {
-       if (stat != MR_NO_MATCH)
-         com_err(program_name, stat, " in AddFSToGroup");
+  int stat, count;
+  struct qelem *elem = NULL;
+  char buf[BUFSIZ], *args[5], *bufp;
+
+  if ((stat = do_mr_query("get_fsgroup_members", 1, argv + 1, StoreInfo,
+                         (char *)&elem)))
+    {
+      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_mr_query("add_filesys_to_fsgroup", 3, args, Scream, NULL);
-       if (stat)
-         com_err(program_name, stat, " in AddFSToGroup");
-       return(DM_NORMAL);
+  if (elem == NULL)
+    {
+      args[0] = argv[1];
+      args[1] = argv[2];
+      args[2] = "M";
+      stat = do_mr_query("add_filesys_to_fsgroup", 3, args, Scream, NULL);
+      if (stat)
+       com_err(program_name, stat, " in AddFSToGroup");
+      return DM_NORMAL;
     }
-    elem = QueueTop(elem);
-    fsgCount = 1;
-    Loop(elem, (void *) PrintFSGMembers);
-    sprintf(buf, "%d", QueueCount(elem));
-    bufp = Strsave(buf);
-    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);
-
-    FreeQueue(QueueTop(elem));
-    args[0] = argv[1];
-    args[1] = argv[2];
-    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)
-      com_err(program_name, stat, " in AddFSToGroup");
-    return(DM_NORMAL);
+  elem = QueueTop(elem);
+  fsgCount = 1;
+  Loop(elem, PrintFSGMembers);
+  sprintf(buf, "%d", QueueCount(elem));
+  bufp = Strsave(buf);
+  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);
+
+  FreeQueue(QueueTop(elem));
+  args[0] = argv[1];
+  args[1] = argv[2];
+  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)
+    com_err(program_name, stat, " in AddFSToGroup");
+  return DM_NORMAL;
 }
 
 
@@ -796,23 +803,18 @@ int argc;
  *     Returns: DM_NORMAL.
  */
 
-/* ARGSUSED */
-int
-RemoveFSFromGroup(argc, argv)
-char **argv;
-int argc;
+int RemoveFSFromGroup(int argc, char **argv)
 {
-    int stat;
-    char buf[BUFSIZ];
-
-    sprintf(buf, "Delete filesystem %s from FS group %s", argv[2], argv[1]);
-    if (!Confirm(buf))
-      return(DM_NORMAL);
-    if ((stat = do_mr_query("remove_filesys_from_fsgroup", 2, argv+1,
-                            Scream, NULL)) != 0) {
-       com_err(program_name, stat, ", not removed.");
-    }
-    return(DM_NORMAL);
+  int stat;
+  char buf[BUFSIZ];
+
+  sprintf(buf, "Delete filesystem %s from FS group %s", argv[2], argv[1]);
+  if (!Confirm(buf))
+    return DM_NORMAL;
+  if ((stat = do_mr_query("remove_filesys_from_fsgroup", 2, argv + 1,
+                         Scream, NULL)))
+    com_err(program_name, stat, ", not removed.");
+  return DM_NORMAL;
 }
 
 /*     Function Name: ChangeFSGroupOrder
@@ -821,87 +823,97 @@ int argc;
  *     Returns: DM_NORMAL.
  */
 
-/* ARGSUSED */
-int
-ChangeFSGroupOrder(argc, argv)
-char **argv;
-int argc;
+int ChangeFSGroupOrder(int argc, char **argv)
 {
-    int stat, src, dst, i;
-    struct qelem *elem = NULL, *top, *tmpelem;
-    char buf[BUFSIZ], *bufp, *args[3];
-
-    if ((stat = do_mr_query("get_fsgroup_members", 1, argv+1, StoreInfo,
-                            (char *)&elem)) != 0) {
-       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");
-       return(DM_NORMAL);
+  int stat, src, dst, i;
+  struct qelem *elem = NULL, *top, *tmpelem;
+  char buf[BUFSIZ], *bufp, *args[3];
+
+  if ((stat = do_mr_query("get_fsgroup_members", 1, argv + 1, StoreInfo,
+                         (char *)&elem)))
+    {
+      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");
+      return DM_NORMAL;
     }
-    top = QueueTop(elem);
-    fsgCount = 1;
-    Loop(top, (void *) PrintFSGMembers);
-    while (1) {
-       bufp = Strsave("1");
-       if (GetValueFromUser("Enter number of the filesystem to move:",
-                            &bufp) == SUB_ERROR)
-         return(DM_NORMAL);
-       src = atoi(bufp);
-       free(bufp);
-       if (src < 0) {
-           Put_message("You must enter a positive number (or 0 to abort).");
-           continue;
-       } else if (src == 0) {
-           Put_message("Aborted.");
-           return(DM_NORMAL);
+  top = QueueTop(elem);
+  fsgCount = 1;
+  Loop(top, PrintFSGMembers);
+  while (1)
+    {
+      bufp = Strsave("1");
+      if (GetValueFromUser("Enter number of the filesystem to move:",
+                          &bufp) == SUB_ERROR)
+       return DM_NORMAL;
+      src = atoi(bufp);
+      free(bufp);
+      if (src < 0)
+       {
+         Put_message("You must enter a positive number (or 0 to abort).");
+         continue;
        }
-       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;
+      else if (src == 0)
+       {
+         Put_message("Aborted.");
+         return DM_NORMAL;
        }
-       break;
+      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;
     }
-    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);
+  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;
+      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;
+      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;
+      break;
     }
-    args[2] = SortAfter(top, dst);
-    args[0] = argv[1];
-    args[1] = ((char **)elem->q_data)[0];
-    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_mr_query("add_filesys_to_fsgroup", 3, args,
-                            Scream, NULL)) != 0) {
-       com_err(program_name, stat, " in ChangeFSGroupOrder");
+  args[2] = SortAfter(top, dst);
+  args[0] = argv[1];
+  args[1] = ((char **)elem->q_data)[0];
+  if ((stat = do_mr_query("remove_filesys_from_fsgroup", 2, args,
+                         Scream, NULL)))
+    {
+      com_err(program_name, stat, " in ChangeFSGroupOrder");
+      return DM_NORMAL;
     }
-    return(DM_NORMAL);
+  if ((stat = do_mr_query("add_filesys_to_fsgroup", 3, args,
+                            Scream, NULL)))
+    com_err(program_name, stat, " in ChangeFSGroupOrder");
+  return DM_NORMAL;
 }
 
 
@@ -915,19 +927,15 @@ int argc;
  *             this will work correctly.
  */
 
-/* ARGSUSED */
-int
-GetFSAlias(argc, argv)
-int argc;
-char **argv;
+int GetFSAlias(int argc, char **argv)
 {
-    struct qelem *top;
+  struct qelem *top;
 
-    top = GetFSInfo(ALIAS, argv[1]);
-    Put_message(" ");          /* blank line. */
-    Loop(top, (void *) PrintFSAlias);
-    FreeQueue(top);
-    return(DM_NORMAL);
+  top = GetFSInfo(ALIAS, argv[1]);
+  Put_message(" ");            /* blank line. */
+  Loop(top, (void *) PrintFSAlias);
+  FreeQueue(top);
+  return DM_NORMAL;
 }
 
 /*     Function Name: CreateFSAlias
@@ -935,96 +943,93 @@ char **argv;
  *     Arguments: argc, argv - name of alias in argv[1].
  *     Returns: DM_NORMAL.
  *      NOTES:  This requires (name, type, transl)  I get {name, translation}
- *              from the user.  I provide type, which is well-known. 
+ *              from the user.  I provide type, which is well-known.
  */
 
-/* ARGSUSED */
-int
-CreateFSAlias(argc, argv)
-int argc;
-char **argv;
+int CreateFSAlias(int argc, char **argv)
 {
-    register int stat;
-    struct qelem *elem, *top;
-    char *args[MAX_ARGS_SIZE], buf[BUFSIZ], **info;
-
-    elem = NULL;
-
-    if (!ValidName(argv[1]))
-       return(DM_NORMAL);
-
-    args[ALIAS_NAME] = Strsave(argv[1]);
-    args[ALIAS_TYPE] = Strsave(FS_ALIAS_TYPE);
-    args[ALIAS_TRANS] = Strsave("*");
-
-/*
- * Check to see if this alias already exists in the database, if so then
- * print out values, free memory used and then exit.
- */
-
-    if ( (stat = do_mr_query("get_alias", 3, args, StoreInfo, 
-                             (char *)&elem)) == 0) {
-       top = elem = QueueTop(elem);
-       while (elem != NULL) {
-           info = (char **) elem->q_data;          
-           sprintf(buf,"The alias: %s currently describes the filesystem %s",
-                   info[ALIAS_NAME], info[ALIAS_TRANS]);
-           Put_message(buf);
-           elem = elem->q_forw;
+  register int stat;
+  struct qelem *elem, *top;
+  char *args[MAX_ARGS_SIZE], buf[BUFSIZ], **info;
+
+  elem = NULL;
+
+  if (!ValidName(argv[1]))
+    return DM_NORMAL;
+
+  args[ALIAS_NAME] = Strsave(argv[1]);
+  args[ALIAS_TYPE] = Strsave(FS_ALIAS_TYPE);
+  args[ALIAS_TRANS] = Strsave("*");
+
+  /*
+   * Check to see if this alias already exists in the database, if so then
+   * print out values, free memory used and then exit.
+   */
+
+  if (!(stat = do_mr_query("get_alias", 3, args, StoreInfo,
+                          (char *)&elem)))
+    {
+      top = elem = QueueTop(elem);
+      while (elem)
+       {
+         info = (char **) elem->q_data;
+         sprintf(buf, "The alias: %s currently describes the filesystem %s",
+                 info[ALIAS_NAME], info[ALIAS_TRANS]);
+         Put_message(buf);
+         elem = elem->q_forw;
        }
-       FreeQueue(top);
-       return(DM_NORMAL);
+      FreeQueue(top);
+      return DM_NORMAL;
     }
-    else if ( stat != MR_NO_MATCH) {
-       com_err(program_name, stat, " in CreateFSAlias.");
-        return(DM_NORMAL);
+  else if (stat != MR_NO_MATCH)
+    {
+      com_err(program_name, stat, " in CreateFSAlias.");
+      return DM_NORMAL;
     }
 
-    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);
+  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_mr_query("add_alias", 3, args, NullFunc, NULL)) != 0)
-       com_err(program_name, stat, " in CreateFSAlias.");
+  if ((stat = do_mr_query("add_alias", 3, args, NullFunc, NULL)))
+    com_err(program_name, stat, " in CreateFSAlias.");
 
-    FreeInfo(args);
-    return (DM_NORMAL);
+  FreeInfo(args);
+  return DM_NORMAL;
 }
-    
+
 /*     Function Name: RealDeleteFSAlias
  *     Description: Does the real deletion work.
  *     Arguments: info - array of char *'s containing all useful info.
- *                 one_item - a Boolean that is true if only one item 
+ *                 one_item - a Boolean that is true if only one item
  *                              in queue that dumped us here.
  *     Returns: none.
  */
 
-void
-RealDeleteFSAlias(info, one_item)
-char ** info;
-Bool one_item;
+void RealDeleteFSAlias(char **info, Bool one_item)
 {
-    int stat;
-    char temp_buf[BUFSIZ];
-
-/* 
- * Deletetions are  performed if the user hits 'y' on a list of multiple 
- * filesystem, or if the user confirms on a unique alias.
- */
-    sprintf(temp_buf, 
-           "Are you sure that you want to delete the filesystem alias %s",
-           info[ALIAS_NAME]);
-    if(!one_item || Confirm(temp_buf)) {
-       if ( (stat = do_mr_query("delete_alias", CountArgs(info),
-                                 info, Scream, NULL)) != 0 )
-           com_err(program_name, stat, " filesystem alias not deleted.");
-       else
-           Put_message("Filesystem alias deleted.");
+  int stat;
+  char temp_buf[BUFSIZ];
+
+  /*
+   * Deletions are  performed if the user hits 'y' on a list of multiple
+   * filesystem, or if the user confirms on a unique alias.
+   */
+  sprintf(temp_buf,
+         "Are you sure that you want to delete the filesystem alias %s",
+         info[ALIAS_NAME]);
+  if (!one_item || Confirm(temp_buf))
+    {
+      if ((stat = do_mr_query("delete_alias", CountArgs(info),
+                             info, Scream, NULL)))
+       com_err(program_name, stat, " filesystem alias not deleted.");
+      else
+       Put_message("Filesystem alias deleted.");
     }
-    else 
-       Put_message("Filesystem alias not deleted.");
+  else
+    Put_message("Filesystem alias not deleted.");
 }
 
 /*     Function Name: DeleteFSAlias
@@ -1032,20 +1037,16 @@ Bool one_item;
  *     Arguments: argc, argv - name of alias in argv[1].
  *     Returns: DM_NORMAL.
  *      NOTES:  This requires (name, type, transl)  I get {name, translation}
- *              from the user.  I provide type, which is well-known. 
+ *              from the user.  I provide type, which is well-known.
  */
 
-/* ARGSUSED */
-int
-DeleteFSAlias(argc, argv)
-int argc;
-char **argv;
+int DeleteFSAlias(int argc, char **argv)
 {
-    struct qelem *elem = GetFSInfo(ALIAS, argv[1]);
-    QueryLoop(elem, PrintFSAlias, RealDeleteFSAlias,
-             "Delete the Filesystem Alias");
-    FreeQueue(elem);
-    return (DM_NORMAL);
+  struct qelem *elem = GetFSInfo(ALIAS, argv[1]);
+  QueryLoop(elem, PrintFSAlias, RealDeleteFSAlias,
+           "Delete the Filesystem Alias");
+  FreeQueue(elem);
+  return DM_NORMAL;
 }
 
 /*     Function Name: AttachHelp
@@ -1054,24 +1055,23 @@ char **argv;
  *     Returns: DM_NORMAL.
  */
 
-int
-AttachHelp()
+int AttachHelp(void)
 {
-    static char *message[] = {
-      "These are the options:",
-      "",
-      "get - get information about a filesystem.",
-      "add - add a new filesystem to the data base.",
-      "update - update the information in the database on a filesystem.",
-      "delete - delete a filesystem from the database.",
-      "check - check information about association of a name and a filesys.",
-      "alias - associate a name with a filesystem.",
-      "unalias - disassociate a name with a filesystem.",
-      "verbose - toggle the request for delete confirmation.",
-       NULL,
-    };
-
-    return(PrintHelp(message));
+  static char *message[] = {
+    "These are the options:",
+    "",
+    "get - get information about a filesystem.",
+    "add - add a new filesystem to the data base.",
+    "update - update the information in the database on a filesystem.",
+    "delete - delete a filesystem from the database.",
+    "check - check information about association of a name and a filesys.",
+    "alias - associate a name with a filesystem.",
+    "unalias - disassociate a name with a filesystem.",
+    "verbose - toggle the request for delete confirmation.",
+    NULL,
+  };
+
+  return PrintHelp(message);
 }
 
 /*     Function Name: FSGroupHelp
@@ -1080,21 +1080,20 @@ AttachHelp()
  *     Returns: DM_NORMAL.
  */
 
-int
-FSGroupHelp()
+int FSGroupHelp(void)
 {
-    static char *message[] = {
-       "A filesystem group is a named sorted list of filesystems.",
-       "",
-       "To create, modify, or delete a group itself, use the menu above",
-       "  this one, and manipulate a filesystem of type FSGROUP.",
-       "Options here are:",
-       "  get - get info about a group and show its members",
-       "  add - add a new member to a group.",
-       "  remove - remove a member from a group.",
-       "  order - change the sorting order of a group.",
-       NULL
-    };
-
-    return(PrintHelp(message));
+  static char *message[] = {
+    "A filesystem group is a named sorted list of filesystems.",
+    "",
+    "To create, modify, or delete a group itself, use the menu above",
+    "  this one, and manipulate a filesystem of type FSGROUP.",
+    "Options here are:",
+    "  get - get info about a group and show its members",
+    "  add - add a new member to a group.",
+    "  remove - remove a member from a group.",
+    "  order - change the sorting order of a group.",
+    NULL
+  };
+
+  return PrintHelp(message);
 }
This page took 0.112411 seconds and 4 git commands to generate.