On Mon, Jun 12, 2017 at 2:52 PM, Pablo Neira Ayuso <[email protected]> wrote:
> On Fri, Jun 09, 2017 at 09:31:00PM +0530, Shyam Saini wrote:
>> diff --git a/tests/shell/testcases/sets/0023unknown_value_to_use_0
>> b/tests/shell/testcases/sets/0023unknown_value_to_use_0
>> new file mode 100755
>> index 0000000..ee9be68
>> --- /dev/null
>> +++ b/tests/shell/testcases/sets/0023unknown_value_to_use_0
>> @@ -0,0 +1,33 @@
>> +#!/bin/bash
>> +
>> + # This test checks bug identified and fixed in the commit Id "986dea8".
>> + # i.e, If in a statement there are multiple src data then it would be
>> totally ambiguous to decide which value to set.
>> +
>> + # Before this commit 986dea8, nft returns 134 which indicates the bug
>> but after this commit it returns 1.
>> + # We don't add this test in python testsuite, because there we can't
>> detect 134 != 1 (returns code stating failure)
>> +
>> +declare -a rules=(
>> +"tcp dport set {1, 2, 3}" "udp dport set {1, 2, 3}"
>> +"meta pkttype set {unicast, multicast, broadcast}"
>> +"meta mark set {0xffff, 0xcc}"
>> +"ct mark set {0x11333, 0x11}" "ct zone set {123, 127}"
>> +"ct label set {123, 127}"
>> +"ct event set {new, related, destroy, label}"
>> +"ether daddr set {01:00:5e:00:01:01, 01:00:5e:00:02:02}"
>> +"ip saddr set {192.19.1.2, 191.1.22.1}"
>> +)
>> +
>> +$NFT add table t
>> +$NFT add chain t c
>> +
>> +for (( i = 0 ; i < ${#rules[@]} ; i++ ))
>> +do
>> + $NFT add rule t c ${rules[$i]}
>> + ret=$?
>> + if [ $ret -ne 1 ];
>> + then
>> + echo "E: failed test: returned $ret instead of 1" >&2
>> + exit 1
>> + fi
>> +done
>
> Better use tests/py/ for this?
We only have "ok" and "fail" as return codes in python test suites and
we can't can't detect 134 != 1 there.
So it was added in the tests/shell/.
further we would need "*.payload" file for that respective test.
$ nft --debug=netlink add rule t c udp dport set {1, 3}
But in our case it is not generated at all.
Is it right ?
Please correct me
Thanks,
shyam
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
