linux-kernel - Re: [Y2038] [PATCH] fs:affs:Replace time_t with time64

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: <4312946.SmG7PJzsvD@wuerfel>
Date:	Thu, 12 Nov 2015 14:45:28 +0100
From:	Arnd Bergmann <arnd@...db.de>
To:	y2038@...ts.linaro.org
Cc:	DengChao <chao.deng@...aro.org>, linux-fsdevel@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [Y2038] [PATCH] fs:affs:Replace time_t with time64_t

On Thursday 12 November 2015 21:40:41 DengChao wrote:
> The affs code uses "time_t" and "get_seconds()". This will cause
> problems on 32-bit architectures in 2038 when time_t overflows.
> This patch replaces them with "time64_t" and
> "ktime_get_real_seconds()". This patch introduces expensive 64-bit
> divsion in "secs_to_datestamp()", considering this function is not
> called so often, the cost should be acceptable.  
> 
> Signed-off-by: DengChao <chao.deng@...aro.org>
> 

Looks good,

Reviewed-by: Arnd Bergmann <arnd@...db.de>

Is anyone picking up patches for random unmaintained file systems,
or should we put it into the y2038 tree?

	Arnd
--
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/