diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-10-08 14:57:44 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-10-08 07:29:56 -0700 |
| commit | 02339374cf3a28c30b5a137ab181eed358589dea (patch) | |
| tree | 96f7342556c38a47013a3f6ed75dcb48233982d4 /drivers/staging/bcm | |
| parent | ef5d2056456d63528daa762b32afa5c430635f33 (diff) | |
| download | linux-stable-02339374cf3a28c30b5a137ab181eed358589dea.tar.gz linux-stable-02339374cf3a28c30b5a137ab181eed358589dea.tar.bz2 linux-stable-02339374cf3a28c30b5a137ab181eed358589dea.zip | |
Staging: bcm: remove unneeded NULL check
The error handling here is wrong. If psIntfAdapter were NULL then we
would have a NULL dereference in the debug output on the error path.
But this function is only called from usbbcm_device_probe() when
psIntfAdapter is non-NULL.
Since the check isn't needed and I removed it instead of fixing it.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/bcm')
| -rw-r--r-- | drivers/staging/bcm/InterfaceInit.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/staging/bcm/InterfaceInit.c b/drivers/staging/bcm/InterfaceInit.c index 81523b1e4f08..824f9a45007a 100644 --- a/drivers/staging/bcm/InterfaceInit.c +++ b/drivers/staging/bcm/InterfaceInit.c @@ -529,11 +529,6 @@ INT InterfaceAdapterInit(PS_INTERFACE_ADAPTER psIntfAdapter) BOOLEAN bBcm16 = FALSE; UINT uiData = 0; - if(psIntfAdapter == NULL) - { - BCM_DEBUG_PRINT(psIntfAdapter->psAdapter,DBG_TYPE_INITEXIT, DRV_ENTRY, DBG_LVL_ALL, "Interface Adapter is NULL"); - return -EINVAL; - } /* Store the usb dev into interface adapter */ psIntfAdapter->udev = usb_get_dev(interface_to_usbdev( psIntfAdapter->interface)); |