tree-wide: trivial tweaks

diff --git a/src/basic/path-util.c b/src/basic/path-util.c
index 94527eff4caa48fb3f954cc09a60f920d6c115c5..e40ab3f5b6f3f836e2f1ab01015b72db9f6f3565 100644 (file)
--- a/src/basic/path-util.c
+++ b/src/basic/path-util.c
@@ -1055,7 +1055,7 @@ int path_extract_filename(const char *path, char **ret) {
                 return -ENOMEM;
 
         *ret = TAKE_PTR(a);
-        return strlen(c) > (size_t)r ? O_DIRECTORY : 0;
+        return strlen(c) > (size_t) r ? O_DIRECTORY : 0;
 }
 
 int path_extract_directory(const char *path, char **ret) {
@@ -1109,7 +1109,7 @@ bool filename_is_valid(const char *p) {
         if (isempty(p))
                 return false;
 
-        if (dot_or_dot_dot(p))
+        if (dot_or_dot_dot(p)) /* Yes, in this context we consider "." and ".." invalid */
                 return false;
 
         e = strchrnul(p, '/');

diff --git a/src/basic/path-util.h b/src/basic/path-util.h
index 3413056b310d878e62ad6295e5d96690985d7586..bb200872216068875c7d1fc98b80a1cbe2a17dc8 100644 (file)
--- a/src/basic/path-util.h
+++ b/src/basic/path-util.h
@@ -161,10 +161,10 @@ int path_extract_directory(const char *path, char **ret);
 bool filename_is_valid(const char *p) _pure_;
 bool path_is_valid_full(const char *p, bool accept_dot_dot) _pure_;
 static inline bool path_is_valid(const char *p) {
-        return path_is_valid_full(p, true);
+        return path_is_valid_full(p, /* accept_dot_dot= */ true);
 }
 static inline bool path_is_safe(const char *p) {
-        return path_is_valid_full(p, false);
+        return path_is_valid_full(p, /* accept_dot_dot= */ false);
 }
 bool path_is_normalized(const char *p) _pure_;