]> git.baikalelectronics.ru Git - kernel.git/commit
bonding: Add struct bond_ipesc to manage SA
authorTaehee Yoo <ap420073@gmail.com>
Mon, 5 Jul 2021 15:38:12 +0000 (15:38 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 6 Jul 2021 17:36:59 +0000 (10:36 -0700)
commita8d51177456512a274be05c28749cfba950b175a
tree08f03bd63df3506bee57bde496cafb3104078270
parent912f0091640c892a29ab192078c1a0e59f5d30ae
bonding: Add struct bond_ipesc to manage SA

bonding has been supporting ipsec offload.
When SA is added, bonding just passes SA to its own active real interface.
But it doesn't manage SA.
So, when events(add/del real interface, active real interface change, etc)
occur, bonding can't handle that well because It doesn't manage SA.
So some problems(panic, UAF, refcnt leak)occur.

In order to make it stable, it should manage SA.
That's the reason why struct bond_ipsec is added.
When a new SA is added to bonding interface, it is stored in the
bond_ipsec list. And the SA is passed to a current active real interface.
If events occur, it uses bond_ipsec data to handle these events.
bond->ipsec_list is protected by bond->ipsec_lock.

If a current active real interface is changed, the following logic works.
1. delete all SAs from old active real interface
2. Add all SAs to the new active real interface.
3. If a new active real interface doesn't support ipsec offload or SA's
option, it sets real_dev to NULL.

Fixes: 6424b693bac6 ("bonding: support hardware encryption offload to slaves")
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c
include/net/bonding.h