[systemd-commits] src/core

Lennart Poettering lennart at kemper.freedesktop.org
Thu Feb 12 03:23:07 PST 2015


 src/core/execute.c |    3 ++-
 src/core/execute.h |    1 +
 src/core/unit.c    |    4 ++++
 3 files changed, 7 insertions(+), 1 deletion(-)

New commits:
commit 4c08c8242a687e00b289e948ccd07b96f0bc4866
Author: Lennart Poettering <lennart at poettering.net>
Date:   Thu Feb 12 12:21:16 2015 +0100

    core: don't fail to run services in --user instances if $HOME is missing
    
    Otherwise we cannot even invoke systemd-exit.service anymore, thus not
    even exit.
    
    https://bugs.freedesktop.org/show_bug.cgi?id=83100
    https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=759320

diff --git a/src/core/execute.c b/src/core/execute.c
index 7b071e8..1f6072c 100644
--- a/src/core/execute.c
+++ b/src/core/execute.c
@@ -1617,7 +1617,8 @@ static int exec_child(
                                 return -errno;
                         }
 
-                if (chdir(context->working_directory ? context->working_directory : "/") < 0) {
+                if (chdir(context->working_directory ?: "/") < 0 &&
+                    !context->working_directory_missing_ok) {
                         *exit_status = EXIT_CHDIR;
                         return -errno;
                 }
diff --git a/src/core/execute.h b/src/core/execute.h
index 2c20139..153867c 100644
--- a/src/core/execute.h
+++ b/src/core/execute.h
@@ -99,6 +99,7 @@ struct ExecContext {
 
         struct rlimit *rlimit[_RLIMIT_MAX];
         char *working_directory, *root_directory;
+        bool working_directory_missing_ok;
 
         mode_t umask;
         int oom_score_adjust;
diff --git a/src/core/unit.c b/src/core/unit.c
index 67425ba..514b649 100644
--- a/src/core/unit.c
+++ b/src/core/unit.c
@@ -3178,6 +3178,10 @@ int unit_patch_contexts(Unit *u) {
                         r = get_home_dir(&ec->working_directory);
                         if (r < 0)
                                 return r;
+
+                        /* Allow user services to run, even if the
+                         * home directory is missing */
+                        ec->working_directory_missing_ok = true;
                 }
 
                 if (u->manager->running_as == SYSTEMD_USER &&



More information about the systemd-commits mailing list