netfilter: bitwise: add helper for dumping boolean operations.
authorJeremy Sowden <jeremy@azazel.net>
Wed, 15 Jan 2020 20:05:54 +0000 (20:05 +0000)
committerPablo Neira Ayuso <pablo@netfilter.org>
Thu, 16 Jan 2020 14:52:00 +0000 (15:52 +0100)
Split the code specific to dumping bitwise boolean operations out into a
separate function.  A similar function will be added later for shift
operations.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nft_bitwise.c

index 88a91c63f213e94783894fea9508125d9c522716..41265134cf0b3f1e538a7fa35a19152f58425829 100644 (file)
@@ -143,9 +143,24 @@ static int nft_bitwise_init(const struct nft_ctx *ctx,
        return err;
 }
 
+static int nft_bitwise_dump_bool(struct sk_buff *skb,
+                                const struct nft_bitwise *priv)
+{
+       if (nft_data_dump(skb, NFTA_BITWISE_MASK, &priv->mask,
+                         NFT_DATA_VALUE, priv->len) < 0)
+               return -1;
+
+       if (nft_data_dump(skb, NFTA_BITWISE_XOR, &priv->xor,
+                         NFT_DATA_VALUE, priv->len) < 0)
+               return -1;
+
+       return 0;
+}
+
 static int nft_bitwise_dump(struct sk_buff *skb, const struct nft_expr *expr)
 {
        const struct nft_bitwise *priv = nft_expr_priv(expr);
+       int err = 0;
 
        if (nft_dump_register(skb, NFTA_BITWISE_SREG, priv->sreg))
                return -1;
@@ -156,15 +171,13 @@ static int nft_bitwise_dump(struct sk_buff *skb, const struct nft_expr *expr)
        if (nla_put_be32(skb, NFTA_BITWISE_OP, htonl(priv->op)))
                return -1;
 
-       if (nft_data_dump(skb, NFTA_BITWISE_MASK, &priv->mask,
-                         NFT_DATA_VALUE, priv->len) < 0)
-               return -1;
-
-       if (nft_data_dump(skb, NFTA_BITWISE_XOR, &priv->xor,
-                         NFT_DATA_VALUE, priv->len) < 0)
-               return -1;
+       switch (priv->op) {
+       case NFT_BITWISE_BOOL:
+               err = nft_bitwise_dump_bool(skb, priv);
+               break;
+       }
 
-       return 0;
+       return err;
 }
 
 static struct nft_data zero;