X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fwrite_http.c;h=b3baa555d2393fb7f3d48903afb5cdd4528a7c5b;hb=c76419c0cf983f1ecd3d36aa236cc4e3f9cff733;hp=41615d3f0ccbd0be9c1cefba46708a4ace10e680;hpb=fd06911c23375dc40beeef0406ddf98cec7d3fcf;p=collectd.git diff --git a/src/write_http.c b/src/write_http.c index 41615d3f..b3baa555 100644 --- a/src/write_http.c +++ b/src/write_http.c @@ -366,7 +366,7 @@ static int wh_write_command (const data_set_t *ds, const value_list_t *vl, /* {{ if ((cb == NULL) || (cb->send_buffer == NULL)) return -1; - if (strcmp (ds->type, vl->type) == 0) { + if (strcmp (ds->type, vl->type) != 0) { ERROR ("write_http plugin: DS type does not match " "value list type"); return -1; @@ -419,6 +419,9 @@ static int wh_write_command (const data_set_t *ds, const value_list_t *vl, /* {{ } assert (command_len < cb->send_buffer_free); + /* Make scan-build happy. */ + assert (cb->send_buffer != NULL); + /* `command_len + 1' because `command_len' does not include the * trailing null byte. Neither does `send_buffer_fill'. */ memcpy (cb->send_buffer + cb->send_buffer_fill, @@ -817,13 +820,10 @@ static int wh_config_node (oconfig_item_t *ci) /* {{{ */ callback_name, cb->location); user_data_t user_data = { - .data = cb + .data = cb, + .free_func = wh_callback_free, }; - plugin_register_flush (callback_name, wh_flush, &user_data); - - user_data.free_func = wh_callback_free; - if (cb->send_metrics) { plugin_register_write (callback_name, wh_write, &user_data);