Skip to content

Commit 27141b6

Browse files
brooniectmarinas
authored andcommitted
kselftest/arm64: Don't leak pipe fds in pac.exec_sign_all()
The PAC exec_sign_all() test spawns some child processes, creating pipes to be stdin and stdout for the child. It cleans up most of the file descriptors that are created as part of this but neglects to clean up the parent end of the child stdin and stdout. Add the missing close() calls. Signed-off-by: Mark Brown <broonie@kernel.org> Link: https://lore.kernel.org/r/20241111-arm64-pac-test-collisions-v1-1-171875f37e44@kernel.org Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
1 parent 3e360ef commit 27141b6

File tree

1 file changed

+3
-0
lines changed
  • tools/testing/selftests/arm64/pauth

1 file changed

+3
-0
lines changed

tools/testing/selftests/arm64/pauth/pac.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -182,6 +182,9 @@ int exec_sign_all(struct signatures *signed_vals, size_t val)
182182
return -1;
183183
}
184184

185+
close(new_stdin[1]);
186+
close(new_stdout[0]);
187+
185188
return 0;
186189
}
187190

0 commit comments

Comments
 (0)