busybox: backport Remove stime() function calls
authorHauke Mehrtens <hauke@hauke-m.de>
Mon, 23 Mar 2020 22:22:04 +0000 (23:22 +0100)
committerHauke Mehrtens <hauke@hauke-m.de>
Sat, 18 Apr 2020 19:06:36 +0000 (21:06 +0200)
glibc 2.31 does not provide stime() any more, backport a fix from
current busybox master to avoid using this function.

Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
package/utils/busybox/patches/001-remove-stime-function-calls.patch [new file with mode: 0644]

diff --git a/package/utils/busybox/patches/001-remove-stime-function-calls.patch b/package/utils/busybox/patches/001-remove-stime-function-calls.patch
new file mode 100644 (file)
index 0000000..ccf9bef
--- /dev/null
@@ -0,0 +1,84 @@
+From d3539be8f27b8cbfdfee460fe08299158f08bcd9 Mon Sep 17 00:00:00 2001
+From: Alistair Francis <alistair.francis@wdc.com>
+Date: Tue, 19 Nov 2019 13:06:40 +0100
+Subject: Remove stime() function calls
+
+stime() has been deprecated in glibc 2.31 and replaced with
+clock_settime(). Let's replace the stime() function calls with
+clock_settime() in preperation.
+
+function                                             old     new   delta
+rdate_main                                           197     224     +27
+clock_settime                                          -      27     +27
+date_main                                            926     941     +15
+stime                                                 37       -     -37
+------------------------------------------------------------------------------
+(add/remove: 2/2 grow/shrink: 2/0 up/down: 69/-37)             Total: 32 bytes
+
+Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
+Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
+---
+ coreutils/date.c         | 6 +++++-
+ libbb/missing_syscalls.c | 8 --------
+ util-linux/rdate.c       | 8 ++++++--
+ 3 files changed, 11 insertions(+), 11 deletions(-)
+
+--- a/coreutils/date.c
++++ b/coreutils/date.c
+@@ -279,6 +279,9 @@ int date_main(int argc UNUSED_PARAM, cha
+               time(&ts.tv_sec);
+ #endif
+       }
++#if !ENABLE_FEATURE_DATE_NANO
++      ts.tv_nsec = 0;
++#endif
+       localtime_r(&ts.tv_sec, &tm_time);
+       /* If date string is given, update tm_time, and maybe set date */
+@@ -301,9 +304,10 @@ int date_main(int argc UNUSED_PARAM, cha
+               if (date_str[0] != '@')
+                       tm_time.tm_isdst = -1;
+               ts.tv_sec = validate_tm_time(date_str, &tm_time);
++              ts.tv_nsec = 0;
+               /* if setting time, set it */
+-              if ((opt & OPT_SET) && stime(&ts.tv_sec) < 0) {
++              if ((opt & OPT_SET) && clock_settime(CLOCK_REALTIME, &ts) < 0) {
+                       bb_perror_msg("can't set date");
+               }
+       }
+--- a/libbb/missing_syscalls.c
++++ b/libbb/missing_syscalls.c
+@@ -15,14 +15,6 @@ pid_t getsid(pid_t pid)
+       return syscall(__NR_getsid, pid);
+ }
+-int stime(const time_t *t)
+-{
+-      struct timeval tv;
+-      tv.tv_sec = *t;
+-      tv.tv_usec = 0;
+-      return settimeofday(&tv, NULL);
+-}
+-
+ int sethostname(const char *name, size_t len)
+ {
+       return syscall(__NR_sethostname, name, len);
+--- a/util-linux/rdate.c
++++ b/util-linux/rdate.c
+@@ -95,9 +95,13 @@ int rdate_main(int argc UNUSED_PARAM, ch
+       if (!(flags & 2)) { /* no -p (-s may be present) */
+               if (time(NULL) == remote_time)
+                       bb_error_msg("current time matches remote time");
+-              else
+-                      if (stime(&remote_time) < 0)
++              else {
++                      struct timespec ts;
++                      ts.tv_sec = remote_time;
++                      ts.tv_nsec = 0;
++                      if (clock_settime(CLOCK_REALTIME, &ts) < 0)
+                               bb_perror_msg_and_die("can't set time of day");
++              }
+       }
+       if (flags != 1) /* not lone -s */