Re: [PATCH net-next] bonding: deal with xfrm state in all modes and add more error-checking

From: David Miller
Date: Wed Jul 08 2020 - 18:38:06 EST


From: Jarod Wilson <jarod@xxxxxxxxxx>
Date: Wed, 8 Jul 2020 13:46:31 -0400

> It's possible that device removal happens when the bond is in non-AB mode,
> and addition happens in AB mode, so bond_ipsec_del_sa() never gets called,
> which leaves security associations in an odd state if bond_ipsec_add_sa()
> then gets called after switching the bond into AB. Just call add and
> delete universally for all modes to keep things consistent.
>
> However, it's also possible that this code gets called when the system is
> shutting down, and the xfrm subsystem has already been disconnected from
> the bond device, so we need to do some error-checking and bail, lest we
> hit a null ptr deref.
>
> Fixes: a3b658cfb664 ("bonding: allow xfrm offload setup post-module-load")
> Signed-off-by: Jarod Wilson <jarod@xxxxxxxxxx>

Applied, thanks Jarod.