diff options
author | Adrian Bunk <bunk@stusta.de> | 2006-04-06 01:21:04 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2006-04-10 01:25:27 -0400 |
commit | ad96b485b4224785bcca478cfba02d06c46ee575 (patch) | |
tree | 9f5ef069ffef7e02e37d771c23444b6b73d1a6f1 /drivers/net | |
parent | e3a5cd9edff9a7a20de3c88c9d479704da98fb85 (diff) |
[TG3]: Fix a memory leak.
This patch fixes a memory leak (buf wasn't freed) spotted by the
Coverity checker.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/tg3.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c index 0b5358072172..d6047bd6805d 100644 --- a/drivers/net/tg3.c +++ b/drivers/net/tg3.c | |||
@@ -8034,9 +8034,13 @@ static int tg3_test_nvram(struct tg3 *tp) | |||
8034 | for (i = 0; i < size; i++) | 8034 | for (i = 0; i < size; i++) |
8035 | csum8 += buf8[i]; | 8035 | csum8 += buf8[i]; |
8036 | 8036 | ||
8037 | if (csum8 == 0) | 8037 | if (csum8 == 0) { |
8038 | return 0; | 8038 | err = 0; |
8039 | return -EIO; | 8039 | goto out; |
8040 | } | ||
8041 | |||
8042 | err = -EIO; | ||
8043 | goto out; | ||
8040 | } | 8044 | } |
8041 | 8045 | ||
8042 | /* Bootstrap checksum at offset 0x10 */ | 8046 | /* Bootstrap checksum at offset 0x10 */ |