On Mon, Apr 04, 2022 at 01:13:48PM +0100, Jeremy Sowden wrote:
> Shifts are of integer type and in HBO.
>
> Signed-off-by: Jeremy Sowden <jeremy@xxxxxxxxxx>
> ---
> src/netlink_delinearize.c | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/src/netlink_delinearize.c b/src/netlink_delinearize.c
> index 12624db4c3a5..8b010fe4d168 100644
> --- a/src/netlink_delinearize.c
> +++ b/src/netlink_delinearize.c
> @@ -2618,8 +2618,17 @@ static void expr_postprocess(struct rule_pp_ctx *ctx, struct expr **exprp)
> }
> expr_postprocess(ctx, &expr->right);
>
> - expr_set_type(expr, expr->left->dtype,
> - expr->left->byteorder);
> + switch (expr->op) {
> + case OP_LSHIFT:
> + case OP_RSHIFT:
> + expr_set_type(expr, &integer_type,
> + BYTEORDER_HOST_ENDIAN);
> + break;
> + default:
> + expr_set_type(expr, expr->left->dtype,
> + expr->left->byteorder);
This is a fix?
If so, would it be possible to provide a standalone example that shows
what this is fixing up?
> + }
> +
> break;
> case EXPR_RELATIONAL:
> switch (expr->left->etype) {
> --
> 2.35.1
>
