|
Zbigniew Jędrzejewski-Szmek |
7f93bc |
From 5763621e1fafe630cfe98181792e7b556db72d21 Mon Sep 17 00:00:00 2001
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
From: Lennart Poettering <lennart@poettering.net>
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
Date: Tue, 17 Jun 2014 01:37:54 +0200
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
Subject: [PATCH] install: simplify symlink --root= logic
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
(cherry picked from commit 278fa5758c8e30f03c8c50f15873d55edfc4cbaf)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
(cherry picked from commit aa5882a2d14c8de0956be0569f20678816dfa1f5)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
---
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
src/shared/install.c | 59 +++++++++++++---------------------------------------
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
1 file changed, 15 insertions(+), 44 deletions(-)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
diff --git a/src/shared/install.c b/src/shared/install.c
|
|
Zbigniew Jędrzejewski-Szmek |
1cc3df |
index 5d93a2eaf3..86a05a47a6 100644
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
--- a/src/shared/install.c
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+++ b/src/shared/install.c
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
@@ -47,9 +47,8 @@ typedef struct {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
#define _cleanup_lookup_paths_free_ _cleanup_(lookup_paths_free)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
#define _cleanup_install_context_done_ _cleanup_(install_context_done)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
-static int in_search_path(const char *path, char **search, const char *root_dir) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+static int in_search_path(const char *path, char **search) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
_cleanup_free_ char *parent = NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- char **i;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
int r;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(path);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
@@ -58,24 +57,7 @@ static int in_search_path(const char *path, char **search, const char *root_dir)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
if (r < 0)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
return r;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- STRV_FOREACH(i, search) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- _cleanup_free_ char *buf = NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- const char *p;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- if (root_dir) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- buf = strjoin(root_dir, "/", *i, NULL);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- if (!buf)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- return -ENOMEM;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- p = buf;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- } else
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- p = *i;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- if (path_equal(parent, p))
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- return 1;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- }
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- return 0;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ return strv_contains(search, parent);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
static int lookup_paths_init_from_scope(LookupPaths *paths,
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
@@ -790,7 +772,7 @@ int unit_file_link(
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
continue;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- q = in_search_path(*i, paths.unit_path, root_dir);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ q = in_search_path(*i, paths.unit_path);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
if (q < 0)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
return q;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
@@ -1037,6 +1019,7 @@ static int unit_file_load(
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
InstallContext *c,
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
InstallInfo *info,
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
const char *path,
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ const char *root_dir,
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
bool allow_symlink) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
const ConfigTableItem items[] = {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
@@ -1048,14 +1031,16 @@ static int unit_file_load(
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
{ NULL, NULL, NULL, 0, NULL }
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
};
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- int fd;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
_cleanup_fclose_ FILE *f = NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- int r;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ int fd, r;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(c);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(info);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(path);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ if (!isempty(root_dir))
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ path = strappenda3(root_dir, "/", path);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
fd = open(path, O_RDONLY|O_CLOEXEC|O_NOCTTY|(allow_symlink ? 0 : O_NOFOLLOW));
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
if (fd < 0)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
return -errno;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
@@ -1091,30 +1076,19 @@ static int unit_file_search(
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(info);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(paths);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- if (info->path) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- const char *path;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- if (isempty(root_dir))
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- path = info->path;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- else
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- path = strappenda(root_dir, info->path);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- return unit_file_load(c, info, path, allow_symlink);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- }
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ if (info->path)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ return unit_file_load(c, info, info->path, root_dir, allow_symlink);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(info->name);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
STRV_FOREACH(p, paths->unit_path) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
_cleanup_free_ char *path = NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- if (isempty(root_dir))
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- path = strjoin(*p, "/", info->name, NULL);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- else
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- path = strjoin(root_dir, "/", *p, "/", info->name, NULL);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ path = strjoin(*p, "/", info->name, NULL);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
if (!path)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
return -ENOMEM;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- r = unit_file_load(c, info, path, allow_symlink);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ r = unit_file_load(c, info, path, root_dir, allow_symlink);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
if (r >= 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
info->path = path;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
path = NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
@@ -1139,14 +1113,11 @@ static int unit_file_search(
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
STRV_FOREACH(p, paths->unit_path) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
_cleanup_free_ char *path = NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- if (isempty(root_dir))
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- path = strjoin(*p, "/", template, NULL);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- else
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- path = strjoin(root_dir, "/", *p, "/", template, NULL);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ path = strjoin(*p, "/", template, NULL);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
if (!path)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
return -ENOMEM;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- r = unit_file_load(c, info, path, allow_symlink);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ r = unit_file_load(c, info, path, root_dir, allow_symlink);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
if (r >= 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
info->path = path;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
path = NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
@@ -1359,7 +1330,7 @@ static int install_info_symlink_link(
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(config_path);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
assert(i->path);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
- r = in_search_path(i->path, paths->unit_path, root_dir);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
+ r = in_search_path(i->path, paths->unit_path);
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
if (r != 0)
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
return r;
|
|
Zbigniew Jędrzejewski-Szmek |
a59965 |
|