Re: [PATCH] staging: rtl8188eu: fix null dereference when kzalloc fails

[Connor Kuehl <connor.kuehl@xxxxxxxxxxxxx>]

On 10/1/19 6:11 AM, Dan Carpenter wrote:
> There is another one earlier in the function as well.
>
> drivers/staging/rtl8188eu/os_dep/usb_intf.c
>     336
>     337          pnetdev = rtw_init_netdev(padapter);
>     338          if (!pnetdev)
>     339                  goto free_adapter;
>     340          SET_NETDEV_DEV(pnetdev, dvobj_to_dev(dvobj));
>     341          padapter = rtw_netdev_priv(pnetdev);
>     342
>     343          if (padapter->registrypriv.monitor_enable) {
>     344                  pmondev = rtl88eu_mon_init();
>     345                  if (!pmondev)
>     346                          netdev_warn(pnetdev, "Failed to initialize monitor interface");
>
> goto free_adapter.
>
>     347                  padapter->pmondev = pmondev;
>     348          }
>     349
>     350          padapter->HalData = kzalloc(sizeof(struct hal_data_8188e), GFP_KERNEL);
>     351          if (!padapter->HalData)
>     352                  DBG_88E("cant not alloc memory for HAL DATA\n");
>     353
>
> >   	padapter->HalData = kzalloc(sizeof(struct hal_data_8188e), GFP_KERNEL);
> > -	if (!padapter->HalData)
> > -		DBG_88E("cant not alloc memory for HAL DATA\n");
> > +	if (!padapter->HalData) {
> > +		DBG_88E("Failed to allocate memory for HAL data\n");
>
> Remove this debug printk.
>
> > +		goto free_adapter;
> > +	}

Hi Dan,

Sorry for such a late response! By the time I saw the e-mail with your 
feedback I also saw another e-mail saying this patch was accepted into a 
staging-linus tree. I'll address your comments in a separate patch.

Thank you,

Connor