diff options
| author | Pablo Neira Ayuso <pablo@netfilter.org> | 2016-06-15 13:41:06 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2016-06-15 13:46:06 +0200 |
| commit | 57468cfa7916aa6e21c977d1ddb6d0a0ad27edf7 (patch) | |
| tree | d1cac680f229f086c2eb7dcbb569c81534ff8573 /tests/nft-expr_lookup-test.c | |
| parent | 23c2ef2f9812a04c3bd8248de70cad37a176550a (diff) | |
tests: stricter string attribute validation
In nft-expr_lookup-test.c, check for the strings instead of size.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'tests/nft-expr_lookup-test.c')
| -rw-r--r-- | tests/nft-expr_lookup-test.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/tests/nft-expr_lookup-test.c b/tests/nft-expr_lookup-test.c index 83fa829..d3b45df 100644 --- a/tests/nft-expr_lookup-test.c +++ b/tests/nft-expr_lookup-test.c @@ -30,18 +30,15 @@ static void print_err(const char *msg) static void cmp_nftnl_expr(struct nftnl_expr *rule_a, struct nftnl_expr *rule_b) { - uint32_t data_lena, data_lenb; - if (nftnl_expr_get_u32(rule_a, NFTNL_EXPR_LOOKUP_SREG) != nftnl_expr_get_u32(rule_b, NFTNL_EXPR_LOOPUP_SREG)) print_err("Expr NFTNL_EXPR_LOOkUP_SREG mismatches"); if (nftnl_expr_get_u32(rule_a, NFTNL_EXPR_LOOKUP_DREG) != nftnl_expr_get_u32(rule_b, NFTNL_EXPR_LOOPUP_DREG)) print_err("Expr NFTNL_EXPR_LOOkUP_DREG mismatches"); - nftnl_expr_get(rule_a, NFTNL_EXPR_LOOKUP_SET, &data_lena); - nftnl_expr_get(rule_b, NFTNL_EXPR_LOOKUP_SET, &data_lenb); - if (data_lena != data_lenb) - print_err("Expr NFTNL_EXPR_LOOKUP_SET size mismatches"); + if (strcmp(nftnl_expr_get_str(rule_a, NFTNL_EXPR_LOOKUP_SET), + nftnl_expr_get_str(rule_b, NFTNL_EXPR_LOOKUP_SET))) + print_err("Expr NFTNL_EXPR_LOOKUP_SET mismatches"); } int main(int argc, char *argv[]) |