Michal Schmidt b568ba
From 822decbb65b09fbbddde87c75996238ce642e475 Mon Sep 17 00:00:00 2001
Michal Schmidt b568ba
From: Lennart Poettering <lennart@poettering.net>
Michal Schmidt b568ba
Date: Sat, 11 Feb 2012 00:27:12 +0100
Michal Schmidt b568ba
Subject: [PATCH] util: fix handling of empty files in read_one_line_file()
Michal Schmidt b568ba
Michal Schmidt b568ba
https://bugs.freedesktop.org/show_bug.cgi?id=45362
Michal Schmidt b568ba
(cherry picked from commit 4099a281bb1e7bbb941c55de559dbfb9abf5897b)
Michal Schmidt b568ba
---
Michal Schmidt b568ba
 src/util.c |   17 ++++++++++++-----
Michal Schmidt b568ba
 1 files changed, 12 insertions(+), 5 deletions(-)
Michal Schmidt b568ba
Michal Schmidt b568ba
diff --git a/src/util.c b/src/util.c
Michal Schmidt b568ba
index d942d31..6065463 100644
Michal Schmidt b568ba
--- a/src/util.c
Michal Schmidt b568ba
+++ b/src/util.c
Michal Schmidt b568ba
@@ -704,15 +704,22 @@ int read_one_line_file(const char *fn, char **line) {
Michal Schmidt b568ba
         assert(fn);
Michal Schmidt b568ba
         assert(line);
Michal Schmidt b568ba
 
Michal Schmidt b568ba
-        if (!(f = fopen(fn, "re")))
Michal Schmidt b568ba
+        f = fopen(fn, "re");
Michal Schmidt b568ba
+        if (!f)
Michal Schmidt b568ba
                 return -errno;
Michal Schmidt b568ba
 
Michal Schmidt b568ba
-        if (!(fgets(t, sizeof(t), f))) {
Michal Schmidt b568ba
-                r = feof(f) ? -EIO : -errno;
Michal Schmidt b568ba
-                goto finish;
Michal Schmidt b568ba
+        if (!fgets(t, sizeof(t), f)) {
Michal Schmidt b568ba
+
Michal Schmidt b568ba
+                if (ferror(f)) {
Michal Schmidt b568ba
+                        r = -errno;
Michal Schmidt b568ba
+                        goto finish;
Michal Schmidt b568ba
+                }
Michal Schmidt b568ba
+
Michal Schmidt b568ba
+                t[0] = 0;
Michal Schmidt b568ba
         }
Michal Schmidt b568ba
 
Michal Schmidt b568ba
-        if (!(c = strdup(t))) {
Michal Schmidt b568ba
+        c = strdup(t);
Michal Schmidt b568ba
+        if (!c) {
Michal Schmidt b568ba
                 r = -ENOMEM;
Michal Schmidt b568ba
                 goto finish;
Michal Schmidt b568ba
         }