From b468dc500a865d1107282e824bab3750c11111a5 Mon Sep 17 00:00:00 2001 From: Manabu Sugimoto Date: Tue, 2 Nov 2021 19:32:54 +0900 Subject: [PATCH] agent: Use dup3 system call in unit tests of seccomp Use `dup3` system call instead of `dup2` in unit tests of seccomp because `dup2` is obsolete on aarch64. Fixes: #2939 Signed-off-by: Manabu Sugimoto --- src/agent/rustjail/src/seccomp.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/agent/rustjail/src/seccomp.rs b/src/agent/rustjail/src/seccomp.rs index 37eb175767..58e85c482f 100644 --- a/src/agent/rustjail/src/seccomp.rs +++ b/src/agent/rustjail/src/seccomp.rs @@ -95,7 +95,7 @@ pub fn init_seccomp(scmp: &LinuxSeccomp) -> Result<()> { mod tests { use super::*; use crate::skip_if_not_root; - use libc::{dup2, process_vm_readv, EPERM}; + use libc::{dup3, process_vm_readv, EPERM, O_CLOEXEC}; use std::io::Error; use std::ptr::null; @@ -135,7 +135,7 @@ mod tests { "syscalls": [ { "names": [ - "dup2" + "dup3" ], "action": "SCMP_ACT_ERRNO" }, @@ -212,7 +212,7 @@ mod tests { init_seccomp(&scmp).unwrap(); // Basic syscall with simple rule - syscall_assert!(unsafe { dup2(0, 1) }, -EPERM); + syscall_assert!(unsafe { dup3(0, 1, O_CLOEXEC) }, -EPERM); // Syscall with permitted arguments syscall_assert!(unsafe { process_vm_readv(1, null(), 0, null(), 0, 0) }, 0);