bonding: fix incorrect return value of bond_ipsec_offload_ok()
authorTaehee Yoo <ap420073@gmail.com>
Mon, 5 Jul 2021 15:38:14 +0000 (15:38 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 6 Jul 2021 17:36:59 +0000 (10:36 -0700)
bond_ipsec_offload_ok() is called to check whether the interface supports
ipsec offload or not.
bonding interface support ipsec offload only in active-backup mode.
So, if a bond interface is not in active-backup mode, it should return
false but it returns true.

Fixes: a3b658cfb664 ("bonding: allow xfrm offload setup post-module-load")
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c

index 780f878..d22d783 100644 (file)
@@ -581,7 +581,7 @@ static bool bond_ipsec_offload_ok(struct sk_buff *skb, struct xfrm_state *xs)
        real_dev = curr_active->dev;
 
        if (BOND_MODE(bond) != BOND_MODE_ACTIVEBACKUP) {
-               err = true;
+               err = false;
                goto out;
        }