1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
|
From: Michael Olbrich <m.olbrich@pengutronix.de>
Date: Sun, 29 Sep 2019 13:43:00 +0200
Subject: [PATCH] Handle d_type == DT_UNKNOWN correctly
As documented in the man-page, readdir() may return a directory entry with
d_type == DT_UNKNOWN. This must be handled for regular filesystems.
dirent_ensure_type() is available to set d_type if necessary. Use it in
some more places.
Without this systemd will fail to boot correctly with nfsroot and some
other filesystems.
Closes #13609
---
src/core/smack-setup.c | 2 ++
src/login/logind-dbus.c | 1 +
src/login/logind.c | 1 +
src/shared/bootspec.c | 1 +
src/shared/unit-file.c | 1 +
5 files changed, 6 insertions(+)
diff --git a/src/core/smack-setup.c b/src/core/smack-setup.c
index b95e6239d4b2..e2d4c0d6e172 100644
--- a/src/core/smack-setup.c
+++ b/src/core/smack-setup.c
@@ -84,6 +84,7 @@ static int write_access2_rules(const char *srcdir) {
FOREACH_DIRENT(entry, dir, return 0) {
_cleanup_fclose_ FILE *policy = NULL;
+ dirent_ensure_type(dir, entry);
if (!dirent_is_file(entry))
continue;
@@ -150,6 +151,7 @@ static int write_cipso2_rules(const char *srcdir) {
FOREACH_DIRENT(entry, dir, return 0) {
_cleanup_fclose_ FILE *policy = NULL;
+ dirent_ensure_type(dir, entry);
if (!dirent_is_file(entry))
continue;
diff --git a/src/login/logind-dbus.c b/src/login/logind-dbus.c
index 30b9a66334fa..679eb6a62ec0 100644
--- a/src/login/logind-dbus.c
+++ b/src/login/logind-dbus.c
@@ -1378,6 +1378,7 @@ static int flush_devices(Manager *m) {
struct dirent *de;
FOREACH_DIRENT_ALL(de, d, break) {
+ dirent_ensure_type(d, de);
if (!dirent_is_file(de))
continue;
diff --git a/src/login/logind.c b/src/login/logind.c
index 3a16e6c8712a..d889d7b9bec9 100644
--- a/src/login/logind.c
+++ b/src/login/logind.c
@@ -291,6 +291,7 @@ static int manager_enumerate_linger_users(Manager *m) {
FOREACH_DIRENT(de, d, return -errno) {
int k;
+ dirent_ensure_type(d, de);
if (!dirent_is_file(de))
continue;
diff --git a/src/shared/bootspec.c b/src/shared/bootspec.c
index b0c6e4885fa0..5e2aa28bcff4 100644
--- a/src/shared/bootspec.c
+++ b/src/shared/bootspec.c
@@ -475,6 +475,7 @@ static int boot_entries_find_unified(
_cleanup_free_ char *j = NULL, *osrelease = NULL, *cmdline = NULL;
_cleanup_close_ int fd = -1;
+ dirent_ensure_type(d, de);
if (!dirent_is_file(de))
continue;
diff --git a/src/shared/unit-file.c b/src/shared/unit-file.c
index 4a5f23e6c1bc..c7a6d6717244 100644
--- a/src/shared/unit-file.c
+++ b/src/shared/unit-file.c
@@ -279,6 +279,7 @@ int unit_file_build_name_map(
if (hashmap_contains(ids, de->d_name))
continue;
+ dirent_ensure_type(d, de);
if (de->d_type == DT_LNK) {
/* We don't explicitly check for alias loops here. unit_ids_map_get() which
* limits the number of hops should be used to access the map. */
|