path-util: introduce filename_or_absolute_path_is_valid() helper

diff --git a/src/basic/path-util.h b/src/basic/path-util.h
index 647a884e39844f6ae19a2d99f09646ff05a10e10..671c1363c41acee7044aa572d11b6526cfc2975e 100644 (file)
--- a/src/basic/path-util.h
+++ b/src/basic/path-util.h
@@ -145,6 +145,12 @@ static inline bool path_is_safe(const char *p) {
         return path_is_valid_full(p, /* accept_dot_dot= */ false);
 }
 bool path_is_normalized(const char *p) _pure_;
+static inline bool filename_or_absolute_path_is_valid(const char *p) {
+        if (path_is_absolute(p))
+                return path_is_valid(p);
+
+        return filename_is_valid(p);
+}
 
 int file_in_same_dir(const char *path, const char *filename, char **ret);
 

diff --git a/src/core/dbus-execute.c b/src/core/dbus-execute.c
index 34511181bfe48351e5238b806ca9fd8e752395d8..560594d8a158835726c007f6377e6b416eacecc5 100644 (file)
--- a/src/core/dbus-execute.c
+++ b/src/core/dbus-execute.c
@@ -1512,7 +1512,7 @@ int bus_set_transient_exec_command(
                 if (r < 0)
                         return r;
 
-                if (!path_is_absolute(path) && !filename_is_valid(path))
+                if (!filename_or_absolute_path_is_valid(path))
                         return sd_bus_error_setf(error, SD_BUS_ERROR_INVALID_ARGS,
                                                  "\"%s\" is neither a valid executable name nor an absolute path",
                                                  path);

diff --git a/src/core/load-fragment.c b/src/core/load-fragment.c
index e718dbc76b935c944f668a0ccf1ce668ada8e24c..ca95f334079114d85f71cd90e4a80711eace54ba 100644 (file)
--- a/src/core/load-fragment.c
+++ b/src/core/load-fragment.c
@@ -974,7 +974,7 @@ int config_parse_exec(
                         return ignore ? 0 : -ENOEXEC;
                 }
 
-                if (!(path_is_absolute(path) ? path_is_valid(path) : filename_is_valid(path))) {
+                if (!filename_or_absolute_path_is_valid(path)) {
                         log_syntax(unit, ignore ? LOG_WARNING : LOG_ERR, filename, line, 0,
                                    "Neither a valid executable name nor an absolute path%s: %s",
                                    ignore ? ", ignoring" : "", path);