ceph plugin: Use compound literals to initialize a VLA.

This also moves the DEBUG() statement before the sanity check in an
attempt to make scan-build happy: scan-build is under the impression
that the DEBUG() macro would write to the g_num_daemons global variable.

While we're at it, use PRIu32 to print the uint32_t.

Issue: #1835

diff --git a/src/ceph.c b/src/ceph.c
index ad0688ccb22245a6131e135c3f09dddca5a93a74..64d87855fbb0f820ea1a8e8f6d09eeb61894154c 100644 (file)
--- a/src/ceph.c
+++ b/src/ceph.c
@@ -1468,21 +1468,22 @@ static int cconn_main_loop(uint32_t request_type)
     struct timeval end_tv;
     struct cconn io_array[g_num_daemons];
 
+    DEBUG ("ceph plugin: entering cconn_main_loop(request_type = %"PRIu32")", request_type);
+
     if (g_num_daemons < 1)
     {
         ERROR ("ceph plugin: No daemons configured. See the \"Daemon\" config option.");
         return ENOENT;
     }
 
-    DEBUG("ceph plugin: entering cconn_main_loop(request_type = %d)", request_type);
-
     /* create cconn array */
-    memset(io_array, 0, sizeof(io_array));
-    for(size_t i = 0; i < g_num_daemons; ++i)
+    for (size_t i = 0; i < g_num_daemons; i++)
     {
-        io_array[i].d = g_daemons[i];
-        io_array[i].request_type = request_type;
-        io_array[i].state = CSTATE_UNCONNECTED;
+        io_array[i] = (struct cconn) {
+            .d = g_daemons[i],
+            .request_type = request_type,
+            .state = CSTATE_UNCONNECTED,
+        };
     }
 
     /** Calculate the time at which we should give up */