From 10191da63a35317aea1d2286225efc03734c10d5 Mon Sep 17 00:00:00 2001 From: J William Piggott Date: Mon, 25 Sep 2017 19:30:24 -0400 Subject: hwclock: add iso-8601 overflow check hwclock wasn't testing for strtimeval_iso() truncation: /sbin/hwclock --utc --noadjfile --predict --date '7982 years'; echo $? 9999-09-25 19:33:01.000000-0400 0 /sbin/hwclock --utc --noadjfile --predict --date '7983 years'; echo $? 10000-09-25 19:33:10.000000- 0 Patched: ./hwclock --utc --noadjfile --predict --date '7982 years'; echo $? 9999-09-25 19:22:15.000000-0400 0 ./hwclock --utc --noadjfile --predict --date '7983 years'; echo $? hwclock: iso-8601 format truncated 1 Signed-off-by: J William Piggott --- sys-utils/hwclock.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) (limited to 'sys-utils/hwclock.c') diff --git a/sys-utils/hwclock.c b/sys-utils/hwclock.c index 36b6204b0..c2c20812c 100644 --- a/sys-utils/hwclock.c +++ b/sys-utils/hwclock.c @@ -551,15 +551,19 @@ set_hardware_clock_exact(const struct hwclock_control *ctl, set_hardware_clock(ctl, newhwtime); } -static void +static int display_time(struct timeval hwctime) { char buf[ISO_8601_BUFSIZ]; - strtimeval_iso(&hwctime, ISO_8601_DATE|ISO_8601_TIME|ISO_8601_DOTUSEC| + if (strtimeval_iso(&hwctime, ISO_8601_DATE|ISO_8601_TIME|ISO_8601_DOTUSEC| ISO_8601_TIMEZONE|ISO_8601_SPACE, - buf, sizeof(buf)); + buf, sizeof(buf))) { + warnx(_("iso-8601 format overflow")); + return EXIT_FAILURE; + } printf("%s\n", buf); + return EXIT_SUCCESS; } /* @@ -931,8 +935,7 @@ manipulate_clock(const struct hwclock_control *ctl, const time_t set_time, printf(_ ("Target date: %ld\n"), set_time); printf(_ ("Predicted RTC: %ld\n"), hclocktime.tv_sec); } - display_time(hclocktime); - return EXIT_SUCCESS; + return display_time(hclocktime); } if (ctl->systz) @@ -978,7 +981,7 @@ manipulate_clock(const struct hwclock_control *ctl, const time_t set_time, time_inc(hclocktime, time_diff(startup_time, read_time)); } if (ctl->show || ctl->get) { - display_time(startup_hclocktime); + return display_time(startup_hclocktime); } else if (ctl->set) { set_hardware_clock_exact(ctl, set_time, startup_time); if (!ctl->noadjfile) -- cgit v1.2.3-55-g7522