Re: [PATCH v3] staging: rtl8192u: add error handling for usb_alloc_urb

[kbuild test robot]

Hi,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on staging/staging-testing]
[also build test ERROR on v4.17 next-20180615]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/linux-kernel-owner-vger-kernel-org/staging-rtl8192u-add-error-handling-for-usb_alloc_urb/20180616-012944
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 8.1.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=8.1.0 make.cross ARCH=xtensa 

All errors (new ones prefixed by >>):

   drivers/staging//rtl8192u/r8192U_core.c: In function 'rtl8192_usb_initendpoints':
>> drivers/staging//rtl8192u/r8192U_core.c:1701:7: error: 'i' undeclared (first use in this function)
     for (i = 0; i < (MAX_RX_URB + 1); i++)
          ^
   drivers/staging//rtl8192u/r8192U_core.c:1701:7: note: each undeclared identifier is reported only once for each function it appears in

vim +/i +1701 drivers/staging//rtl8192u/r8192U_core.c

  1688	
  1689		memset(priv->rx_urb, 0, sizeof(struct urb *) * MAX_RX_URB);
  1690		priv->pp_rxskb = kcalloc(MAX_RX_URB, sizeof(struct sk_buff *),
  1691					 GFP_KERNEL);
  1692		if (!priv->pp_rxskb) {
  1693			DMESGE("Endpoint Alloc Failure");
  1694			goto out_release_mem;
  1695		}
  1696	
  1697		netdev_dbg(dev, "End of initendpoints\n");
  1698		return 0;
  1699	
  1700	out_release_mem:
> 1701		for (i = 0; i < (MAX_RX_URB + 1); i++)
  1702			kfree(priv->rx_urb[i]);
  1703		kfree(priv->rx_urb);
  1704		priv->rx_urb = NULL;
  1705		return -ENOMEM;
  1706	}
  1707	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip