diff options
| author | Ma Ke <make24@iscas.ac.cn> | 2024-06-23 21:17:53 +0800 |
|---|---|---|
| committer | Andrii Nakryiko <andrii@kernel.org> | 2024-06-26 16:36:59 -0700 |
| commit | d07980f7373b4c57c85478f8e815cc7b9b394c05 (patch) | |
| tree | ff6d8308e02da9c27c0e04ec2e8d4b2934ee74e0 /tools/testing/selftests/bpf/prog_tests/fexit_stress.c | |
| parent | aa293983d2020390e286544b120f3cd0a3d40749 (diff) | |
selftests/bpf: Don't close(-1) in serial_test_fexit_stress()
Guard close() with extra link_fd[i] > 0 and fexit_fd[i] > 0
check to prevent close(-1).
Signed-off-by: Ma Ke <make24@iscas.ac.cn>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/bpf/20240623131753.2133829-1-make24@iscas.ac.cn
Diffstat (limited to 'tools/testing/selftests/bpf/prog_tests/fexit_stress.c')
| -rw-r--r-- | tools/testing/selftests/bpf/prog_tests/fexit_stress.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/testing/selftests/bpf/prog_tests/fexit_stress.c b/tools/testing/selftests/bpf/prog_tests/fexit_stress.c index 596536def43d..49b1ffc9af1f 100644 --- a/tools/testing/selftests/bpf/prog_tests/fexit_stress.c +++ b/tools/testing/selftests/bpf/prog_tests/fexit_stress.c @@ -50,9 +50,9 @@ void serial_test_fexit_stress(void) out: for (i = 0; i < bpf_max_tramp_links; i++) { - if (link_fd[i]) + if (link_fd[i] > 0) close(link_fd[i]); - if (fexit_fd[i]) + if (fexit_fd[i] > 0) close(fexit_fd[i]); } free(fd); |