]> git.ipfire.org Git - thirdparty/nftables.git/commitdiff
netlink_delinearize: fix listing of set members in host byteorder using integer_type
authorPablo Neira Ayuso <pablo@netfilter.org>
Mon, 8 Dec 2014 21:14:09 +0000 (22:14 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Tue, 9 Dec 2014 18:12:20 +0000 (19:12 +0100)
 nft list table filter
 ...
        cpu { 5033164833554432, 0, 16777216} counter packets 8 bytes 344

Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
src/netlink_delinearize.c

index c809bb6b863118804922193da80227293462cd98..e9a04dd4a907aa2ad9ef0db2265dd1750b21d93d 100644 (file)
@@ -742,6 +742,29 @@ static void payload_dependency_store(struct rule_pp_ctx *ctx,
        ctx->pdep  = stmt;
 }
 
+static void integer_type_postprocess(struct expr *expr)
+{
+       struct expr *i;
+
+       switch (expr->ops->type) {
+       case EXPR_VALUE:
+               if (expr->byteorder == BYTEORDER_HOST_ENDIAN) {
+                       uint32_t len = div_round_up(expr->len, BITS_PER_BYTE);
+
+                       mpz_switch_byteorder(expr->value, len);
+               }
+               break;
+       case EXPR_SET_REF:
+               list_for_each_entry(i, &expr->set->init->expressions, list) {
+                       expr_set_type(i, expr->dtype, expr->byteorder);
+                       integer_type_postprocess(i);
+               }
+               break;
+       default:
+               break;
+       }
+}
+
 static void payload_match_postprocess(struct rule_pp_ctx *ctx,
                                      struct stmt *stmt, struct expr *expr)
 {
@@ -805,6 +828,12 @@ static void meta_match_postprocess(struct rule_pp_ctx *ctx,
                    left->flags & EXPR_F_PROTOCOL)
                        payload_dependency_store(ctx, stmt, left->meta.base);
                break;
+       case OP_LOOKUP:
+               expr_set_type(expr->right, expr->left->dtype,
+                             expr->left->byteorder);
+               if (expr->right->dtype == &integer_type)
+                       integer_type_postprocess(expr->right);
+               break;
        default:
                break;
        }