in_winevtlog: Suppress warnings

cosmo0920 · cosmo0920 · commit e75b7234cc5d · 2025-05-21T11:37:32.000+09:00
Signed-off-by: Hiroshi Hatake &lt;hiroshi@chronosphere.io&gt;
diff --git a/plugins/in_winevtlog/in_winevtlog.c b/plugins/in_winevtlog/in_winevtlog.c
@@ -177,7 +177,7 @@ static int in_winevtlog_init(struct flb_input_instance *in,
         flb_plg_error(in, "could not initialize event encoder");
         flb_free(ctx);
 
-        return NULL;
+        return -1;
     }
 
     /* Load the config map */
diff --git a/plugins/in_winevtlog/pack.c b/plugins/in_winevtlog/pack.c
@@ -34,7 +34,7 @@
 
 static int pack_nullstr(struct winevtlog_config *ctx)
 {
-    flb_log_event_encoder_append_body_cstring(ctx->log_encoder, "");
+    return flb_log_event_encoder_append_body_cstring(ctx->log_encoder, "");
 }
 
 static int pack_wstr(struct winevtlog_config *ctx, const wchar_t *wstr)
@@ -314,6 +314,8 @@ static int pack_sid(struct winevtlog_config *ctx, PSID sid, int extract_sid)
 
             _snprintf_s(formatted, result_len, _TRUNCATE, "%s\\%s", domain, account);
 
+            size = strlen(formatted);
+
             if (size > 0) {
                 flb_log_event_encoder_append_body_cstring(ctx->log_encoder, formatted);
 

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ static int in_winevtlog_init(struct flb_input_instance *in,`
`177`	`177`	`flb_plg_error(in, "could not initialize event encoder");`
`178`	`178`	`flb_free(ctx);`
`179`	`179`
`180`		`- return NULL;`
	`180`	`+ return -1;`
`181`	`181`	`}`
`182`	`182`
`183`	`183`	`/* Load the config map */`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@`
`34`	`34`
`35`	`35`	`static int pack_nullstr(struct winevtlog_config *ctx)`
`36`	`36`	`{`
`37`		`- flb_log_event_encoder_append_body_cstring(ctx->log_encoder, "");`
	`37`	`+ return flb_log_event_encoder_append_body_cstring(ctx->log_encoder, "");`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`static int pack_wstr(struct winevtlog_config ctx, const wchar_t wstr)`
`@@ -314,6 +314,8 @@ static int pack_sid(struct winevtlog_config *ctx, PSID sid, int extract_sid)`
`314`	`314`
`315`	`315`	`_snprintf_s(formatted, result_len, _TRUNCATE, "%s\\%s", domain, account);`
`316`	`316`
	`317`	`+ size = strlen(formatted);`
	`318`	`+`
`317`	`319`	`if (size > 0) {`
`318`	`320`	`flb_log_event_encoder_append_body_cstring(ctx->log_encoder, formatted);`
`319`	`321`