daandemeyer / rpms / systemd

Forked from rpms/systemd 2 years ago
Clone
b677e7
From 354b894aa3e79f54ab75bf6fae76ce28ca80db38 Mon Sep 17 00:00:00 2001
b677e7
From: Lennart Poettering <lennart@poettering.net>
b677e7
Date: Thu, 2 Aug 2018 20:56:34 +0200
b677e7
Subject: [PATCH] user-runtime-dir: downgrade a few log messages to LOG_DEBUG
b677e7
 that we ignore
b677e7
b677e7
As the comments already say it might be quite likely that
b677e7
$XDG_RUNTIME_DIR is not set up as mount, and we shouldn't complain about
b677e7
that.
b677e7
b677e7
Moreover, let's make this idempotent, so that a runtime dir that is
b677e7
already gone and is removed again doesn't cause failure.
b677e7
b677e7
(cherry picked from commit 3a13442bbf72e7ebdd0b4d60c2922ea7c5cc9496)
b677e7
b677e7
Related: #1946453
b677e7
---
b677e7
 src/login/user-runtime-dir.c | 15 +++++++--------
b677e7
 1 file changed, 7 insertions(+), 8 deletions(-)
b677e7
b677e7
diff --git a/src/login/user-runtime-dir.c b/src/login/user-runtime-dir.c
b677e7
index 1bb26c99e4..9693821990 100644
b677e7
--- a/src/login/user-runtime-dir.c
b677e7
+++ b/src/login/user-runtime-dir.c
b677e7
@@ -95,20 +95,19 @@ static int user_remove_runtime_path(const char *runtime_path) {
b677e7
 
b677e7
         r = rm_rf(runtime_path, 0);
b677e7
         if (r < 0)
b677e7
-                log_error_errno(r, "Failed to remove runtime directory %s (before unmounting): %m", runtime_path);
b677e7
+                log_debug_errno(r, "Failed to remove runtime directory %s (before unmounting), ignoring: %m", runtime_path);
b677e7
 
b677e7
-        /* Ignore cases where the directory isn't mounted, as that's
b677e7
-         * quite possible, if we lacked the permissions to mount
b677e7
-         * something */
b677e7
+        /* Ignore cases where the directory isn't mounted, as that's quite possible, if we lacked the permissions to
b677e7
+         * mount something */
b677e7
         r = umount2(runtime_path, MNT_DETACH);
b677e7
         if (r < 0 && !IN_SET(errno, EINVAL, ENOENT))
b677e7
-                log_error_errno(errno, "Failed to unmount user runtime directory %s: %m", runtime_path);
b677e7
+                log_debug_errno(errno, "Failed to unmount user runtime directory %s, ignoring: %m", runtime_path);
b677e7
 
b677e7
         r = rm_rf(runtime_path, REMOVE_ROOT);
b677e7
-        if (r < 0)
b677e7
-                log_error_errno(r, "Failed to remove runtime directory %s (after unmounting): %m", runtime_path);
b677e7
+        if (r < 0 && r != -ENOENT)
b677e7
+                return log_error_errno(r, "Failed to remove runtime directory %s (after unmounting): %m", runtime_path);
b677e7
 
b677e7
-        return r;
b677e7
+        return 0;
b677e7
 }
b677e7
 
b677e7
 static int do_mount(const char *runtime_path, uid_t uid, gid_t gid) {