Attachment 825147 Details for Bug 1031383 – patch to correctly check match_rx return codes

[patch] patch to correctly check match_rx return codes

cronie-anacron-nodelay.rhel6.patch (text/plain), 1.51 KB, created by Martin Poole on 2013-11-17 14:05:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Poole

Created: 2013-11-17 14:05:38 UTC

Size: 1.51 KB

patch

obsolete

>diff -up cronie-1.4.4/anacron/readtab.c.nodelay cronie-1.4.4/anacron/readtab.c
>--- cronie-1.4.4/anacron/readtab.c.nodelay	2009-09-25 07:23:18.000000000 +0100
>+++ cronie-1.4.4/anacron/readtab.c	2013-11-17 10:06:03.719798100 +0000
>@@ -271,21 +271,27 @@ parse_tab_line(char *line)
>         if (strncmp(env_var, "START_HOURS_RANGE", 17) == 0)
>         {
>             r = match_rx("^([[:digit:]]+)-([[:digit:]]+)$", value, 2, &from, &to);
>-            if ((r == -1) || (from == NULL) || (to == NULL)) goto reg_invalid;
>+	    if (r == -1) goto reg_err;
>+	    if (r == 0) goto reg_invalid;
>             range_start = atoi(from);
>             range_stop = atoi(to);
>             Debug(("Jobs will start in the %02d:00-%02d:00 range.", range_start, range_stop));
>         }
>         if (strncmp(env_var, "RANDOM_DELAY", 12) == 0) {
>             r = match_rx("^([[:digit:]]+)$", value, 0);
>-            if (r != -1) {
>-                int i = random();
>-                double x = 0;
>-                x = (double) i / (double) RAND_MAX * (double) (atoi(value));
>-                random_number = (int)x;
>+	    if (r == -1) goto reg_err;
>+	    if (r)
>+	    {
>+		random_number = 0;
>+		if (atoi(value) > 0) {
>+		    int i = random();
>+		    double x = 0;
>+		    x = (double) i / (double) RAND_MAX * (double) (atoi(value));
>+		    random_number = (int)x;
>+		}
>                 Debug(("Randomized delay set: %d", random_number));
>             }
>-        else goto reg_invalid;
>+	    else goto reg_invalid;
>         }
> 	register_env(env_var, value);
> 	return;

Actions: View | Diff

Attachments on bug 1031383: 825147