Re: [PATCH nftables] Allow concatenation of rt nexthop etc.

[Pablo Neira Ayuso <pablo@netfilter.org>]

On Sat, Oct 29, 2016 at 09:49:09AM +0000, Anders K. Pedersen | Cohaesio wrote:
> From: Anders K. Pedersen <akp@cohaesio.com>
> 
> Concatenations of rt nexthop or ct {orignal | reply} {saddr | daddr} fail
> due to
> 
>  # nft add rule ip filter postrouting flow table acct \{ ip saddr . rt nexthop counter \}
>  <cmdline>:1:61-70: Error: can not use variable sized data types (invalid) in concat expressions
>  add rule ip filter postrouting flow table acct { ip saddr . rt nexthop counter }
>                                                   ~~~~~~~~~~~^^^^^^^^^^
> 
> Fix this by reordering the check for variable size data types in
> expr_evaluate_concat() to happen after expr_evaluate() has been called (via
> list_member_evaluate()) for the sub expression. This allows
> expr_evaluate_[cr]t() to call [cr]t_expr_update_type() and set the data type
> before the check.

Applied, thanks Anders.

P.S: I can see git am sometimes complain about trailing CRLF, also
patchwork seems to skip some of your patches for some reason, which
makes it easier for us to lose stuff from you. I guess it's probably
MUA related problem, it would be great if you find the root cause.