Thadeu Lima de Souza Cascardo <cascardo@xxxxxxxxxx> wrote:
> If ieee80211_register_hw fails, the memory allocated for the firmware will
> not be released. Call deinit_sw_vars as the function that undoes the
> allocationes done by init_sw_vars.
>
> Fixes: cefe3dfdb9f5 ("rtl8192cu: Call ieee80211_register_hw from rtl_usb_probe")
> Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@xxxxxxxxxx>
> ---
> drivers/net/wireless/realtek/rtlwifi/usb.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/wireless/realtek/rtlwifi/usb.c b/drivers/net/wireless/realtek/rtlwifi/usb.c
> index c27b116ccdff..8ec687fab572 100644
> --- a/drivers/net/wireless/realtek/rtlwifi/usb.c
> +++ b/drivers/net/wireless/realtek/rtlwifi/usb.c
> @@ -1037,6 +1037,7 @@ int rtl_usb_probe(struct usb_interface *intf,
>
> error_init_vars:
> wait_for_completion(&rtlpriv->firmware_loading_complete);
> + rtlpriv->cfg->ops->deinit_sw_vars(hw);
Ah. You did this in this patch. Seemingly we can combine these two small
patches.
> error_out:
> rtl_deinit_core(hw);
> error_out2:
> --
> 2.34.1
