libadmin: minor warning fix
[openafs.git] / src / libadmin / test / bos.c
index 790d6c4..22cf926 100644 (file)
@@ -14,8 +14,9 @@
 #include <afsconfig.h>
 #include <afs/param.h>
 
-RCSID
-    ("$Header$");
+#include <roken.h>
+
+#include <ctype.h>
 
 #include "bos.h"
 
@@ -63,10 +64,10 @@ static char *day[] = {
     "sat"
 };
 
-static
+static int
 LocalFreeTokens(struct token *alist)
 {
-    register struct token *nlist;
+    struct token *nlist;
     for (; alist; alist = nlist) {
        nlist = alist->next;
        free(alist->key);
@@ -75,7 +76,7 @@ LocalFreeTokens(struct token *alist)
     return 0;
 }
 
-static
+static int
 space(int x)
 {
     if (x == 0 || x == ' ' || x == '\t' || x == '\n')
@@ -84,15 +85,15 @@ space(int x)
        return 0;
 }
 
-static
+static int
 LocalParseLine(char *aline, struct token **alist)
 {
     char tbuffer[256];
-    register char *tptr;
+    char *tptr = NULL;
     int inToken;
     struct token *first, *last;
-    register struct token *ttok;
-    register int tc;
+    struct token *ttok;
+    int tc;
 
     inToken = 0;               /* not copying token chars at start */
     first = NULL;
@@ -140,20 +141,23 @@ static struct ptemp {
     char *key;
     afs_int32 value;
 } ptkeys[] = {
-"sun", 0x10000, "mon", 0x10001, "tue", 0x10002, "wed", 0x10003, "thu",
-       0x10004, "fri", 0x10005, "sat", 0x10006, "sunday", 0x10000,
-       "monday", 0x10001, "tuesday", 0x10002, "wednesday", 0x10003,
-       "thursday", 0x10004, "thur", 0x10004, "friday", 0x10005,
-       "saturday", 0x10006, "am", 0x20000, "pm", 0x20001, "a.m.",
-       0x20000, "p.m.", 0x20001, 0, 0,};
-
-static
+       {"sun", 0x10000}, {"mon", 0x10001}, {"tue", 0x10002},
+       {"wed", 0x10003}, {"thu", 0x10004}, {"fri", 0x10005},
+       {"sat", 0x10006},
+       {"sunday", 0x10000}, {"monday", 0x10001},
+       {"tuesday", 0x10002}, {"wednesday", 0x10003},
+       {"thursday", 0x10004}, {"thur", 0x10004},
+       {"friday", 0x10005}, {"saturday", 0x10006},
+       {"am", 0x20000}, {"pm", 0x20001},
+       {"a.m.", 0x20000}, {"p.m.", 0x20001}, {0, 0}};
+
+static int
 ParseTime(bos_RestartTime_p ak, char *astr)
 {
     int field;
     short temp;
-    register char *tp;
-    register int tc;
+    char *tp;
+    int tc;
 
     field = 0;                 /* 0=hour, 1=min, 2=sec */
     temp = 0;
@@ -192,7 +196,7 @@ int
 ktime_ParsePeriodic(char *adate, bos_RestartTime_p ak)
 {
     struct token *tt;
-    register afs_int32 code;
+    afs_int32 code;
     struct ptemp *tp;
 
     memset(ak, 0, sizeof(*ak));
@@ -268,7 +272,7 @@ ktime_ParsePeriodic(char *adate, bos_RestartTime_p ak)
 }
 
 int
-DoBosProcessCreate(struct cmd_syndesc *as, char *arock)
+DoBosProcessCreate(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS, BINARY, CRON, CRONTIME,
        NOTIFIER
@@ -323,7 +327,7 @@ DoBosProcessCreate(struct cmd_syndesc *as, char *arock)
     }
 
     if (!bos_ProcessCreate
-       (bos_server, process, process_type, binary, cron_time, notifier,
+       (bos_server, (char *)process, process_type, (char *)binary, (char *)cron_time, (char *)notifier,
         &st)) {
        ERR_ST_EXT("bos_ProcessCreate", st);
     }
@@ -334,7 +338,7 @@ DoBosProcessCreate(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosFSProcessCreate(struct cmd_syndesc *as, char *arock)
+DoBosFSProcessCreate(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS, FILESERVER, VOLSERVER, SALVAGER,
        NOTIFIER
@@ -375,7 +379,7 @@ DoBosFSProcessCreate(struct cmd_syndesc *as, char *arock)
     }
 
     if (!bos_FSProcessCreate
-       (bos_server, process, fileserver, volserver, salvager, notifier,
+       (bos_server, (char *)process, (char *)fileserver, (char *)volserver, (char *)salvager, (char *)notifier,
         &st)) {
        ERR_ST_EXT("bos_FSProcessCreate", st);
     }
@@ -386,7 +390,7 @@ DoBosFSProcessCreate(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessDelete(struct cmd_syndesc *as, char *arock)
+DoBosProcessDelete(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS } DoBosProcessDelete_parm_t;
     afs_status_t st = 0;
@@ -404,7 +408,7 @@ DoBosProcessDelete(struct cmd_syndesc *as, char *arock)
        process = as->parms[PROCESS].items->data;
     }
 
-    if (!bos_ProcessDelete(bos_server, process, &st)) {
+    if (!bos_ProcessDelete(bos_server, (char *)process, &st)) {
        ERR_ST_EXT("bos_ProcessDelete", st);
     }
 
@@ -435,7 +439,7 @@ Print_bos_ProcessExecutionState_p(bos_ProcessExecutionState_p state,
 }
 
 int
-DoBosProcessExecutionStateGet(struct cmd_syndesc *as, char *arock)
+DoBosProcessExecutionStateGet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS } DoBosProcessExecutionStateGet_parm_t;
     afs_status_t st = 0;
@@ -456,7 +460,7 @@ DoBosProcessExecutionStateGet(struct cmd_syndesc *as, char *arock)
     }
 
     if (!bos_ProcessExecutionStateGet
-       (bos_server, process, &state, aux_status, &st)) {
+       (bos_server, (char *)process, &state, aux_status, &st)) {
        ERR_ST_EXT("bos_ProcessExecutionStateGet", st);
     }
 
@@ -471,7 +475,7 @@ DoBosProcessExecutionStateGet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessExecutionStateSet(struct cmd_syndesc *as, char *arock)
+DoBosProcessExecutionStateSet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS, STOPPED,
        RUNNING
@@ -481,7 +485,7 @@ DoBosProcessExecutionStateSet(struct cmd_syndesc *as, char *arock)
     const char *process = NULL;
     int stop = 0;
     int run = 0;
-    bos_ProcessExecutionState_t state;
+    bos_ProcessExecutionState_t state = 0;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -522,7 +526,7 @@ DoBosProcessExecutionStateSet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessExecutionStateSetTemporary(struct cmd_syndesc *as, char *arock)
+DoBosProcessExecutionStateSetTemporary(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS, STOPPED,
        RUNNING
@@ -532,7 +536,7 @@ DoBosProcessExecutionStateSetTemporary(struct cmd_syndesc *as, char *arock)
     const char *process = NULL;
     int stop = 0;
     int run = 0;
-    bos_ProcessExecutionState_t state;
+    bos_ProcessExecutionState_t state = 0;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -564,7 +568,7 @@ DoBosProcessExecutionStateSetTemporary(struct cmd_syndesc *as, char *arock)
     }
 
     if (!bos_ProcessExecutionStateSetTemporary
-       (bos_server, process, state, &st)) {
+       (bos_server, (char *)process, state, &st)) {
        ERR_ST_EXT("bos_ProcessExecutionStateSetTemporary", st);
     }
 
@@ -574,7 +578,7 @@ DoBosProcessExecutionStateSetTemporary(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessNameList(struct cmd_syndesc *as, char *arock)
+DoBosProcessNameList(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosProcessNameList_parm_t;
     afs_status_t st = 0;
@@ -632,6 +636,7 @@ static void
 Print_bos_ProcessState_p(bos_ProcessState_p state, const char *prefix)
 {
     printf("%sProcess state:\n", prefix);
+    /* FIXME: BOS_PROCESS_OK is 0, so this test is not right */
     if (*state & BOS_PROCESS_OK) {
        printf("%s\tBOS_PROCESS_OK:\n", prefix);
     }
@@ -662,7 +667,7 @@ Print_bos_ProcessInfo_p(bos_ProcessInfo_p info, const char *prefix)
 }
 
 int
-DoBosProcessInfoGet(struct cmd_syndesc *as, char *arock)
+DoBosProcessInfoGet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS } DoBosProcessInfoGet_parm_t;
     afs_status_t st = 0;
@@ -682,7 +687,7 @@ DoBosProcessInfoGet(struct cmd_syndesc *as, char *arock)
        process = as->parms[PROCESS].items->data;
     }
 
-    if (!bos_ProcessInfoGet(bos_server, process, &type, &info, &st)) {
+    if (!bos_ProcessInfoGet(bos_server, (char *)process, &type, &info, &st)) {
        ERR_ST_EXT("bos_ProcessInfoGet", st);
     }
 
@@ -693,7 +698,7 @@ DoBosProcessInfoGet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessParameterList(struct cmd_syndesc *as, char *arock)
+DoBosProcessParameterList(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS } DoBosProcessParameterList_parm_t;
     afs_status_t st = 0;
@@ -737,7 +742,7 @@ DoBosProcessParameterList(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessNotifierGet(struct cmd_syndesc *as, char *arock)
+DoBosProcessNotifierGet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS } DoBosProcessNotifierGet_parm_t;
     afs_status_t st = 0;
@@ -772,7 +777,7 @@ DoBosProcessNotifierGet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessRestart(struct cmd_syndesc *as, char *arock)
+DoBosProcessRestart(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PROCESS } DoBosProcessRestart_parm_t;
     afs_status_t st = 0;
@@ -800,7 +805,7 @@ DoBosProcessRestart(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessAllStop(struct cmd_syndesc *as, char *arock)
+DoBosProcessAllStop(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosProcessAllStop_parm_t;
     afs_status_t st = 0;
@@ -823,7 +828,7 @@ DoBosProcessAllStop(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessAllStart(struct cmd_syndesc *as, char *arock)
+DoBosProcessAllStart(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosProcessAllStart_parm_t;
     afs_status_t st = 0;
@@ -846,7 +851,7 @@ DoBosProcessAllStart(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessAllWaitStop(struct cmd_syndesc *as, char *arock)
+DoBosProcessAllWaitStop(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosProcessAllWaitStop_parm_t;
     afs_status_t st = 0;
@@ -869,7 +874,7 @@ DoBosProcessAllWaitStop(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessAllWaitTransition(struct cmd_syndesc *as, char *arock)
+DoBosProcessAllWaitTransition(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosProcessAllWaitTransition_parm_t;
     afs_status_t st = 0;
@@ -892,7 +897,7 @@ DoBosProcessAllWaitTransition(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosProcessAllStopAndRestart(struct cmd_syndesc *as, char *arock)
+DoBosProcessAllStopAndRestart(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, INCLUDEBOS } DoBosProcessAllStopAndRestart_parm_t;
     afs_status_t st = 0;
@@ -920,12 +925,12 @@ DoBosProcessAllStopAndRestart(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosAdminCreate(struct cmd_syndesc *as, char *arock)
+DoBosAdminCreate(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, ADMIN } DoBosAdminCreate_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    const char *admin;
+    const char *admin = NULL;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -948,12 +953,12 @@ DoBosAdminCreate(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosAdminDelete(struct cmd_syndesc *as, char *arock)
+DoBosAdminDelete(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, ADMIN } DoBosAdminDelete_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    const char *admin;
+    const char *admin = NULL;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -976,7 +981,7 @@ DoBosAdminDelete(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosAdminList(struct cmd_syndesc *as, char *arock)
+DoBosAdminList(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosAdminList_parm_t;
     afs_status_t st = 0;
@@ -1015,12 +1020,12 @@ DoBosAdminList(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosKeyCreate(struct cmd_syndesc *as, char *arock)
+DoBosKeyCreate(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, VERSIONNUMBER, KEY } DoBosKeyCreate_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    int version_number;
+    int version_number = 0;
     kas_encryptionKey_t key = { {0, 0, 0, 0, 0, 0, 0, 0} };
     const char *cell;
 
@@ -1057,12 +1062,12 @@ DoBosKeyCreate(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosKeyDelete(struct cmd_syndesc *as, char *arock)
+DoBosKeyDelete(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, VERSIONNUMBER } DoBosKeyDelete_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    int version_number;
+    int version_number = 0;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -1105,7 +1110,7 @@ Print_bos_KeyInfo_p(bos_KeyInfo_p key, const char *prefix)
 }
 
 int
-DoBosKeyList(struct cmd_syndesc *as, char *arock)
+DoBosKeyList(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosKeyList_parm_t;
     afs_status_t st = 0;
@@ -1144,12 +1149,12 @@ DoBosKeyList(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosCellSet(struct cmd_syndesc *as, char *arock)
+DoBosCellSet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, CELL } DoBosCellSet_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    const char *cell;
+    const char *cell = NULL;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -1172,7 +1177,7 @@ DoBosCellSet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosCellGet(struct cmd_syndesc *as, char *arock)
+DoBosCellGet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosCellGet_parm_t;
     afs_status_t st = 0;
@@ -1198,12 +1203,12 @@ DoBosCellGet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosHostCreate(struct cmd_syndesc *as, char *arock)
+DoBosHostCreate(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, HOST } DoBosHostCreate_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    const char *host;
+    const char *host = NULL;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -1226,12 +1231,12 @@ DoBosHostCreate(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosHostDelete(struct cmd_syndesc *as, char *arock)
+DoBosHostDelete(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, HOST } DoBosHostDelete_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    const char *host;
+    const char *host = NULL;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -1254,7 +1259,7 @@ DoBosHostDelete(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosHostList(struct cmd_syndesc *as, char *arock)
+DoBosHostList(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER } DoBosHostList_parm_t;
     afs_status_t st = 0;
@@ -1267,13 +1272,14 @@ DoBosHostList(struct cmd_syndesc *as, char *arock)
            (cellHandle, as->parms[SERVER].items->data, &bos_server, &st)) {
            ERR_ST_EXT("bos_ServerOpen", st);
        }
+
+        printf("Listing hosts at server %s\n", as->parms[SERVER].items->data);
     }
 
     if (!bos_HostGetBegin(bos_server, &iter, &st)) {
        ERR_ST_EXT("bos_HostGetBegin", st);
     }
 
-    printf("Listing hosts at server %s\n", as->parms[SERVER].items->data);
 
     while (bos_HostGetNext(iter, host, &st)) {
        printf("\t%s\n", host);
@@ -1293,7 +1299,7 @@ DoBosHostList(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosExecutableCreate(struct cmd_syndesc *as, char *arock)
+DoBosExecutableCreate(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, BINARY, DEST } DoBosExecutableCreate_parm_t;
     afs_status_t st = 0;
@@ -1326,7 +1332,7 @@ DoBosExecutableCreate(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosExecutableRevert(struct cmd_syndesc *as, char *arock)
+DoBosExecutableRevert(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, EXECUTABLE } DoBosExecutableRevert_parm_t;
     afs_status_t st = 0;
@@ -1354,15 +1360,15 @@ DoBosExecutableRevert(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosExecutableTimestampGet(struct cmd_syndesc *as, char *arock)
+DoBosExecutableTimestampGet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, EXECUTABLE } DoBosExecutableTimestampGet_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
     const char *executable = NULL;
-    unsigned long new_time = 0;
-    unsigned long old_time = 0;
-    unsigned long bak_time = 0;
+    afs_int32 new_time = 0;
+    afs_int32 old_time = 0;
+    afs_int32 bak_time = 0;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -1386,7 +1392,7 @@ DoBosExecutableTimestampGet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosExecutablePrune(struct cmd_syndesc *as, char *arock)
+DoBosExecutablePrune(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, OLDFILES, BAKFILES,
        COREFILES
@@ -1427,14 +1433,14 @@ DoBosExecutablePrune(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosExecutableRestartTimeSet(struct cmd_syndesc *as, char *arock)
+DoBosExecutableRestartTimeSet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, DAILY, WEEKLY,
        TIME
     } DoBosExecutableRestartTimeSet_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    bos_Restart_t type;
+    bos_Restart_t type = 0;
     int have_daily = 0;
     int have_weekly = 0;
     bos_RestartTime_t time;
@@ -1526,14 +1532,14 @@ Print_bos_RestartTime_p(bos_RestartTime_p restart, const char *prefix)
 }
 
 int
-DoBosExecutableRestartTimeGet(struct cmd_syndesc *as, char *arock)
+DoBosExecutableRestartTimeGet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, DAILY,
        WEEKLY
     } DoBosExecutableRestartTimeGet_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    bos_Restart_t type;
+    bos_Restart_t type = 0;
     int have_daily = 0;
     int have_weekly = 0;
     bos_RestartTime_t restart_time;
@@ -1577,12 +1583,12 @@ DoBosExecutableRestartTimeGet(struct cmd_syndesc *as, char *arock)
 #define INITIAL_BUF_SIZE 4096
 
 int
-DoBosLogGet(struct cmd_syndesc *as, char *arock)
+DoBosLogGet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, LOGFILE } DoBosLogGet_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    const char *log_file;
+    const char *log_file = NULL;
     unsigned long buf_size = INITIAL_BUF_SIZE;
     char *buf = NULL;
 
@@ -1625,12 +1631,12 @@ DoBosLogGet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosAuthSet(struct cmd_syndesc *as, char *arock)
+DoBosAuthSet(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, REQUIREAUTH, DISABLEAUTH } DoBosAuthSet_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    bos_Auth_t auth;
+    bos_Auth_t auth = 0;
     int have_req = 0;
     int have_dis = 0;
 
@@ -1669,12 +1675,12 @@ DoBosAuthSet(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosCommandExecute(struct cmd_syndesc *as, char *arock)
+DoBosCommandExecute(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, COMMAND } DoBosCommandExecute_parm_t;
     afs_status_t st = 0;
     void *bos_server = NULL;
-    const char *command;
+    const char *command = NULL;
 
     if (as->parms[SERVER].items) {
        if (!bos_ServerOpen
@@ -1697,7 +1703,7 @@ DoBosCommandExecute(struct cmd_syndesc *as, char *arock)
 }
 
 int
-DoBosSalvage(struct cmd_syndesc *as, char *arock)
+DoBosSalvage(struct cmd_syndesc *as, void *arock)
 {
     typedef enum { SERVER, PARTITION, VOLUME, NUMSALVAGERS, TMPDIR, LOGFILE,
        FORCE, NOWRITE, INODES, ROOTINODES, SALVAGEDIRS, BLOCKREADS
@@ -1782,6 +1788,7 @@ DoBosSalvage(struct cmd_syndesc *as, char *arock)
     return 0;
 }
 
+#if 0
 static void
 Print_afs_RPCStatsState_p(afs_RPCStatsState_p state, const char *prefix)
 {
@@ -1795,13 +1802,14 @@ Print_afs_RPCStatsState_p(afs_RPCStatsState_p state, const char *prefix)
        break;
     }
 }
+#endif
 
 void
 SetupBosAdminCmd(void)
 {
     struct cmd_syndesc *ts;
 
-    ts = cmd_CreateSyntax("BosProcessCreate", DoBosProcessCreate, 0,
+    ts = cmd_CreateSyntax("BosProcessCreate", DoBosProcessCreate, NULL,
                          "create a new bos process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process will be created");
@@ -1817,7 +1825,7 @@ SetupBosAdminCmd(void)
                "path to notifier binary that is run when process terminates");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosFSProcessCreate", DoBosFSProcessCreate, 0,
+    ts = cmd_CreateSyntax("BosFSProcessCreate", DoBosFSProcessCreate, NULL,
                          "create a fs bos process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process will be created");
@@ -1833,7 +1841,7 @@ SetupBosAdminCmd(void)
                "path to notifier binary that is run when process terminates");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosProcessDelete", DoBosProcessDelete, 0,
+    ts = cmd_CreateSyntax("BosProcessDelete", DoBosProcessDelete, NULL,
                          "delete a bos process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process will be deleted");
@@ -1842,7 +1850,7 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosProcessExecutionStateGet",
-                         DoBosProcessExecutionStateGet, 0,
+                         DoBosProcessExecutionStateGet, NULL,
                          "get the process execution state of a process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process exists");
@@ -1851,7 +1859,7 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosProcessExecutionStateSet",
-                         DoBosProcessExecutionStateSet, 0,
+                         DoBosProcessExecutionStateSet, NULL,
                          "set the process execution state of a process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process exists");
@@ -1864,7 +1872,7 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosProcessExecutionStateSetTemporary",
-                         DoBosProcessExecutionStateSetTemporary, 0,
+                         DoBosProcessExecutionStateSetTemporary, NULL,
                          "set the process execution state "
                          "of a process temporarily");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
@@ -1877,12 +1885,12 @@ SetupBosAdminCmd(void)
                "set the process state to running");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosProcessNameList", DoBosProcessNameList, 0,
+    ts = cmd_CreateSyntax("BosProcessNameList", DoBosProcessNameList, NULL,
                          "list the names of all processes at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to query");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosProcessInfoGet", DoBosProcessInfoGet, 0,
+    ts = cmd_CreateSyntax("BosProcessInfoGet", DoBosProcessInfoGet, NULL,
                          "get information about a process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process exists");
@@ -1891,7 +1899,7 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosProcessParameterList",
-                         DoBosProcessParameterList, 0,
+                         DoBosProcessParameterList, NULL,
                          "list the parameters of a process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process exists");
@@ -1899,7 +1907,7 @@ SetupBosAdminCmd(void)
                "the name of the process");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosProcessNotifierGet", DoBosProcessNotifierGet, 0,
+    ts = cmd_CreateSyntax("BosProcessNotifierGet", DoBosProcessNotifierGet, NULL,
                          "get the notifier for a process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process exists");
@@ -1907,7 +1915,7 @@ SetupBosAdminCmd(void)
                "the name of the process");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosProcessRestart", DoBosProcessRestart, 0,
+    ts = cmd_CreateSyntax("BosProcessRestart", DoBosProcessRestart, NULL,
                          "restart a process");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where process exists");
@@ -1915,13 +1923,13 @@ SetupBosAdminCmd(void)
                "the name of the process");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosProcessAllStop", DoBosProcessAllStop, 0,
+    ts = cmd_CreateSyntax("BosProcessAllStop", DoBosProcessAllStop, NULL,
                          "stop all processes at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where processes exists");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosProcessAllWaitStop", DoBosProcessAllWaitStop, 0,
+    ts = cmd_CreateSyntax("BosProcessAllWaitStop", DoBosProcessAllWaitStop, NULL,
                          "stop all processes at a bos server and block "
                          "until they all exit");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
@@ -1929,7 +1937,7 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosProcessAllWaitTransition",
-                         DoBosProcessAllWaitTransition, 0,
+                         DoBosProcessAllWaitTransition, NULL,
                          "wait until all processes have transitioned to "
                          "their desired state");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
@@ -1937,7 +1945,7 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosProcessAllStopAndRestart",
-                         DoBosProcessAllStopAndRestart, 0,
+                         DoBosProcessAllStopAndRestart, NULL,
                          "stop all processes at a bos server and "
                          "then restart them");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
@@ -1946,7 +1954,7 @@ SetupBosAdminCmd(void)
                "include the bos server in the processes to be restarted");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosAdminCreate", DoBosAdminCreate, 0,
+    ts = cmd_CreateSyntax("BosAdminCreate", DoBosAdminCreate, NULL,
                          "create an admin user at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where admin will be created");
@@ -1954,7 +1962,7 @@ SetupBosAdminCmd(void)
                "the name of the administrator to add");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosAdminDelete", DoBosAdminDelete, 0,
+    ts = cmd_CreateSyntax("BosAdminDelete", DoBosAdminDelete, NULL,
                          "delete an admin user at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where admin will be deleted");
@@ -1962,13 +1970,13 @@ SetupBosAdminCmd(void)
                "the name of the administrator to delete");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosAdminList", DoBosAdminList, 0,
+    ts = cmd_CreateSyntax("BosAdminList", DoBosAdminList, NULL,
                          "list all admin users at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where admins will be listed");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosKeyCreate", DoBosKeyCreate, 0,
+    ts = cmd_CreateSyntax("BosKeyCreate", DoBosKeyCreate, NULL,
                          "create a key at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where key will be created");
@@ -1977,7 +1985,7 @@ SetupBosAdminCmd(void)
     cmd_AddParm(ts, "-key", CMD_SINGLE, CMD_REQUIRED, "new encryption key");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosKeyDelete", DoBosKeyDelete, 0,
+    ts = cmd_CreateSyntax("BosKeyDelete", DoBosKeyDelete, NULL,
                          "delete a key at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where key will be deleted");
@@ -1985,41 +1993,41 @@ SetupBosAdminCmd(void)
                "version number of the key");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosKeyList", DoBosKeyList, 0,
+    ts = cmd_CreateSyntax("BosKeyList", DoBosKeyList, NULL,
                          "list keys at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where keys exist");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosCellSet", DoBosCellSet, 0,
+    ts = cmd_CreateSyntax("BosCellSet", DoBosCellSet, NULL,
                          "set the cell at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to modify");
     cmd_AddParm(ts, "-cell", CMD_SINGLE, CMD_REQUIRED, "new cell");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosCellGet", DoBosCellGet, 0,
+    ts = cmd_CreateSyntax("BosCellGet", DoBosCellGet, NULL,
                          "get the cell at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to query");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosHostCreate", DoBosHostCreate, 0,
+    ts = cmd_CreateSyntax("BosHostCreate", DoBosHostCreate, NULL,
                          "add a host entry to the server CellServDB");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to modify");
     cmd_AddParm(ts, "-host", CMD_SINGLE, CMD_REQUIRED, "host to add");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosHostDelete", DoBosHostDelete, 0,
+    ts = cmd_CreateSyntax("BosHostDelete", DoBosHostDelete, NULL,
                          "delete a host entry from the server CellServDB");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to modify");
     cmd_AddParm(ts, "-host", CMD_SINGLE, CMD_REQUIRED, "host to delete");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosHostList", DoBosHostList, 0,
+    ts = cmd_CreateSyntax("BosHostList", DoBosHostList, NULL,
                          "list all host entries from the server CellServDB");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to query");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosExecutableCreate", DoBosExecutableCreate, 0,
+    ts = cmd_CreateSyntax("BosExecutableCreate", DoBosExecutableCreate, NULL,
                          "create a new binary at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to modify");
     cmd_AddParm(ts, "-binary", CMD_SINGLE, CMD_REQUIRED,
@@ -2028,7 +2036,7 @@ SetupBosAdminCmd(void)
                "path where the binary will be stored");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosExecutableRevert", DoBosExecutableRevert, 0,
+    ts = cmd_CreateSyntax("BosExecutableRevert", DoBosExecutableRevert, NULL,
                          "revert a binary at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to modify");
     cmd_AddParm(ts, "-executable", CMD_SINGLE, CMD_REQUIRED,
@@ -2036,14 +2044,14 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosExecutableTimestampGet",
-                         DoBosExecutableTimestampGet, 0,
+                         DoBosExecutableTimestampGet, NULL,
                          "get the timestamps for a binary at bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to query");
     cmd_AddParm(ts, "-executable", CMD_SINGLE, CMD_REQUIRED,
                "path to the binary to revert");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosExecutablePrune", DoBosExecutablePrune, 0,
+    ts = cmd_CreateSyntax("BosExecutablePrune", DoBosExecutablePrune, NULL,
                          "prune various files at bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to modify");
     cmd_AddParm(ts, "-oldfiles", CMD_FLAG, CMD_OPTIONAL, "prune .old files");
@@ -2052,7 +2060,7 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosExecutableRestartTimeSet",
-                         DoBosExecutableRestartTimeSet, 0,
+                         DoBosExecutableRestartTimeSet, NULL,
                          "set the restart times at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to modify");
     cmd_AddParm(ts, "-daily", CMD_FLAG, CMD_OPTIONAL,
@@ -2064,7 +2072,7 @@ SetupBosAdminCmd(void)
     SetupCommonCmdArgs(ts);
 
     ts = cmd_CreateSyntax("BosExecutableRestartTimeGet",
-                         DoBosExecutableRestartTimeGet, 0,
+                         DoBosExecutableRestartTimeGet, NULL,
                          "get the restart times at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to query");
     cmd_AddParm(ts, "-daily", CMD_FLAG, CMD_OPTIONAL,
@@ -2073,14 +2081,14 @@ SetupBosAdminCmd(void)
                "get weekly restart time");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosLogGet", DoBosLogGet, 0,
+    ts = cmd_CreateSyntax("BosLogGet", DoBosLogGet, NULL,
                          "get a log file from the bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to query");
     cmd_AddParm(ts, "-logfile", CMD_SINGLE, CMD_REQUIRED,
                "path to the log file to retrieve");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosAuthSet", DoBosAuthSet, 0,
+    ts = cmd_CreateSyntax("BosAuthSet", DoBosAuthSet, NULL,
                          "set the authorization level at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED, "server to modify");
     cmd_AddParm(ts, "-requireauth", CMD_FLAG, CMD_OPTIONAL,
@@ -2097,7 +2105,7 @@ SetupBosAdminCmd(void)
                "command to execute");
     SetupCommonCmdArgs(ts);
 
-    ts = cmd_CreateSyntax("BosSalvage", DoBosSalvage, 0,
+    ts = cmd_CreateSyntax("BosSalvage", DoBosSalvage, NULL,
                          "execute a salvage command at a bos server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_REQUIRED,
                "server where salvager will execute");