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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux