linux-kernel - Re: [PATCH v2] isofs: Fix bug in the way to check if the year is a leap year

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <9B989AC4-AAF6-4C63-9815-E68F2235F1BD@gmail.com>
Date:	Wed, 7 Jan 2015 11:15:57 +0000
From:	Òscar Forner Martínez 
	<oscar.forner.martinez@...il.com>
To:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] isofs: Fix bug in the way to check if the year is a leap year

I have sent that e-mail three days ago. As nobody is the maintainer It is only send for the general list. This is just a reminder for someone who can check it.

On 4 Jan 2015, at 12:42, Oscar Forner Martinez <oscar.forner.martinez@...il.com> wrote:

> Changed the whole algorithm for a call to mktime64 that takes
> care of all that details.
> 
> Signed-off-by: Oscar Forner Martinez <oscar.forner.martinez@...il.com>
> ---
> fs/isofs/util.c | 14 ++------------
> 1 file changed, 2 insertions(+), 12 deletions(-)
> 
> diff --git a/fs/isofs/util.c b/fs/isofs/util.c
> index 01e1ee7..a3f953a 100644
> --- a/fs/isofs/util.c
> +++ b/fs/isofs/util.c
> @@ -2,6 +2,7 @@
>  *  linux/fs/isofs/util.c
>  */
> 
> +#include <linux/time.h>
> #include "isofs.h"
> 
> /* 
> @@ -31,18 +32,7 @@ int iso_date(char * p, int flag)
> 	if (year < 0) {
> 		crtime = 0;
> 	} else {
> -		int monlen[12] = {31,28,31,30,31,30,31,31,30,31,30,31};
> -
> -		days = year * 365;
> -		if (year > 2)
> -			days += (year+1) / 4;
> -		for (i = 1; i < month; i++)
> -			days += monlen[i-1];
> -		if (((year+2) % 4) == 0 && month > 2)
> -			days++;
> -		days += day - 1;
> -		crtime = ((((days * 24) + hour) * 60 + minute) * 60)
> -			+ second;
> +		crtime = mktime64(year+1970, month, day, hour, minute, second);
> 
> 		/* sign extend */
> 		if (tz & 0x80)
> -- 
> 1.9.1
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/