we need to pass back the rtc device name that was actually used so that we can turn around and check its sysfs status

2 files changed, 8 insertions, 8 deletions

diff --git a/util-linux/hwclock.c b/util-linux/hwclock.c
index b581d2604..f226535fa 100644
--- a/util-linux/hwclock.c
+++ b/util-linux/hwclock.c
@@ -25,7 +25,7 @@ static time_t read_rtc(int utc)
         time_t ret;
         int fd;
 
-        fd = rtc_xopen(rtcname, O_RDONLY);
+        fd = rtc_xopen(&rtcname, O_RDONLY);
         ret = rtc_read_time(fd, utc);
         close(fd);
 
@@ -35,7 +35,7 @@ static time_t read_rtc(int utc)
 static void write_rtc(time_t t, int utc)
 {
         struct tm tm;
-        int rtc = rtc_xopen(rtcname, O_WRONLY);
+        int rtc = rtc_xopen(&rtcname, O_WRONLY);
 
         tm = *(utc ? gmtime(&t) : localtime(&t));
         tm.tm_isdst = 0;
diff --git a/util-linux/rtcwake.c b/util-linux/rtcwake.c
index c7d5a4c12..2b1ab1fca 100644
--- a/util-linux/rtcwake.c
+++ b/util-linux/rtcwake.c
@@ -45,7 +45,7 @@ static int may_wakeup(const char *rtcname)
                 return 0;
 
         /* wakeup events could be disabled or not supported */
-        return strcmp(buf, "enabled\n") == 0;
+        return strncmp(buf, "enabled\n", 8) == 0;
 }
 
 static void setup_alarm(int fd, time_t *wakeup)
@@ -159,11 +159,11 @@ int rtcwake_main(int argc, char **argv)
         /* the rtcname is relative to /dev */
         xchdir("/dev");
 
-        if (strcmp(suspend, "on") != 0 && !may_wakeup(rtcname))
-                bb_error_msg_and_die("%s not enabled for wakeup events", rtcname);
-
         /* this RTC must exist and (if we'll sleep) be wakeup-enabled */
-        fd = rtc_xopen(rtcname, O_RDONLY);
+        fd = rtc_xopen(&rtcname, O_RDONLY);
+
+        if (strcmp(suspend, "on") && !may_wakeup(rtcname))
+                bb_error_msg_and_die("%s not enabled for wakeup events", rtcname);
 
         /* relative or absolute alarm time, normalized to time_t */
         sys_time = time(0);
@@ -184,7 +184,7 @@ int rtcwake_main(int argc, char **argv)
         fflush(stdout);
         usleep(10 * 1000);
 
-        if (!strcmp(suspend, "on"))
+        if (strcmp(suspend, "on"))
                 suspend_system(suspend);
         else {
                 /* "fake" suspend ... we'll do the delay ourselves */