Skip to content

Commit c57f589

Browse files
Alexandra Iordachesandreim
authored andcommitted
seccomp: enable for unit tests (remove workaround)
Fixes #1377 Signed-off-by: Alexandra Iordache <[email protected]>
1 parent b221d31 commit c57f589

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

src/main.rs

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -149,12 +149,6 @@ fn main() {
149149
// It's safe to unwrap here because clap's been provided with a default value
150150
let instance_id = cmd_arguments.value_of("id").unwrap().to_string();
151151

152-
// We disable seccomp filtering when testing, because when running the test_gnutests
153-
// integration test from test_unittests.py, an invalid syscall is issued, and we crash
154-
// otherwise.
155-
#[cfg(test)]
156-
let seccomp_level = seccomp::SECCOMP_LEVEL_NONE;
157-
#[cfg(not(test))]
158152
// It's safe to unwrap here because clap's been provided with a default value,
159153
// and allowed values are guaranteed to parse to u32.
160154
let seccomp_level = cmd_arguments

0 commit comments

Comments
 (0)