diff options
author | Arturo Borrero Gonzalez <arturo.borrero.glez@gmail.com> | 2013-06-26 13:37:02 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2013-06-27 19:11:11 +0200 |
commit | 783a972f6acd680bed92fac36c5388d15620e754 (patch) | |
tree | 1a95b955f5f83f84711c9dec39cee2ac6d251308 | |
parent | 34caf0add23aab075c2bbe1a99927c081014cadd (diff) |
expr: xml: registers must be <= NFT_REG_MAX
With this patch, all expressions validate that registers are <= NFT_REG_MAX.
Signed-off-by: Arturo Borrero Gonzalez <arturo.borrero.glez@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
-rw-r--r-- | src/expr/bitwise.c | 10 | ||||
-rw-r--r-- | src/expr/byteorder.c | 6 | ||||
-rw-r--r-- | src/expr/cmp.c | 5 | ||||
-rw-r--r-- | src/expr/ct.c | 3 | ||||
-rw-r--r-- | src/expr/exthdr.c | 5 | ||||
-rw-r--r-- | src/expr/immediate.c | 5 | ||||
-rw-r--r-- | src/expr/lookup.c | 10 | ||||
-rw-r--r-- | src/expr/meta.c | 5 | ||||
-rw-r--r-- | src/expr/payload.c | 5 |
9 files changed, 54 insertions, 0 deletions
diff --git a/src/expr/bitwise.c b/src/expr/bitwise.c index c8c4f40..80c4f20 100644 --- a/src/expr/bitwise.c +++ b/src/expr/bitwise.c @@ -237,6 +237,11 @@ nft_rule_expr_bitwise_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + bitwise->sreg = (uint32_t)tmp; e->flags |= (1 << NFT_EXPR_BITWISE_SREG); @@ -252,6 +257,11 @@ nft_rule_expr_bitwise_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + bitwise->dreg = (uint32_t)tmp; e->flags |= (1 << NFT_EXPR_BITWISE_DREG); diff --git a/src/expr/byteorder.c b/src/expr/byteorder.c index 201a943..c2f38a8 100644 --- a/src/expr/byteorder.c +++ b/src/expr/byteorder.c @@ -225,6 +225,9 @@ nft_rule_expr_byteorder_xml_parse(struct nft_rule_expr *e, char *xml) if (tmp > UINT32_MAX || tmp < 0 || *endptr) goto err; + if (tmp > NFT_REG_MAX) + goto err; + byteorder->sreg = tmp; e->flags |= (1 << NFT_EXPR_BYTEORDER_SREG); @@ -236,6 +239,9 @@ nft_rule_expr_byteorder_xml_parse(struct nft_rule_expr *e, char *xml) if (tmp > UINT32_MAX || tmp < 0 || *endptr) goto err; + if (tmp > NFT_REG_MAX) + goto err; + byteorder->dreg = tmp; e->flags |= (1 << NFT_EXPR_BYTEORDER_DREG); diff --git a/src/expr/cmp.c b/src/expr/cmp.c index dac1f54..9507a0e 100644 --- a/src/expr/cmp.c +++ b/src/expr/cmp.c @@ -203,6 +203,11 @@ static int nft_rule_expr_cmp_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + cmp->sreg = (uint8_t)tmp; e->flags |= (1 << NFT_EXPR_CMP_SREG); } diff --git a/src/expr/ct.c b/src/expr/ct.c index 7a239fa..61a8fef 100644 --- a/src/expr/ct.c +++ b/src/expr/ct.c @@ -177,6 +177,9 @@ static int nft_rule_expr_ct_xml_parse(struct nft_rule_expr *e, char *xml) if (tmp > UINT8_MAX || tmp < 0 || *endptr) goto err; + if (tmp > NFT_REG_MAX) + goto err; + ct->dreg = tmp; e->flags |= (1 << NFT_EXPR_CT_DREG); diff --git a/src/expr/exthdr.c b/src/expr/exthdr.c index 8af6a63..7e16878 100644 --- a/src/expr/exthdr.c +++ b/src/expr/exthdr.c @@ -205,6 +205,11 @@ nft_rule_expr_exthdr_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + exthdr->dreg = tmp; e->flags |= (1 << NFT_EXPR_EXTHDR_DREG); } diff --git a/src/expr/immediate.c b/src/expr/immediate.c index b5a6a41..8bc810c 100644 --- a/src/expr/immediate.c +++ b/src/expr/immediate.c @@ -236,6 +236,11 @@ nft_rule_expr_immediate_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + imm->dreg = (uint32_t)tmp; e->flags |= (1 << NFT_EXPR_IMM_DREG); diff --git a/src/expr/lookup.c b/src/expr/lookup.c index 0ae93ce..ecc07cb 100644 --- a/src/expr/lookup.c +++ b/src/expr/lookup.c @@ -204,6 +204,11 @@ nft_rule_expr_lookup_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + lookup->sreg = (uint32_t)tmp; e->flags |= (1 << NFT_EXPR_LOOKUP_SREG); @@ -217,6 +222,11 @@ nft_rule_expr_lookup_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + lookup->dreg = (uint32_t)tmp; e->flags |= (1 << NFT_EXPR_LOOKUP_DREG); } diff --git a/src/expr/meta.c b/src/expr/meta.c index 535b456..41fcff1 100644 --- a/src/expr/meta.c +++ b/src/expr/meta.c @@ -163,6 +163,11 @@ static int nft_rule_expr_meta_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + meta->dreg = (uint8_t)tmp; e->flags |= (1 << NFT_EXPR_META_DREG); diff --git a/src/expr/payload.c b/src/expr/payload.c index 28c52ca..dc42918 100644 --- a/src/expr/payload.c +++ b/src/expr/payload.c @@ -200,6 +200,11 @@ nft_rule_expr_payload_xml_parse(struct nft_rule_expr *e, char *xml) return -1; } + if (tmp > NFT_REG_MAX) { + mxmlDelete(tree); + return -1; + } + payload->dreg = (uint32_t)tmp; e->flags |= (1 << NFT_EXPR_PAYLOAD_DREG); } |