From: Lennart Poettering <lennart@poettering.net>
Date: Tue, 26 Sep 2017 15:41:53 +0000 (+0200)
Subject: execute: drop explicit log_open()/log_close() now that it is unnecessary
X-Git-Tag: v235~55^2~3
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=86ffb3256030e8cf22ae17bebce77c0a7fb1637b;p=thirdparty%2Fsystemd.git

execute: drop explicit log_open()/log_close() now that it is unnecessary
---

diff --git a/src/core/execute.c b/src/core/execute.c
index edeb857e155..451a05b9de5 100644
--- a/src/core/execute.c
+++ b/src/core/execute.c
@@ -1323,9 +1323,7 @@ static bool skip_seccomp_unavailable(const Unit* u, const char* msg) {
         if (is_seccomp_available())
                 return false;
 
-        log_open();
         log_unit_debug(u, "SECCOMP features not detected in the kernel, skipping %s", msg);
-        log_close();
         return true;
 }
 
@@ -2109,10 +2107,8 @@ static int apply_mount_namespace(
         /* If we couldn't set up the namespace this is probably due to a
          * missing capability. In this case, silently proceeed. */
         if (IN_SET(r, -EPERM, -EACCES)) {
-                log_open();
                 log_unit_debug_errno(u, r, "Failed to set up namespace, assuming containerized execution, ignoring: %m");
-                log_close();
-                r = 0;
+                return 0;
         }
 
         return r;