selftests: mptcp: uniform 'rndh' variable
authorMatthieu Baerts <matthieu.baerts@tessares.net>
Wed, 30 Nov 2022 14:06:25 +0000 (15:06 +0100)
committerJakub Kicinski <kuba@kernel.org>
Fri, 2 Dec 2022 04:06:06 +0000 (20:06 -0800)
The definition of 'rndh' was probably copied from one script to another
but some times, 'sec' was not defined, not used and/or not spelled
properly.

Here all the 'rndh' are now defined the same way.

Reviewed-by: Mat Martineau <mathew.j.martineau@linux.intel.com>
Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/net/mptcp/diag.sh
tools/testing/selftests/net/mptcp/mptcp_connect.sh
tools/testing/selftests/net/mptcp/mptcp_join.sh
tools/testing/selftests/net/mptcp/mptcp_sockopt.sh
tools/testing/selftests/net/mptcp/simult_flows.sh
tools/testing/selftests/net/mptcp/userspace_pm.sh

index 515859a..24bcd7b 100755 (executable)
@@ -1,6 +1,7 @@
 #!/bin/bash
 # SPDX-License-Identifier: GPL-2.0
 
+sec=$(date +%s)
 rndh=$(printf %x $sec)-$(mktemp -u XXXXXX)
 ns="ns1-$rndh"
 ksft_skip=4
index 63b722b..a43d3e2 100755 (executable)
@@ -274,8 +274,7 @@ check_transfer()
 
 check_mptcp_disabled()
 {
-       local disabled_ns
-       disabled_ns="ns_disabled-$sech-$(mktemp -u XXXXXX)"
+       local disabled_ns="ns_disabled-$rndh"
        ip netns add ${disabled_ns} || exit $ksft_skip
 
        # net.mptcp.enabled should be enabled by default
index 2eeaf4a..2a402b3 100755 (executable)
@@ -59,8 +59,9 @@ init_partial()
 {
        capout=$(mktemp)
 
-       local rndh
-       rndh=$(mktemp -u XXXXXX)
+       local sec rndh
+       sec=$(date +%s)
+       rndh=$(printf %x $sec)-$(mktemp -u XXXXXX)
 
        ns1="ns1-$rndh"
        ns2="ns2-$rndh"
index 6e8f459..dbee386 100755 (executable)
@@ -30,6 +30,8 @@ add_mark_rules()
 
 init()
 {
+       local sec rndh
+       sec=$(date +%s)
        rndh=$(printf %x $sec)-$(mktemp -u XXXXXX)
 
        ns1="ns1-$rndh"
index 189a664..9f22f7e 100755 (executable)
@@ -1,6 +1,7 @@
 #!/bin/bash
 # SPDX-License-Identifier: GPL-2.0
 
+sec=$(date +%s)
 rndh=$(printf %x $sec)-$(mktemp -u XXXXXX)
 ns1="ns1-$rndh"
 ns2="ns2-$rndh"
index 3229725..5dfc3ee 100755 (executable)
@@ -33,7 +33,7 @@ client_addr_id=${RANDOM:0:2}
 server_addr_id=${RANDOM:0:2}
 
 sec=$(date +%s)
-rndh=$(stdbuf -o0 -e0 printf %x "$sec")-$(mktemp -u XXXXXX)
+rndh=$(printf %x "$sec")-$(mktemp -u XXXXXX)
 ns1="ns1-$rndh"
 ns2="ns2-$rndh"