On Sat, Oct 29, 2016 at 09:49:09AM +0000, Anders K. Pedersen | Cohaesio wrote: > From: Anders K. Pedersen <akp@xxxxxxxxxxxx> > > 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. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html