wifi: brcmfmac: avoid invalid list operation when vendor attach fails

[ Upstream commit b822015a1f ]

When the brcmf_fwvid_attach() fails the driver instance is not added
to the vendor list. Hence we should not try to delete it from that
list when the brcmf_fwvid_detach() function is called in cleanup path.

Cc: stable@vger.kernel.org # 6.2.x
Fixes: d6a5c56221 ("wifi: brcmfmac: add support for vendor-specific firmware api")
Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://msgid.link/20240106103835.269149-3-arend.vanspriel@broadcom.com
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Arend van Spriel 2024-01-06 11:38:34 +01:00 committed by Greg Kroah-Hartman
parent 0a7591e14a
commit 6ab797497f
1 changed files with 4 additions and 3 deletions

View File

@ -187,9 +187,10 @@ void brcmf_fwvid_detach_ops(struct brcmf_pub *drvr)
mutex_lock(&fwvid_list_lock);
drvr->vops = NULL;
list_del(&drvr->bus_if->list);
if (drvr->vops) {
drvr->vops = NULL;
list_del(&drvr->bus_if->list);
}
mutex_unlock(&fwvid_list_lock);
}