From: Pierre-Yves Ritschard <pyr@spootnik.org>
Date: Tue, 3 May 2016 07:59:22 +0000 (+0200)
Subject: write_kafka: reliability when no key provided
X-Git-Tag: collectd-5.6.0~305^2
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=556c6901c4ca312bd9f0f6ca0f92b1e588e1da3b;p=collectd.git

write_kafka: reliability when no key provided
---

diff --git a/src/write_kafka.c b/src/write_kafka.c
index 37b3e14f..10ae5a57 100644
--- a/src/write_kafka.c
+++ b/src/write_kafka.c
@@ -202,6 +202,8 @@ static int kafka_write(const data_set_t *ds, /* {{{ */
     key = ctx->key;
     if (key != NULL)
         keylen = strlen (key);
+    else
+        keylen = 0;
 
     rd_kafka_produce(ctx->topic, RD_KAFKA_PARTITION_UA,
                      RD_KAFKA_MSG_F_COPY, buffer, blen,
@@ -252,6 +254,7 @@ static void kafka_config_topic(rd_kafka_conf_t *conf, oconfig_item_t *ci) /* {{{
     tctx->escape_char = '.';
     tctx->store_rates = 1;
     tctx->format = KAFKA_FORMAT_JSON;
+    tctx->key = NULL;
 
     if ((tctx->kafka_conf = rd_kafka_conf_dup(conf)) == NULL) {
         sfree(tctx);
@@ -315,6 +318,7 @@ static void kafka_config_topic(rd_kafka_conf_t *conf, oconfig_item_t *ci) /* {{{
 
         } else if (strcasecmp ("Key", child->key) == 0)  {
             cf_util_get_string (child, &tctx->key);
+            assert (tctx->key != NULL);
         } else if (strcasecmp ("Format", child->key) == 0) {
             status = cf_util_get_string(child, &key);
             if (status != 0)