diff options
| author | Guillaume Nault <[email protected]> | 2022-03-08 22:15:03 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2022-03-10 04:23:32 +0000 |
| commit | 94a4a4fe4c696413932eed8bdec46574de9576b8 (patch) | |
| tree | 7b851b74e05a66adbadc2e0426975a5e0f1a0e61 /net/unix/af_unix.c | |
| parent | selftests: pmtu.sh: Kill tcpdump processes launched by subshell. (diff) | |
| download | kernel-94a4a4fe4c696413932eed8bdec46574de9576b8.tar.gz kernel-94a4a4fe4c696413932eed8bdec46574de9576b8.zip | |
selftests: pmtu.sh: Kill nettest processes launched in subshell.
When using "run_cmd <command> &", then "$!" refers to the PID of the
subshell used to run <command>, not the command itself. Therefore
nettest_pids actually doesn't contain the list of the nettest commands
running in the background. So cleanup() can't kill them and the nettest
processes run until completion (fortunately they have a 5s timeout).
Fix this by defining a new command for running processes in the
background, for which "$!" really refers to the PID of the command run.
Also, double quote variables on the modified lines, to avoid shellcheck
warnings.
Fixes: ece1278a9b81 ("selftests: net: add ESP-in-UDP PMTU test")
Signed-off-by: Guillaume Nault <[email protected]>
Reviewed-by: Shuah Khan <[email protected]>
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions