| From 7ff106b0c11d0a819199e34289f700efbddf8455 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Mon, 5 Jul 2021 15:38:14 +0000 |
| Subject: bonding: fix incorrect return value of bond_ipsec_offload_ok() |
| |
| From: Taehee Yoo <ap420073@gmail.com> |
| |
| [ Upstream commit 168e696a36792a4a3b2525a06249e7472ef90186 ] |
| |
| 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> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| drivers/net/bonding/bond_main.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c |
| index d267791a06c0..bf8ade982940 100644 |
| --- a/drivers/net/bonding/bond_main.c |
| +++ b/drivers/net/bonding/bond_main.c |
| @@ -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; |
| } |
| |
| -- |
| 2.30.2 |
| |