diff options
| author | Mark Brown <broonie@kernel.org> | 2024-11-11 19:18:55 +0300 |
|---|---|---|
| committer | Catalin Marinas <catalin.marinas@arm.com> | 2024-11-12 18:05:44 +0300 |
| commit | 27141b690547da5650a420f26ec369ba142a9ebb (patch) | |
| tree | bd188425991a6b570aa2ccb2970bd5f60a7a655b | |
| parent | 3e360ef0c0a1fb6ce9a302e40b8057c41ba8a9d2 (diff) | |
| download | linux-27141b690547da5650a420f26ec369ba142a9ebb.tar.xz | |
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>
| -rw-r--r-- | tools/testing/selftests/arm64/pauth/pac.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tools/testing/selftests/arm64/pauth/pac.c b/tools/testing/selftests/arm64/pauth/pac.c index b743daa772f5..5a07b3958fbf 100644 --- a/tools/testing/selftests/arm64/pauth/pac.c +++ b/tools/testing/selftests/arm64/pauth/pac.c @@ -182,6 +182,9 @@ int exec_sign_all(struct signatures *signed_vals, size_t val) return -1; } + close(new_stdin[1]); + close(new_stdout[0]); + return 0; } |