chiark / gitweb /
fs-util: refuse taking a relative path to chase if "root" is specified and CHASE_PREF...
authorLennart Poettering <lennart@poettering.net>
Wed, 17 Jan 2018 11:00:40 +0000 (12:00 +0100)
committerSven Eden <yamakuzure@gmx.net>
Wed, 30 May 2018 05:50:10 +0000 (07:50 +0200)
If we take a relative path we first make it absolute, based on the
current working directory. But if CHASE_PREFIX_ROOT is passe we are
supposed to make the path absolute taking the specified root path into
account, but that makes no sense if we talk about the current working
directory as that is relative to the host's root in any case. Hence,
let's refuse this politely.

src/basic/fs-util.c

index bbc464e31c8de935d6f7a09affd800af53c1cd4d..bd7e3db64207f3034e6f9a26824f2d59a5d82dca 100644 (file)
@@ -688,8 +688,14 @@ int chase_symlinks(const char *path, const char *original_root, unsigned flags,
                 if (r < 0)
                         return r;
 
-                if (flags & CHASE_PREFIX_ROOT)
+                if (flags & CHASE_PREFIX_ROOT) {
+
+                        /* We don't support relative paths in combination with a root directory */
+                        if (!path_is_absolute(path))
+                                return -EINVAL;
+
                         path = prefix_roota(root, path);
+                }
         }
 
         r = path_make_absolute_cwd(path, &buffer);