]> git.baikalelectronics.ru Git - kernel.git/commit
net: dsa: fix NULL pointer dereference in dsa_port_reset_vlan_filtering
authorVladimir Oltean <vladimir.oltean@nxp.com>
Fri, 15 Jul 2022 15:16:59 +0000 (18:16 +0300)
committerJakub Kicinski <kuba@kernel.org>
Tue, 19 Jul 2022 03:14:23 +0000 (20:14 -0700)
commitd1592ec3d5f2a79fc370e9d3c9bc6b47da4cac6a
treed76489dac824b6415b530d268b3cf6891ea58edc
parent2d6fbb4d50bbe0bd2dd543f70ac2d1d05d07e1e7
net: dsa: fix NULL pointer dereference in dsa_port_reset_vlan_filtering

The "ds" iterator variable used in dsa_port_reset_vlan_filtering() ->
dsa_switch_for_each_port() overwrites the "dp" received as argument,
which is later used to call dsa_port_vlan_filtering() proper.

As a result, switches which do enter that code path (the ones with
vlan_filtering_is_global=true) will dereference an invalid dp in
dsa_port_reset_vlan_filtering() after leaving a VLAN-aware bridge.

Use a dedicated "other_dp" iterator variable to avoid this from
happening.

Fixes: 167aaa473955 ("net: dsa: remove the "dsa_to_port in a loop" antipattern from the core")
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/dsa/port.c