From 86ffb3256030e8cf22ae17bebce77c0a7fb1637b Mon Sep 17 00:00:00 2001
From: Lennart Poettering <lennart@poettering.net>
Date: Tue, 26 Sep 2017 17:41:53 +0200
Subject: [PATCH] execute: drop explicit log_open()/log_close() now that it is
 unnecessary

---
 src/core/execute.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/core/execute.c b/src/core/execute.c
index edeb857e15..451a05b9de 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;