staging: rtl8723bs: fix missing status update on sdio_alloc_irq() failure

The return value of sdio_alloc_irq() was not stored in status.
If sdio_alloc_irq() fails after rtw_drv_register_netdev() succeeds,
status remains _SUCCESS and the error path skips resource cleanup,
while rtw_drv_init() still returns success.

Store the return value of sdio_alloc_irq() in status and reuse the
existing error handling which relies on status.

Reviewed-by: fanggeng <fanggeng@lixiang.com>
Signed-off-by: Liang Jie <liangjie@lixiang.com>
Link: https://patch.msgid.link/20251208092730.262499-1-buaajxlj@163.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Liang Jie 2025-12-08 17:27:28 +08:00 committed by Greg Kroah-Hartman
parent ec34242097
commit 618b4aec12

View File

@ -377,7 +377,8 @@ static int rtw_drv_init(
if (status != _SUCCESS)
goto free_if1;
if (sdio_alloc_irq(dvobj) != _SUCCESS)
status = sdio_alloc_irq(dvobj);
if (status != _SUCCESS)
goto free_if1;
status = _SUCCESS;