From 6ad1dab62ca3efe38709efad27ca1cd776e4ca6f Mon Sep 17 00:00:00 2001 From: Pablo Neira Ayuso Date: Sat, 24 Aug 2013 18:27:58 +0200 Subject: src: json: s/nft_jansson_value_parse_reg/nft_jansson_parse_reg/g This patch is a cleanup to save LOC. I have also changed it to use < 0 to check for error conditions. Signed-off-by: Pablo Neira Ayuso --- src/expr/bitwise.c | 4 ++-- src/expr/byteorder.c | 6 ++---- src/expr/ct.c | 2 +- src/expr/exthdr.c | 3 +-- src/expr/immediate.c | 2 +- src/expr/lookup.c | 4 ++-- src/expr/meta.c | 2 +- src/expr/nat.c | 16 ++++++++-------- src/expr/payload.c | 2 +- 9 files changed, 19 insertions(+), 22 deletions(-) (limited to 'src/expr') diff --git a/src/expr/bitwise.c b/src/expr/bitwise.c index dc424b5..11738a7 100644 --- a/src/expr/bitwise.c +++ b/src/expr/bitwise.c @@ -187,12 +187,12 @@ nft_rule_expr_bitwise_json_parse(struct nft_rule_expr *e, json_t *root) struct nft_expr_bitwise *bitwise = nft_expr_data(e); uint32_t reg, len; - if (nft_jansson_value_parse_reg(root, "sreg", NFT_TYPE_U32, ®) == -1) + if (nft_jansson_parse_reg(root, "sreg", NFT_TYPE_U32, ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_BITWISE_SREG, reg); - if (nft_jansson_value_parse_reg(root, "dreg", NFT_TYPE_U32, ®) == -1) + if (nft_jansson_parse_reg(root, "dreg", NFT_TYPE_U32, ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_BITWISE_DREG, reg); diff --git a/src/expr/byteorder.c b/src/expr/byteorder.c index 9fea753..0277812 100644 --- a/src/expr/byteorder.c +++ b/src/expr/byteorder.c @@ -201,14 +201,12 @@ nft_rule_expr_byteorder_json_parse(struct nft_rule_expr *e, json_t *root) uint32_t uval32; int ntoh; - if (nft_jansson_value_parse_reg(root, "sreg", NFT_TYPE_U32, - &uval32) != 0) + if (nft_jansson_parse_reg(root, "sreg", NFT_TYPE_U32, &uval32) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_SREG, uval32); - if (nft_jansson_value_parse_reg(root, "dreg", NFT_TYPE_U32, - &uval32) != 0) + if (nft_jansson_parse_reg(root, "dreg", NFT_TYPE_U32, &uval32) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_DREG, uval32); diff --git a/src/expr/ct.c b/src/expr/ct.c index 78575f2..4655a96 100644 --- a/src/expr/ct.c +++ b/src/expr/ct.c @@ -186,7 +186,7 @@ static int nft_rule_expr_ct_json_parse(struct nft_rule_expr *e, json_t *root) uint8_t dir; int key; - if (nft_jansson_value_parse_reg(root, "dreg", NFT_TYPE_U32, ®) != 0) + if (nft_jansson_parse_reg(root, "dreg", NFT_TYPE_U32, ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_CT_DREG, reg); diff --git a/src/expr/exthdr.c b/src/expr/exthdr.c index 2c71462..af9d6bf 100644 --- a/src/expr/exthdr.c +++ b/src/expr/exthdr.c @@ -200,8 +200,7 @@ nft_rule_expr_exthdr_json_parse(struct nft_rule_expr *e, json_t *root) uint32_t uval32; int type; - if (nft_jansson_value_parse_reg(root, "dreg", NFT_TYPE_U32, - &uval32) != 0) + if (nft_jansson_parse_reg(root, "dreg", NFT_TYPE_U32, &uval32) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_EXTHDR_DREG, uval32); diff --git a/src/expr/immediate.c b/src/expr/immediate.c index d548b75..0556999 100644 --- a/src/expr/immediate.c +++ b/src/expr/immediate.c @@ -185,7 +185,7 @@ nft_rule_expr_immediate_json_parse(struct nft_rule_expr *e, json_t *root) int datareg_type; uint32_t reg; - if (nft_jansson_value_parse_reg(root, "dreg", NFT_TYPE_U32, ®) != 0) + if (nft_jansson_parse_reg(root, "dreg", NFT_TYPE_U32, ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_IMM_DREG, reg); diff --git a/src/expr/lookup.c b/src/expr/lookup.c index cbc4d42..56ab253 100644 --- a/src/expr/lookup.c +++ b/src/expr/lookup.c @@ -155,12 +155,12 @@ nft_rule_expr_lookup_json_parse(struct nft_rule_expr *e, json_t *root) nft_rule_expr_set_str(e, NFT_EXPR_LOOKUP_SET, set_name); - if (nft_jansson_value_parse_reg(root, "sreg", NFT_TYPE_U32, ®) != 0) + if (nft_jansson_parse_reg(root, "sreg", NFT_TYPE_U32, ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_LOOKUP_SREG, reg); - if (nft_jansson_value_parse_reg(root, "dreg", NFT_TYPE_U32, ®) != 0) + if (nft_jansson_parse_reg(root, "dreg", NFT_TYPE_U32, ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_LOOKUP_DREG, reg); diff --git a/src/expr/meta.c b/src/expr/meta.c index bbe5496..d914569 100644 --- a/src/expr/meta.c +++ b/src/expr/meta.c @@ -167,7 +167,7 @@ static int nft_rule_expr_meta_json_parse(struct nft_rule_expr *e, json_t *root) uint32_t reg; int key; - if (nft_jansson_value_parse_reg(root, "dreg", NFT_TYPE_U32, ®) != 0) + if (nft_jansson_parse_reg(root, "dreg", NFT_TYPE_U32, ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_META_DREG, reg); diff --git a/src/expr/nat.c b/src/expr/nat.c index 6cde333..1ebecda 100644 --- a/src/expr/nat.c +++ b/src/expr/nat.c @@ -223,26 +223,26 @@ static int nft_rule_expr_nat_json_parse(struct nft_rule_expr *e, json_t *root) nft_rule_expr_set_u32(e, NFT_EXPR_NAT_FAMILY, val32); - if (nft_jansson_value_parse_reg(root, "sreg_addr_min", NFT_TYPE_U32, - ®) != 0) + if (nft_jansson_parse_reg(root, "sreg_addr_min", NFT_TYPE_U32, + ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_NAT_REG_ADDR_MIN, reg); - if (nft_jansson_value_parse_reg(root, "sreg_addr_max", NFT_TYPE_U32, - ®) != 0) + if (nft_jansson_parse_reg(root, "sreg_addr_max", NFT_TYPE_U32, + ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_NAT_REG_ADDR_MAX, reg); - if (nft_jansson_value_parse_reg(root, "sreg_proto_min", NFT_TYPE_U32, - ®) != 0) + if (nft_jansson_parse_reg(root, "sreg_proto_min", NFT_TYPE_U32, + ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_NAT_REG_PROTO_MIN, reg); - if (nft_jansson_value_parse_reg(root, "sreg_proto_max", NFT_TYPE_U32, - ®) != 0) + if (nft_jansson_parse_reg(root, "sreg_proto_max", NFT_TYPE_U32, + ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_NAT_REG_PROTO_MAX, reg); diff --git a/src/expr/payload.c b/src/expr/payload.c index 2345649..38362c4 100644 --- a/src/expr/payload.c +++ b/src/expr/payload.c @@ -201,7 +201,7 @@ nft_rule_expr_payload_json_parse(struct nft_rule_expr *e, json_t *root) uint32_t reg, uval32; int base; - if (nft_jansson_value_parse_reg(root, "dreg", NFT_TYPE_U32, ®) != 0) + if (nft_jansson_parse_reg(root, "dreg", NFT_TYPE_U32, ®) < 0) return -1; nft_rule_expr_set_u32(e, NFT_EXPR_PAYLOAD_DREG, reg); -- cgit v1.2.3