From: Tomas Mraz <tmraz@fedoraproject.org>
Date: Fri, 11 Mar 2011 16:42:56 +0000 (+0100)
Subject: The last bit to set is always LAST_ - FIRST_.
X-Git-Tag: cronie1.4.7~8
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c23324a87dc27969ee58087b6facb9ab4bae908e;p=cronie

The last bit to set is always LAST_ - FIRST_.
---

diff --git a/src/entry.c b/src/entry.c
index c5027db..5d377c6 100644
--- a/src/entry.c
+++ b/src/entry.c
@@ -140,38 +140,38 @@ entry *load_entry(FILE * file, void (*error_func) (), struct passwd *pw,
 			bit_set(e->hour, 0);
 			bit_set(e->dom, 0);
 			bit_set(e->month, 0);
-			bit_nset(e->dow, 0, (LAST_DOW - FIRST_DOW + 1));
+			bit_nset(e->dow, 0, LAST_DOW - FIRST_DOW);
 			e->flags |= DOW_STAR;
 		}
 		else if (!strcmp("monthly", cmd)) {
 			bit_set(e->minute, 0);
 			bit_set(e->hour, 0);
 			bit_set(e->dom, 0);
-			bit_nset(e->month, 0, (LAST_MONTH - FIRST_MONTH + 1));
-			bit_nset(e->dow, 0, (LAST_DOW - FIRST_DOW + 1));
+			bit_nset(e->month, 0, LAST_MONTH - FIRST_MONTH);
+			bit_nset(e->dow, 0, LAST_DOW - FIRST_DOW);
 			e->flags |= DOW_STAR;
 		}
 		else if (!strcmp("weekly", cmd)) {
 			bit_set(e->minute, 0);
 			bit_set(e->hour, 0);
-			bit_nset(e->dom, 0, (LAST_DOM - FIRST_DOM + 1));
-			bit_nset(e->month, 0, (LAST_MONTH - FIRST_MONTH + 1));
+			bit_nset(e->dom, 0, LAST_DOM - FIRST_DOM);
+			bit_nset(e->month, 0, LAST_MONTH - FIRST_MONTH);
 			bit_set(e->dow, 0);
 			e->flags |= DOW_STAR;
 		}
 		else if (!strcmp("daily", cmd) || !strcmp("midnight", cmd)) {
 			bit_set(e->minute, 0);
 			bit_set(e->hour, 0);
-			bit_nset(e->dom, 0, (LAST_DOM - FIRST_DOM + 1));
-			bit_nset(e->month, 0, (LAST_MONTH - FIRST_MONTH + 1));
-			bit_nset(e->dow, 0, (LAST_DOW - FIRST_DOW + 1));
+			bit_nset(e->dom, 0, LAST_DOM - FIRST_DOM);
+			bit_nset(e->month, 0, LAST_MONTH - FIRST_MONTH);
+			bit_nset(e->dow, 0, LAST_DOW - FIRST_DOW);
 		}
 		else if (!strcmp("hourly", cmd)) {
 			bit_set(e->minute, 0);
-			bit_nset(e->hour, 0, (LAST_HOUR - FIRST_HOUR + 1));
-			bit_nset(e->dom, 0, (LAST_DOM - FIRST_DOM + 1));
-			bit_nset(e->month, 0, (LAST_MONTH - FIRST_MONTH + 1));
-			bit_nset(e->dow, 0, (LAST_DOW - FIRST_DOW + 1));
+			bit_nset(e->hour, 0, LAST_HOUR - FIRST_HOUR);
+			bit_nset(e->dom, 0, LAST_DOM - FIRST_DOM);
+			bit_nset(e->month, 0, LAST_MONTH - FIRST_MONTH);
+			bit_nset(e->dow, 0, LAST_DOW - FIRST_DOW);
 			e->flags |= HR_STAR;
 		}
 		else {