From 883b41f5af2fc46d2d08e8dd23308d8d263e2c22 Mon Sep 17 00:00:00 2001 From: Ling Huang Date: Wed, 27 Nov 2019 16:32:39 -0500 Subject: [PATCH] Fix unreachable clause where both tmp_plugin and tmp_plugin_instance are non-empty. --- src/aggregation.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/aggregation.c b/src/aggregation.c index 2c8ef880..d7b8e72b 100644 --- a/src/aggregation.c +++ b/src/aggregation.c @@ -194,17 +194,21 @@ static int agg_instance_create_name(agg_instance_t *inst, /* {{{ */ sstrncpy(tmp_plugin_instance, agg->ident.plugin_instance, sizeof(tmp_plugin_instance)); + // Both tmp_plugin and tmp_plugin_instance are empty. if ((strcmp("", tmp_plugin) == 0) && (strcmp("", tmp_plugin_instance) == 0)) sstrncpy(inst->ident.plugin_instance, AGG_FUNC_PLACEHOLDER, sizeof(inst->ident.plugin_instance)); - else if (strcmp("", tmp_plugin) != 0) + // tmp_plugin is non-empty and tmp_plugin_instance is empty. + else if (strcmp("", tmp_plugin_instance) == 0) ssnprintf(inst->ident.plugin_instance, sizeof(inst->ident.plugin_instance), "%s-%s", tmp_plugin, AGG_FUNC_PLACEHOLDER); - else if (strcmp("", tmp_plugin_instance) != 0) + // tmp_plugin is empty and tmp_plugin_instance is non-empty. + else if (strcmp("", tmp_plugin) == 0) ssnprintf(inst->ident.plugin_instance, sizeof(inst->ident.plugin_instance), "%s-%s", tmp_plugin_instance, AGG_FUNC_PLACEHOLDER); + // Both tmp_plugin and tmp_plugin_instance are non-empty. else ssnprintf(inst->ident.plugin_instance, sizeof(inst->ident.plugin_instance), "%s-%s-%s", tmp_plugin, -- 2.11.0