Paul Fagerburg has submitted this change. ( https://review.coreboot.org/c/coreboot/+/52370 )
Change subject: lib/rtc: Fix incomplete leap year macro ......................................................................
lib/rtc: Fix incomplete leap year macro
Signed-off-by: Jakub Czapiga jacz@semihalf.com Found-by: Paul Fagerburg pfagerburg@chromium.org Change-Id: Ic434c969141c67ce317a5db0c8805de02c84eb08 Reviewed-on: https://review.coreboot.org/c/coreboot/+/52370 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Paul Fagerburg pfagerburg@chromium.org --- M src/lib/rtc.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Paul Fagerburg: Looks good to me, approved
diff --git a/src/lib/rtc.c b/src/lib/rtc.c index 5e03cfa..c5fffa0 100644 --- a/src/lib/rtc.c +++ b/src/lib/rtc.c @@ -11,7 +11,7 @@ #define STARTOFTIME 1970 #define SECDAY 86400L #define SECYR (SECDAY * 365) -#define LEAP_YEAR(year) ((year) % 4 == 0) +#define LEAP_YEAR(year) (((year) % 4 == 0 && (year) % 100 != 0) || (year) % 400 == 0) #define DAYS_IN_YEAR(a) (LEAP_YEAR(a) ? 366 : 365) #define DAYS_IN_MONTH(a) (month_days[(a) - 1])