Thread (3 messages) 3 messages, 3 authors, 2021-08-12

Re: [PATCH net-next] net: dsa: mt7530: fix VLAN traffic leaks again

flat view

From: Vladimir Oltean <olteanv@gmail.com>
Date: 2021-08-11 10:01:32
Also in: linux-arm-kernel, linux-mediatek, lkml

On Wed, Aug 11, 2021 at 05:50:43PM +0800, DENG Qingfang wrote:
When a port leaves a VLAN-aware bridge, the current code does not clear
other ports' matrix field bit. If the bridge is later set to VLAN-unaware
mode, traffic in the bridge may leak to that port.

Remove the VLAN filtering check in mt7530_port_bridge_leave.

Fixes: 474a2ddaa192 ("net: dsa: mt7530: fix VLAN traffic leaks")
Fixes: 83163f7dca56 ("net: dsa: mediatek: add VLAN support for MT7530")
Signed-off-by: DENG Qingfang <dqfext@gmail.com>
---
That hunk looked indeed very strange when I went over it with commit
'net: dsa: remove the "dsa_to_port in a loop" antipattern from drivers',
so I'm happy to see it go away.

Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help