diff options
author | Phil Sutter <phil@nwl.cc> | 2019-05-14 13:46:00 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2019-05-15 17:14:40 +0200 |
commit | 8b4bfef29dcbddcb302461a85ab7589da75e50a6 (patch) | |
tree | 159cd2a7b868710536e7440f6370fd2f99647c57 | |
parent | 8b59c16fc3d51bf9d7896d5430910e9014dfda57 (diff) |
tests: Fix ipt-restore/0004-restore-race_0 testcase
Two issues fixed:
* XTABLES_LIBDIR was set wrong (CWD is not topdir but tests/). Drop the
export altogether, the testscript does this already.
* $LINES is a variable set by bash, so initial dump sanity check failed
all the time complaining about a spurious initial dump line count. Use
$LINES1 instead.
Fixes: 4000b4cf2ea38 ("tests: add test script for race-free restore")
Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
-rwxr-xr-x | iptables/tests/shell/testcases/ipt-restore/0004-restore-race_0 | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/iptables/tests/shell/testcases/ipt-restore/0004-restore-race_0 b/iptables/tests/shell/testcases/ipt-restore/0004-restore-race_0 index 14b910eb..a92d18dc 100755 --- a/iptables/tests/shell/testcases/ipt-restore/0004-restore-race_0 +++ b/iptables/tests/shell/testcases/ipt-restore/0004-restore-race_0 @@ -1,6 +1,5 @@ #!/bin/bash -export XTABLES_LIBDIR=$(pwd)/extensions have_nft=false nft -v > /dev/null && have_nft=true @@ -77,12 +76,12 @@ dumpfile=$(mktemp) || exit 1 make_dummy_rules > $dumpfile $XT_MULTI iptables-restore -w < $dumpfile -LINES=$(wc -l < $dumpfile) +LINES1=$(wc -l < $dumpfile) $XT_MULTI iptables-save | grep -v '^#' > $dumpfile LINES2=$(wc -l < $dumpfile) -if [ $LINES -ne $LINES2 ]; then - echo "Original dump has $LINES, not $LINES2" 1>&2 +if [ $LINES1 -ne $LINES2 ]; then + echo "Original dump has $LINES1, not $LINES2" 1>&2 exit 111 fi |