Skip to content

Commit

Permalink
pfsync tests: check for the correct IP address
Browse files Browse the repository at this point in the history
When checking if the state synced over we should look for
198.51.100.254, not 198.51.100.2. The test worked because the incorrect
address is a substring of the correct one, but we should fix it anyway.

Reported by:	Naman Sood <naman@freebsdfoundation.org>
MFC after:	1 week
  • Loading branch information
kprovost committed May 18, 2023
1 parent 9a19595 commit 0d574d8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/sys/netpfil/pf/pfsync.sh
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ common_body()
sleep 2

if ! jexec two pfctl -s states | grep icmp | grep 198.51.100.1 | \
grep 198.51.100.2 ; then
grep 198.51.100.254 ; then
atf_fail "state not found on synced host"
fi
}
Expand Down

0 comments on commit 0d574d8

Please sign in to comment.