Blame 0150-util-fix-handling-of-empty-files-in-read_one_line_fi.patch
|
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 |
}
|