mwifiex: Properly initialize private structure on interface type changes
authorJonas Dreßler <verdre@v0yd.nl>
Tue, 14 Sep 2021 19:59:08 +0000 (21:59 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 21 Sep 2021 15:02:18 +0000 (18:02 +0300)
commitc606008b70627a2fc485732a53cc22f0f66d0981
tree4af7a236026df7d133c7f7c2fa676e80befa2b1b
parent5e2e1a4bf4a162d9369a35e62fbb8c7977b54a12
mwifiex: Properly initialize private structure on interface type changes

When creating a new virtual interface in mwifiex_add_virtual_intf(), we
update our internal driver states like bss_type, bss_priority, bss_role
and bss_mode to reflect the mode the firmware will be set to.

When switching virtual interface mode using
mwifiex_init_new_priv_params() though, we currently only update bss_mode
and bss_role. In order for the interface mode switch to actually work,
we also need to update bss_type to its proper value, so do that.

This fixes a crash of the firmware (because the driver tries to execute
commands that are invalid in AP mode) when switching from station mode
to AP mode.

Signed-off-by: Jonas Dreßler <verdre@v0yd.nl>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20210914195909.36035-9-verdre@v0yd.nl
drivers/net/wireless/marvell/mwifiex/cfg80211.c