diff options
author | Dan Carpenter <error27@gmail.com> | 2010-10-08 08:57:44 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-10-08 10:29:56 -0400 |
commit | 02339374cf3a28c30b5a137ab181eed358589dea (patch) | |
tree | 96f7342556c38a47013a3f6ed75dcb48233982d4 /drivers | |
parent | ef5d2056456d63528daa762b32afa5c430635f33 (diff) |
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')
-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 81523b1e4f0..824f9a45007 100644 --- a/drivers/staging/bcm/InterfaceInit.c +++ b/drivers/staging/bcm/InterfaceInit.c | |||
@@ -529,11 +529,6 @@ INT InterfaceAdapterInit(PS_INTERFACE_ADAPTER psIntfAdapter) | |||
529 | BOOLEAN bBcm16 = FALSE; | 529 | BOOLEAN bBcm16 = FALSE; |
530 | UINT uiData = 0; | 530 | UINT uiData = 0; |
531 | 531 | ||
532 | if(psIntfAdapter == NULL) | ||
533 | { | ||
534 | BCM_DEBUG_PRINT(psIntfAdapter->psAdapter,DBG_TYPE_INITEXIT, DRV_ENTRY, DBG_LVL_ALL, "Interface Adapter is NULL"); | ||
535 | return -EINVAL; | ||
536 | } | ||
537 | /* Store the usb dev into interface adapter */ | 532 | /* Store the usb dev into interface adapter */ |
538 | psIntfAdapter->udev = usb_get_dev(interface_to_usbdev( | 533 | psIntfAdapter->udev = usb_get_dev(interface_to_usbdev( |
539 | psIntfAdapter->interface)); | 534 | psIntfAdapter->interface)); |