clock-util: excorcise fgets()

diff --git a/src/basic/clock-util.c b/src/basic/clock-util.c
index 8555cc795251b3ad5f9c0cadfc8e84261c9e23b2..b77e780bed893169b053050c740758569d8855a7 100644 (file)
--- a/src/basic/clock-util.c
+++ b/src/basic/clock-util.c
@@ -10,8 +10,11 @@
 #include <sys/ioctl.h>
 #include <sys/time.h>
 
+#include "alloc-util.h"
 #include "clock-util.h"
+#include "def.h"
 #include "fd-util.h"
+#include "fileio.h"
 #include "macro.h"
 #include "string-util.h"
 #include "util.h"
@@ -54,6 +57,7 @@ int clock_set_hwclock(const struct tm *tm) {
 
 int clock_is_localtime(const char* adjtime_path) {
         _cleanup_fclose_ FILE *f;
+        int r;
 
         if (!adjtime_path)
                 adjtime_path = "/etc/adjtime";
@@ -67,24 +71,30 @@ int clock_is_localtime(const char* adjtime_path) {
          */
         f = fopen(adjtime_path, "re");
         if (f) {
-                char line[LINE_MAX];
-                bool b;
+                _cleanup_free_ char *line = NULL;
+                unsigned i;
+
+                for (i = 0; i < 2; i++) { /* skip the first two lines */
+                        r = read_line(f, LONG_LINE_MAX, NULL);
+                        if (r < 0)
+                                return r;
+                        if (r == 0)
+                                return false; /* less than three lines → default to UTC */
+                }
+
+                r = read_line(f, LONG_LINE_MAX, &line);
+                if (r < 0)
+                        return r;
+                if (r == 0)
+                        return false; /* less than three lines → default to UTC */
 
-                b = fgets(line, sizeof(line), f) &&
-                        fgets(line, sizeof(line), f) &&
-                        fgets(line, sizeof(line), f);
-                if (!b)
-                        /* less than three lines -> default to UTC */
-                        return 0;
-
-                truncate_nl(line);
                 return streq(line, "LOCAL");
 
         } else if (errno != ENOENT)
                 return -errno;
 
-        /* adjtime not present -> default to UTC */
-        return 0;
+        /* adjtime not present → default to UTC */
+        return false;
 }
 
 int clock_set_timezone(int *min) {