From: Lennart Poettering <lennart@poettering.net>
Date: Fri, 21 Oct 2016 18:03:51 +0000 (+0200)
Subject: core: simplify skip_seccomp_unavailable() a bit
X-Git-Tag: v232~39^2~8
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f673b62df67dfa67ddfa4f5a72d78ea3c002278f;p=thirdparty%2Fsystemd.git

core: simplify skip_seccomp_unavailable() a bit

Let's prefer early-exit over deep-indented if blocks. Not behavioural change.
---

diff --git a/src/core/execute.c b/src/core/execute.c
index 53356c3c06b..b69297241ba 100644
--- a/src/core/execute.c
+++ b/src/core/execute.c
@@ -1185,13 +1185,14 @@ static void rename_process_from_path(const char *path) {
 #ifdef HAVE_SECCOMP
 
 static bool skip_seccomp_unavailable(const Unit* u, const char* msg) {
-        if (!is_seccomp_available()) {
-                log_open();
-                log_unit_debug(u, "SECCOMP features not detected in the kernel, skipping %s", msg);
-                log_close();
-                return true;
-        }
-        return false;
+
+        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;
 }
 
 static int apply_seccomp(const Unit* u, const ExecContext *c) {