diff options
author | Adrian Bunk <bunk@stusta.de> | 2007-07-23 04:02:32 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-07-24 16:28:41 -0400 |
commit | dde9f93c205a791c5647c14d43003c578ebf6f2d (patch) | |
tree | cffa37e490a386cb88f68030ffbe60065c924e06 /drivers/net/acenic.c | |
parent | fcdff13905934874016f1899b15a9221c055d6ea (diff) |
drivers/net/acenic.c: fix check-after-use
The Coverity checker noted that we've already dereferenced "dev" when we
check whether it's NULL.
Since it's impossible that "dev" is NULL at this place this patch
removes the NULL check.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net/acenic.c')
-rw-r--r-- | drivers/net/acenic.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/net/acenic.c b/drivers/net/acenic.c index b78a4e5ceeb2..62e660a79387 100644 --- a/drivers/net/acenic.c +++ b/drivers/net/acenic.c | |||
@@ -3128,12 +3128,6 @@ static int __devinit read_eeprom_byte(struct net_device *dev, | |||
3128 | int result = 0; | 3128 | int result = 0; |
3129 | short i; | 3129 | short i; |
3130 | 3130 | ||
3131 | if (!dev) { | ||
3132 | printk(KERN_ERR "No device!\n"); | ||
3133 | result = -ENODEV; | ||
3134 | goto out; | ||
3135 | } | ||
3136 | |||
3137 | /* | 3131 | /* |
3138 | * Don't take interrupts on this CPU will bit banging | 3132 | * Don't take interrupts on this CPU will bit banging |
3139 | * the %#%#@$ I2C device | 3133 | * the %#%#@$ I2C device |