From 6dadf44d7062c38c3f4f782da3cd88114dceb959 Mon Sep 17 00:00:00 2001

From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>

Date: Mon, 21 Apr 2014 19:17:40 -0400

Subject: [PATCH] shared/install: do not prefix created symlink with root path

Before: /var/tmp/inst1//etc/systemd/system/default.target -> /var/tmp/inst1//usr/lib/systemd/system/graphical.target

After: /var/tmp/inst1/etc/systemd/system/default.target -> /usr/lib/systemd/system/graphical.target

(cherry picked from commit 62b002337727093c21d020c730bd65971f7783a7)

Related: #1111199

---

 src/shared/install.c | 89 +++++++++++++++++++++++++---------------------------

 1 file changed, 43 insertions(+), 46 deletions(-)

diff --git a/src/shared/install.c b/src/shared/install.c

index 7dad66d..955a9bc 100644

--- a/src/shared/install.c

+++ b/src/shared/install.c

@@ -1073,67 +1073,64 @@ static int unit_file_search(

         assert(info);

         assert(paths);

-        if (info->path)

-                return unit_file_load(c, info, info->path, allow_symlink);

+        if (info->path) {

+                char *full_path = NULL;

+

+                if (!isempty(root_dir))

+                        full_path = strappenda(root_dir, info->path);

+

+                return unit_file_load(c, info, full_path ?: info->path, allow_symlink);

+        }

         assert(info->name);

         STRV_FOREACH(p, paths->unit_path) {

-                char *path = NULL;

-

-                if (isempty(root_dir))

-                        asprintf(&path, "%s/%s", *p, info->name);

-                else

-                        asprintf(&path, "%s/%s/%s", root_dir, *p, info->name);

+                _cleanup_free_ char *path = NULL, *full_path = NULL;

+                path = strjoin(*p, "/", info->name, NULL);

                 if (!path)

                         return -ENOMEM;

-                r = unit_file_load(c, info, path, allow_symlink);

+                if (!isempty(root_dir)) {

+                        full_path = strappend(root_dir, path);

+                        if (!full_path)

+                                return -ENOMEM;

+                }

-                if (r >= 0)

+                r = unit_file_load(c, info, full_path ?: path, allow_symlink);

+                if (r >= 0) {

                         info->path = path;

-                else {

-                        if (r == -ENOENT && unit_name_is_instance(info->name)) {

-                                /* Unit file doesn't exist, however instance enablement was requested.

-                                 * We will check if it is possible to load template unit file. */

-                                char *template = NULL,

-                                     *template_path = NULL,

-                                     *template_dir = NULL;

-

-                                template = unit_name_template(info->name);

-                                if (!template) {

-                                        free(path);

-                                        return -ENOMEM;

-                                }

+                        path = NULL;

+                } else if (r == -ENOENT && unit_name_is_instance(info->name)) {

+                        /* Unit file doesn't exist, however instance enablement was requested.

+                         * We will check if it is possible to load template unit file. */

+                        _cleanup_free_ char *template = NULL, *template_dir = NULL;

+

+                        template = unit_name_template(info->name);

+                        if (!template)

+                                return -ENOMEM;

-                                /* We will reuse path variable since we don't need it anymore. */

-                                template_dir = path;

-                                *(strrchr(path, '/') + 1) = '\0';

+                        /* We will reuse path variable since we don't need it anymore. */

+                        template_dir = path;

+                        *(strrchr(template_dir, '/') + 1) = '\0';

-                                template_path = strjoin(template_dir, template, NULL);

-                                if (!template_path) {

-                                        free(path);

-                                        free(template);

-                                        return -ENOMEM;

-                                }

+                        path = strappend(template_dir, template);

+                        if (!path)

+                                return -ENOMEM;

-                                /* Let's try to load template unit. */

-                                r = unit_file_load(c, info, template_path, allow_symlink);

-                                if (r >= 0) {

-                                        info->path = strdup(template_path);

-                                        if (!info->path) {

-                                                free(path);

-                                                free(template);

-                                                free(template_path);

-                                                return -ENOMEM;

-                                        }

-                                }

+                        if (!isempty(root_dir)) {

+                                free(full_path);

+                                full_path = strappend(root_dir, path);

+                                if (!full_path)

+                                        return -ENOMEM;

+                        }

-                                free(template);

-                                free(template_path);

+                        /* Let's try to load template unit. */

+                        r = unit_file_load(c, info, full_path ?: path, allow_symlink);

+                        if (r >= 0) {

+                                info->path = path;

+                                path = NULL;

                         }

-                        free(path);

                 }

                 if (r != -ENOENT && r != -ELOOP)