Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1283470 > unrolled thread

[PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm

Started bySasha Levin <sasha.levin@oracle.com>
First post2015-12-04 01:20 +0100
Last post2015-12-04 03:10 +0100
Articles 6 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm Sasha Levin <sasha.levin@oracle.com> - 2015-12-04 01:20 +0100
    Re: [PATCH] rtc: fix overflow and incorrect calculation in  rtc_time64_to_tm kbuild test robot <lkp@intel.com> - 2015-12-04 01:30 +0100
    Re: [PATCH] rtc: fix overflow and incorrect calculation in  rtc_time64_to_tm kbuild test robot <lkp@intel.com> - 2015-12-04 02:50 +0100
    Re: [PATCH] rtc: fix overflow and incorrect calculation in  rtc_time64_to_tm kbuild test robot <lkp@intel.com> - 2015-12-04 02:50 +0100
    Re: [PATCH] rtc: fix overflow and incorrect calculation in  rtc_time64_to_tm Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2015-12-04 03:00 +0100
      Re: [PATCH] rtc: fix overflow and incorrect calculation in  rtc_time64_to_tm Sasha Levin <sasha.levin@oracle.com> - 2015-12-04 03:10 +0100

#1283470 — [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm

FromSasha Levin <sasha.levin@oracle.com>
Date2015-12-04 01:20 +0100
Subject[PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm
Message-ID<qBMgp-3Ms-7@gated-at.bofh.it>
At some point after humans go extinct and robots cotrol the world, dividing
he time64_t by 86400 to extract the days will overflow a 32bit integer,
leading to incorrect conversion into rtc_time in rtc_time64_to_tm().

Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
---
 drivers/rtc/rtc-lib.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/rtc/rtc-lib.c b/drivers/rtc/rtc-lib.c
index e6bfb9c..459cd4d 100644
--- a/drivers/rtc/rtc-lib.c
+++ b/drivers/rtc/rtc-lib.c
@@ -54,11 +54,11 @@ void rtc_time64_to_tm(time64_t time, struct rtc_time *tm)
 {
 	unsigned int month, year;
 	unsigned long secs;
-	int days;
+	time64_t days;
 
 	/* time must be positive */
 	days = div_s64(time, 86400);
-	secs = time - (unsigned int) days * 86400;
+	secs = time - days * 86400;
 
 	/* day of the week, 1970-01-01 was a Thursday */
 	tm->tm_wday = (days + 4) % 7;
-- 
1.7.10.4

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

[toc] | [next] | [standalone]

#1283480 — Re: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm

Fromkbuild test robot <lkp@intel.com>
Date2015-12-04 01:30 +0100
SubjectRe: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm
Message-ID<qBMq6-3PE-3@gated-at.bofh.it>
In reply to#1283470

[Multipart message — attachments visible in raw view] — view raw

[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
Hi Sasha,

[auto build test ERROR on abelloni/rtc-next]
[also build test ERROR on v4.4-rc3 next-20151202]

url:    https://github.com/0day-ci/linux/commits/Sasha-Levin/rtc-fix-overflow-and-incorrect-calculation-in-rtc_time64_to_tm/20151204-081508
base:   https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: i386-tinyconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   drivers/built-in.o: In function `rtc_time64_to_tm':
>> (.text+0x6795): undefined reference to `__moddi3'
   drivers/built-in.o: In function `rtc_time64_to_tm':
>> (.text+0x67aa): undefined reference to `__divdi3'

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[toc] | [prev] | [next] | [standalone]

#1283533 — Re: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm

Fromkbuild test robot <lkp@intel.com>
Date2015-12-04 02:50 +0100
SubjectRe: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm
Message-ID<qBNFz-4zO-61@gated-at.bofh.it>
In reply to#1283470

[Multipart message — attachments visible in raw view] — view raw

[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
Hi Sasha,

[auto build test ERROR on abelloni/rtc-next]
[also build test ERROR on v4.4-rc3 next-20151202]

url:    https://github.com/0day-ci/linux/commits/Sasha-Levin/rtc-fix-overflow-and-incorrect-calculation-in-rtc_time64_to_tm/20151204-081508
base:   https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: arm-prima2_defconfig (attached as .config)
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=arm 

All errors (new ones prefixed by >>):

   drivers/built-in.o: In function `rtc_time64_to_tm':
>> drivers/rtc/rtc-lib.c:64: undefined reference to `__aeabi_ldivmod'
   drivers/rtc/rtc-lib.c:66: undefined reference to `__aeabi_ldivmod'

vim +64 drivers/rtc/rtc-lib.c

c58411e9 Alessandro Zummo 2006-03-27  58  
c2c11ae4 pang.xunlei      2014-11-18  59  	/* time must be positive */
c2c11ae4 pang.xunlei      2014-11-18  60  	days = div_s64(time, 86400);
ee9fd0e8 Sasha Levin      2015-12-03  61  	secs = time - days * 86400;
c58411e9 Alessandro Zummo 2006-03-27  62  
c58411e9 Alessandro Zummo 2006-03-27  63  	/* day of the week, 1970-01-01 was a Thursday */
c58411e9 Alessandro Zummo 2006-03-27 @64  	tm->tm_wday = (days + 4) % 7;
c58411e9 Alessandro Zummo 2006-03-27  65  
c58411e9 Alessandro Zummo 2006-03-27  66  	year = 1970 + days / 365;
c58411e9 Alessandro Zummo 2006-03-27  67  	days -= (year - 1970) * 365

:::::: The code at line 64 was first introduced by commit
:::::: c58411e95d7f5062dedd1a3064af4d359da1e633 [PATCH] RTC Subsystem: library functions

:::::: TO: Alessandro Zummo <a.zummo@towertech.it>
:::::: CC: Linus Torvalds <torvalds@g5.osdl.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[toc] | [prev] | [next] | [standalone]

#1283535 — Re: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm

Fromkbuild test robot <lkp@intel.com>
Date2015-12-04 02:50 +0100
SubjectRe: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm
Message-ID<qBNFA-4zO-77@gated-at.bofh.it>
In reply to#1283470

[Multipart message — attachments visible in raw view] — view raw

[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
Hi Sasha,

[auto build test ERROR on abelloni/rtc-next]
[also build test ERROR on v4.4-rc3 next-20151202]

url:    https://github.com/0day-ci/linux/commits/Sasha-Levin/rtc-fix-overflow-and-incorrect-calculation-in-rtc_time64_to_tm/20151204-081508
base:   https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: avr32-mimc200_defconfig (attached as .config)
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=avr32 

All errors (new ones prefixed by >>):

   crypto/built-in.o: warning: input is not relaxable
   virt/built-in.o: warning: input is not relaxable
   drivers/built-in.o: In function `phy_device_create':
>> (.text+0x3c70c): undefined reference to `__avr32_smod64'
   drivers/built-in.o: In function `phy_device_create':
   (.text+0x3c710): undefined reference to `__avr32_sdiv64'

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[toc] | [prev] | [next] | [standalone]

#1283541 — Re: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm

FromAlexandre Belloni <alexandre.belloni@free-electrons.com>
Date2015-12-04 03:00 +0100
SubjectRe: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm
Message-ID<qBNPb-4Dl-9@gated-at.bofh.it>
In reply to#1283470
On 03/12/2015 at 19:12:24 -0500, Sasha Levin wrote :
> At some point after humans go extinct and robots cotrol the world, dividing
> he time64_t by 86400 to extract the days will overflow a 32bit integer,
> leading to incorrect conversion into rtc_time in rtc_time64_to_tm().
> 

And at that time, the robots won't care about 32bit platforms :)

> Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
> ---
>  drivers/rtc/rtc-lib.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/rtc/rtc-lib.c b/drivers/rtc/rtc-lib.c
> index e6bfb9c..459cd4d 100644
> --- a/drivers/rtc/rtc-lib.c
> +++ b/drivers/rtc/rtc-lib.c
> @@ -54,11 +54,11 @@ void rtc_time64_to_tm(time64_t time, struct rtc_time *tm)
>  {
>  	unsigned int month, year;
>  	unsigned long secs;
> -	int days;
> +	time64_t days;
>  
>  	/* time must be positive */
>  	days = div_s64(time, 86400);
> -	secs = time - (unsigned int) days * 86400;
> +	secs = time - days * 86400;
>  
>  	/* day of the week, 1970-01-01 was a Thursday */
>  	tm->tm_wday = (days + 4) % 7;
> -- 
> 1.7.10.4
> 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1283548 — Re: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm

FromSasha Levin <sasha.levin@oracle.com>
Date2015-12-04 03:10 +0100
SubjectRe: [PATCH] rtc: fix overflow and incorrect calculation in rtc_time64_to_tm
Message-ID<qBNYR-4Wx-11@gated-at.bofh.it>
In reply to#1283541
On 12/03/2015 08:55 PM, Alexandre Belloni wrote:
> On 03/12/2015 at 19:12:24 -0500, Sasha Levin wrote :
>> > At some point after humans go extinct and robots cotrol the world, dividing
>> > he time64_t by 86400 to extract the days will overflow a 32bit integer,
>> > leading to incorrect conversion into rtc_time in rtc_time64_to_tm().
>> > 
> And at that time, the robots won't care about 32bit platforms :)

One can only hope... :)

I'll resend.


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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web