diff options
Diffstat (limited to 'src/shared')
| -rw-r--r-- | src/shared/generator.c | 9 | ||||
| -rw-r--r-- | src/shared/path-util.c | 21 |
2 files changed, 25 insertions, 5 deletions
diff --git a/src/shared/generator.c b/src/shared/generator.c index 5ac7b5f02f..dcfddaa731 100644 --- a/src/shared/generator.c +++ b/src/shared/generator.c @@ -49,10 +49,13 @@ int generator_write_fsck_deps( if (!isempty(fstype) && !streq(fstype, "auto")) { int r; r = fsck_exists(fstype); - if (r < 0) { - log_warning("Checking was requested for %s, but fsck.%s cannot be used: %s", what, fstype, strerror(-r)); + if (r == -ENOENT) { /* treat missing check as essentially OK */ - return r == -ENOENT ? 0 : r; + log_debug("Checking was requested for %s, but fsck.%s does not exist: %s", what, fstype, strerror(-r)); + return 0; + } else if (r < 0) { + log_warning("Checking was requested for %s, but fsck.%s cannot be used: %s", what, fstype, strerror(-r)); + return r; } } diff --git a/src/shared/path-util.c b/src/shared/path-util.c index fd35e0c786..e68d3677a7 100644 --- a/src/shared/path-util.c +++ b/src/shared/path-util.c @@ -541,7 +541,7 @@ int path_is_os_tree(const char *path) { int find_binary(const char *name, char **filename) { assert(name); - if (strchr(name, '/')) { + if (is_path(name)) { if (access(name, X_OK) < 0) return -errno; @@ -626,8 +626,25 @@ bool paths_check_timestamp(const char* const* paths, usec_t *timestamp, bool upd } int fsck_exists(const char *fstype) { + _cleanup_free_ char *p = NULL, *d = NULL; const char *checker; + int r; checker = strappenda("fsck.", fstype); - return find_binary(checker, NULL); + + r = find_binary(checker, &p); + if (r < 0) + return r; + + /* An fsck that is linked to /bin/true is a non-existant + * fsck */ + + r = readlink_malloc(p, &d); + if (r >= 0 && + (path_equal(d, "/bin/true") || + path_equal(d, "/usr/bin/true") || + path_equal(d, "/dev/null"))) + return -ENOENT; + + return 0; } |