Code

write_riemann plugin: Ensure that "statuses" is always initialized.
authorFlorian Forster <octo@collectd.org>
Mon, 29 Jun 2015 10:52:03 +0000 (12:52 +0200)
committerFlorian Forster <octo@collectd.org>
Mon, 29 Jun 2015 10:52:03 +0000 (12:52 +0200)
Hopefully fixes "Uninitialized argument value" warning.

src/write_riemann.c

index a67f94cc62732972421a4d85196cdce012916bc3..26fa3f45680358ee156f8576b6adfaf30be620a9 100644 (file)
@@ -619,7 +619,7 @@ static Msg *riemann_value_list_to_protobuf (struct riemann_host const *host, /*
        msg__init (msg);
 
        /* Set up events. First, the list of pointers. */
-       msg->n_events = (size_t) vl->values_len;
+       msg->n_events = vl->values_len;
        msg->events = calloc (msg->n_events, sizeof (*msg->events));
        if (msg->events == NULL)
        {
@@ -789,6 +789,8 @@ static int riemann_write(const data_set_t *ds, /* {{{ */
                status = write_riemann_threshold_check(ds, vl, statuses);
                if (status != 0)
                        return status;
+       } else {
+               memset (statuses, 0, sizeof (statuses));
        }
 
        if (host->use_tcp == 1 && host->batch_mode) {