libadmin: Don't dereference NULL pointer in cmd

If DoBosHostList ends up being called without the -server argument
in its command structure, then a printf will attempt to dereference
a NULL pointer to the -server field.

Caught by clang-analyzer

Change-Id: I1114406d03b265f4d780efd8e84fea53013cb51c
Reviewed-on: http://gerrit.openafs.org/2143
Reviewed-by: Russ Allbery <rra@stanford.edu>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

diff --git a/src/libadmin/test/bos.c b/src/libadmin/test/bos.c
index 3b283b3714eb7cd0cf2859d79783858920f3e38d..3661d2cc4f2bac87796079c1e8f14aa59febfcb6 100644 (file)
--- a/src/libadmin/test/bos.c
+++ b/src/libadmin/test/bos.c
@@ -1270,13 +1270,14 @@ DoBosHostList(struct cmd_syndesc *as, void *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);