| To: | netdev@xxxxxxxxxxx |
|---|---|
| Subject: | [PATCH] Bogus return status in drivers/net/amd8111e.c |
| From: | Francois Romieu <romieu@xxxxxxxxxxxxx> |
| Date: | Sat, 20 Dec 2003 01:17:18 +0100 |
| Cc: | Jeff Garzik <jgarzik@xxxxxxxxx> |
| In-reply-to: | <20031219233231.A2420@electric-eye.fr.zoreil.com>; from romieu@fr.zoreil.com on Fri, Dec 19, 2003 at 11:32:31PM +0100 |
| References: | <20031219233231.A2420@electric-eye.fr.zoreil.com> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.2.5.1i |
Patch applies on top of 2.6.0 + Jeff Garzik's 2.6.0-test11-bk5-netdrvr-exp2
The status variable is not set before branching to the error path.
drivers/net/amd8111e.c | 2 ++
1 files changed, 2 insertions(+)
diff -puN drivers/net/amd8111e.c~trivial-amd8111e-fscked-return-status
drivers/net/amd8111e.c
---
linux-2.6.0-vanilla/drivers/net/amd8111e.c~trivial-amd8111e-fscked-return-status
2003-12-19 23:34:46.000000000 +0100
+++ linux-2.6.0-vanilla-fr/drivers/net/amd8111e.c 2003-12-19
23:37:15.000000000 +0100
@@ -1819,6 +1819,8 @@ static int __devinit amd8111e_probe_one(
pci_set_master(pdev);
+ err = -EIO;
+
/* Find power-management capability. */
if((pm_cap = pci_find_capability(pdev, PCI_CAP_ID_PM))==0){
printk(KERN_ERR "amd8111e: No Power Management capability, "
_
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [RFC] multiple eth0 mixed PCI/ISA init, Stephen Hemminger |
|---|---|
| Next by Date: | [PATCH] 2.6.0 - r8169 64 bit, Francois Romieu |
| Previous by Thread: | [PATCH] Bogus return status in drivers/net/tg3.c, Francois Romieu |
| Next by Thread: | [PATCH] 2.6.0 - r8169 64 bit, Francois Romieu |
| Indexes: | [Date] [Thread] [Top] [All Lists] |