]> www.infradead.org Git - nvme.git/commitdiff
selftests: mptcp: lib: fix shellcheck errors
authorMatthieu Baerts (NGI0) <matttbe@kernel.org>
Fri, 12 Jul 2024 10:00:15 +0000 (12:00 +0200)
committerJakub Kicinski <kuba@kernel.org>
Sat, 13 Jul 2024 22:46:26 +0000 (15:46 -0700)
It looks like we missed these two errors recently:

  - SC2068: Double quote array expansions to avoid re-splitting elements.
  - SC2145: Argument mixes string and array. Use * or separate argument.

Two simple fixes, it is not supposed to change the behaviour as the
variable names should not have any spaces in their names. Still, better
to fix them to easily spot new issues.

Fixes: f265d3119a29 ("selftests: mptcp: lib: use setup/cleanup_ns helpers")
Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://patch.msgid.link/20240712-upstream-net-next-20240712-selftests-mptcp-fix-shellcheck-v1-1-1cb7180db40a@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/net/mptcp/mptcp_lib.sh

index 194c8fc2e55a3a398fa9bb8419cebe3ba3f49427..438280e684346ccd301d94ec6bb528809dadcb3d 100644 (file)
@@ -428,8 +428,8 @@ mptcp_lib_check_tools() {
 }
 
 mptcp_lib_ns_init() {
-       if ! setup_ns ${@}; then
-               mptcp_lib_pr_fail "Failed to setup namespace ${@}"
+       if ! setup_ns "${@}"; then
+               mptcp_lib_pr_fail "Failed to setup namespaces ${*}"
                exit ${KSFT_FAIL}
        fi