[PATCH 0/2] bonding: 3ad: fix 2 RCU issues in ad_mux_machine()

Hangbin Liu posted 2 patches 1 month, 1 week ago
drivers/net/bonding/bond_3ad.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
[PATCH 0/2] bonding: 3ad: fix 2 RCU issues in ad_mux_machine()
Posted by Hangbin Liu 1 month, 1 week ago
This series addresses a NULL pointer dereference in ad_mux_machine()
when ad_port_selection_logic() detaches a port from its aggregator
but does not assign a replacement.

Patch 1 adds an early null aggregator check in ad_mux_machine() to
prevent the oops. Patch 2 is a minor RCU fix that uses RCU_INIT_POINTER()
when initializing port->aggregator to NULL.

Signed-off-by: Hangbin Liu <liuhangbin@kylinos.cn>
---
Hangbin Liu (2):
      bonding: 3ad: fix NULL pointer dereference in ad_mux_machine()
      bonding: 3ad: use RCU_INIT_POINTER for rcu pointer initialization

 drivers/net/bonding/bond_3ad.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
---
base-commit: 44871eadd07a7f004aa00cb87399461eea08c630
change-id: 20260806-bond_rcu-5d99f64547b7

Best regards,
-- 
Hangbin Liu <liuhangbin@kylinos.cn>
Re: [PATCH 0/2] bonding: 3ad: fix 2 RCU issues in ad_mux_machine()
Posted by Hangbin Liu 1 month, 1 week ago
Hi,

On Mon, Aug 17, 2026 at 04:32:35PM +0800, Hangbin Liu wrote:
> This series addresses a NULL pointer dereference in ad_mux_machine()
> when ad_port_selection_logic() detaches a port from its aggregator
> but does not assign a replacement.
> 
> Patch 1 adds an early null aggregator check in ad_mux_machine() to
> prevent the oops. Patch 2 is a minor RCU fix that uses RCU_INIT_POINTER()
> when initializing port->aggregator to NULL.

Please ignore this patch set. I just got back to upstream after a few months'
leave and didn't catch up with the new policy before sending this series.

The first patch is only theoretical and has no reproducer. The second patch
is a NOP, as Eric said. So let's drop this patch set.

Thanks
Hangbin