diff options
author | Adrian Bunk <bunk@stusta.de> | 2007-07-23 10:02:32 +0200 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-07-24 22:28:41 +0200 |
commit | dde9f93c205a791c5647c14d43003c578ebf6f2d (patch) | |
tree | cffa37e490a386cb88f68030ffbe60065c924e06 /drivers/net/acenic.c | |
parent | defxx: Use __maybe_unused rather than a local hack (diff) | |
download | linux-dde9f93c205a791c5647c14d43003c578ebf6f2d.tar.xz linux-dde9f93c205a791c5647c14d43003c578ebf6f2d.zip |
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, int result = 0; short i; - if (!dev) { - printk(KERN_ERR "No device!\n"); - result = -ENODEV; - goto out; - } - /* * Don't take interrupts on this CPU will bit banging * the %#%#@$ I2C device |