parser: Fix hour validation in timezone offset parsing

Signed-off-by: Sijmen Huizenga <[email protected]>

Author: SijmenHuizenga

src/flb_parser.c

@@ -530,7 +530,8 @@ static int parser_conf_file(const char *cfg, struct flb_cf *cf,
                       name, cfg);
             goto fconf_early_error;
         }
-        
+
+
         /* skip_empty_values */
         skip_empty = FLB_TRUE;
         tmp_str = get_parser_key(config, cf, s, "skip_empty_values");
@@ -1040,7 +1041,7 @@ int flb_parser_tzone_offset(const char *str, int len, int *tmdiff)
         min = ((p[2] - '0') * 10) + (p[3] - '0');
     }
 
-    if (hour < 0 || hour > 59 || min < 0 || min > 59) {
+    if (hour < 0 || hour > 23 || min < 0 || min > 59) {
         return -1;
     }
 

tests/internal/parser.c

@@ -33,6 +33,7 @@ struct tz_check tz_entries_ok[] = {
     {"+0000",       0},
     {"+00:00",      0},
     {"+00:59",   3540},
+    {"+23:59",  86340},
     {"-0600",  -21600},
     {"-06:00", -21600},
     {"Z",           0},
@@ -42,6 +43,7 @@ struct tz_check tz_entries_error[] = {
     {"0000",   0},
     {"+00:90", 0},
     {"--600",  0},
+    {"+24:00", 0},
     {"foo",    0},
 };