re: staging: r8712u: Fix regression caused by commit 8c213fa

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Larry Finger,

This is a semi-automatic email about new static checker warnings.

The patch 2080913e017a: "staging: r8712u: Fix regression caused by 
commit 8c213fa" from Feb 25, 2012, leads to the following Smatch 
complaint:

drivers/staging/rtl8712/usb_intf.c:629 r871xu_dev_remove()
	 warn: variable dereferenced before check 'padapter' (see line 625)

drivers/staging/rtl8712/usb_intf.c
   624		if (padapter->fw_found)
                    ^^^^^^^^^^^^^^^^^^
new

   625			release_firmware(padapter->fw);
                                         ^^^^^^^^^^^^
new
   626		/* never exit with a firmware callback pending */
   627		wait_for_completion(&padapter->rtl8712_fw_ready);
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^
new

   628		usb_set_intfdata(pusb_intf, NULL);
   629		if (padapter) {
                    ^^^^^^^^
Old check.

   630			if (drvpriv.drv_registered == true)
   631				padapter->bSurpriseRemoved = true;


regards,
dan carpenter

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux