Re: [PATCH] staging: r8188eu: handle errors from ReadAdapterInfo8188EU

From: Greg Kroah-Hartman
Date: Sun Jul 24 2022 - 12:19:12 EST


On Sun, Jul 24, 2022 at 06:14:05PM +0200, Martin Kaiser wrote:
> --- a/drivers/staging/r8188eu/os_dep/usb_intf.c
> +++ b/drivers/staging/r8188eu/os_dep/usb_intf.c
> @@ -332,7 +332,8 @@ static struct adapter *rtw_usb_if1_init(struct dvobj_priv *dvobj,
> rtl8188eu_interface_configure(padapter);
>
> /* step read efuse/eeprom data and get mac_addr */
> - ReadAdapterInfo8188EU(padapter);
> + if (ReadAdapterInfo8188EU(padapter) < 0)
> + goto handle_dualmac;

You threw away the error returned to you here, why? Shouldn't it
propagate upward?

thanks,

greg k-h