Skip to content

Commit 0a95d62

Browse files
cosmo0920edsiper
authored andcommitted
in_winevtlog: Handle capped values
Signed-off-by: Hiroshi Hatake <hiroshi@chronosphere.io>
1 parent 4510a89 commit 0a95d62

1 file changed

Lines changed: 11 additions & 2 deletions

File tree

plugins/in_winevtlog/in_winevtlog.c

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -200,15 +200,21 @@ static int in_winevtlog_init(struct flb_input_instance *in,
200200
ctx->backoff.multiplier_x1000 = (DWORD)(mult * 1000.0);
201201

202202
/* normalize base/max/jitter/retries to sane ranges */
203-
if (ctx->backoff.base_ms == 0) {
203+
if (ctx->backoff.base_ms <= 0) {
204204
ctx->backoff.base_ms = 500;
205205
}
206-
if (ctx->backoff.max_ms == 0) {
206+
if (ctx->backoff.max_ms <= 0) {
207207
ctx->backoff.max_ms = 30000;
208208
}
209+
if (ctx->backoff.jitter_pct < 0) {
210+
ctx->backoff.jitter_pct = 0;
211+
}
209212
if (ctx->backoff.jitter_pct == 0) {
210213
ctx->backoff.jitter_pct = 20;
211214
}
215+
if (ctx->backoff.max_retries < 0) {
216+
ctx->backoff.max_retries = 0;
217+
}
212218
if (ctx->backoff.max_retries == 0) {
213219
ctx->backoff.max_retries = 8;
214220
}
@@ -223,6 +229,9 @@ static int in_winevtlog_init(struct flb_input_instance *in,
223229
if (ctx->backoff.jitter_pct > 100U) { /* jitter as percentage */
224230
ctx->backoff.jitter_pct = 100U;
225231
}
232+
if ((unsigned) ctx->backoff.max_retries > 100U) { /* cap retries */
233+
ctx->backoff.max_retries = 100;
234+
}
226235
/* ensure ordering */
227236
if (ctx->backoff.max_ms < ctx->backoff.base_ms) {
228237
flb_plg_warn(in, "reconnect.max_ms < reconnect.base_ms, swapping values");

0 commit comments

Comments
 (0)