Skip to content

Commit 64226fe

Browse files
committed
fix: implemented minor coderabbitai suggestions
Signed-off-by: Lucas Tembras <[email protected]>
1 parent bdd9179 commit 64226fe

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

plugins/out_datadog/datadog_conf.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,11 @@ struct flb_out_datadog *flb_datadog_conf_create(struct flb_output_instance *ins,
103103
tmp = flb_output_get_property("site", ins);
104104
if (tmp){
105105
ctx->site = flb_sds_create(tmp);
106+
if (!ctx->site) {
107+
flb_plg_error(ctx->ins, "failed to allocate memory for site parameter");
108+
flb_datadog_conf_destroy(ctx);
109+
return NULL;
110+
}
106111
flb_plg_debug(ctx->ins, "site parameter set to: %s", ctx->site);
107112
} else {
108113
flb_plg_debug(ctx->ins, "no site parameter found");
@@ -158,10 +163,7 @@ struct flb_out_datadog *flb_datadog_conf_create(struct flb_output_instance *ins,
158163
flb_datadog_conf_destroy(ctx);
159164
return NULL;
160165
}
161-
flb_plg_debug(ctx->ins, "created base hostname: %s", tmp_sds);
162-
flb_plg_debug(ctx->ins, "about to concatenate site: %s (length: %zu)", ctx->site, flb_sds_len(ctx->site));
163166
tmp_sds = flb_sds_cat(tmp_sds, (const char *)ctx->site, flb_sds_len(ctx->site));
164-
flb_plg_debug(ctx->ins, "after concatenation: %s", tmp_sds);
165167
ctx->host = tmp_sds;
166168
flb_plg_debug(ctx->ins, "host constructed from site: %s", ctx->host);
167169
} else {

0 commit comments

Comments
 (0)