summaryrefslogtreecommitdiff
path: root/tools/testing/selftests/net/veth.sh
diff options
context:
space:
mode:
authorAndrea Righi <andrea.righi@canonical.com>2021-06-16 17:03:21 +0300
committerDavid S. Miller <davem@davemloft.net>2021-06-16 22:50:24 +0300
commit0fd158b89b50b3a31c97a639ff496e1c59686e97 (patch)
tree5b4e98dda4d59fbf1afb36d8f48928d6b6c2d759 /tools/testing/selftests/net/veth.sh
parent1d2ac2033d790f5deaf3d6edfff6a4d901949de2 (diff)
downloadlinux-0fd158b89b50b3a31c97a639ff496e1c59686e97.tar.xz
selftests: net: veth: make test compatible with dash
veth.sh is a shell script that uses /bin/sh; some distro (Ubuntu for example) use dash as /bin/sh and in this case the test reports the following error: # ./veth.sh: 21: local: -r: bad variable name # ./veth.sh: 21: local: -r: bad variable name This happens because dash doesn't support the option "-r" with local. Moreover, in case of missing bpf object, the script is exiting -1, that is an illegal number for dash: exit: Illegal number: -1 Change the script to be compatible both with bash and dash and prevent the errors above. Signed-off-by: Andrea Righi <andrea.righi@canonical.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools/testing/selftests/net/veth.sh')
-rwxr-xr-xtools/testing/selftests/net/veth.sh5
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/testing/selftests/net/veth.sh b/tools/testing/selftests/net/veth.sh
index 2fedc0781ce8..11d7cdb898c0 100755
--- a/tools/testing/selftests/net/veth.sh
+++ b/tools/testing/selftests/net/veth.sh
@@ -18,7 +18,8 @@ ret=0
cleanup() {
local ns
- local -r jobs="$(jobs -p)"
+ local jobs
+ readonly jobs="$(jobs -p)"
[ -n "${jobs}" ] && kill -1 ${jobs} 2>/dev/null
rm -f $STATS
@@ -108,7 +109,7 @@ chk_gro() {
if [ ! -f ../bpf/xdp_dummy.o ]; then
echo "Missing xdp_dummy helper. Build bpf selftest first"
- exit -1
+ exit 1
fi
create_ns