Skip to content

seccomp: fix inverted notify socket setup logic#656

Open
caxu-rh wants to merge 1 commit into
containers:mainfrom
caxu-rh:seccomp-notify-socket-ifdef
Open

seccomp: fix inverted notify socket setup logic#656
caxu-rh wants to merge 1 commit into
containers:mainfrom
caxu-rh:seccomp-notify-socket-ifdef

Conversation

@caxu-rh
Copy link
Copy Markdown

@caxu-rh caxu-rh commented May 15, 2026

In 3a8c913, the conditional in src/conmon.c was inadvertently reversed when the syntax was changed.

3a8c913#diff-737dc81597d3e3189077545b4907aae69cce646d12b5bc5938cfdd707882bd31R181

diff --git a/src/conmon.c b/src/conmon.c
index 795701c68f754cee5d22422e12872ca24748472a..7dd1f0b560c0e593dc289f42e35931554dcb7fd9 100644
--- a/src/conmon.c
+++ b/src/conmon.c
@@ -178,7 +178,7 @@ int main(int argc, char *argv[])
 	}
 
 	if (opt_seccomp_notify_socket != NULL) {
-#if !USE_SECCOMP
+#ifdef USE_SECCOMP
 		pexit("seccomp support not present");
 #else
 		if (opt_seccomp_notify_plugins == NULL)

In 3a8c913, the conditional in
src/conmon.c was inadvertently reversed when the syntax was
changed.

Signed-off-by: Caleb Xu <caxu@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant