Merge pull request iovisor#4162 from chantra/fix_test_wrapper_escaping

[test] Fix Wrapper script to properly handle arguments
e-dant · Aug 12, 2022 · f9aa1e7 · f9aa1e7
2 parents 3130f0f + 770a590
commit f9aa1e7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/tests/wrapper.sh.in b/tests/wrapper.sh.in
@@ -32,11 +32,11 @@ function ns_run() {
  sudo ip netns exec $ns ethtool -K eth0 tx off
  sudo ip addr add dev $ns.out 172.16.1.1/24
  sudo ip link set $ns.out up
- sudo bash -c "PYTHONPATH=$PYTHONPATH PYTHON_TEST_LOGFILE=$PYTHON_TEST_LOGFILE LD_LIBRARY_PATH=$LD_LIBRARY_PATH ip netns exec $ns $cmd \"$@\""
+ sudo --preserve-env=PYTHONPATH,PYTHON_TEST_LOGFILE,LD_LIBRARY_PATH ip netns exec $ns $cmd "$@"
  return $?
 }
 function sudo_run() {
- sudo bash -c "PYTHONPATH=$PYTHONPATH PYTHON_TEST_LOGFILE=$PYTHON_TEST_LOGFILE LD_LIBRARY_PATH=$LD_LIBRARY_PATH $cmd \"$@\""
+ sudo --preserve-env=PYTHONPATH,PYTHON_TEST_LOGFILE,LD_LIBRARY_PATH $cmd "$@"
  return $?
 }
 function simple_run() {