summaryrefslogtreecommitdiffstats
path: root/drivers/pci
diff options
context:
space:
mode:
authorJulia Lawall <julia@diku.dk>2007-11-20 08:41:16 +0100
committerGreg Kroah-Hartman <gregkh@suse.de>2007-11-28 14:35:26 -0800
commit151fc5dfc87964e85a1cbbb9cc2c0703c017c2ed (patch)
tree3c5bf6adaee5a5c3545071de5154c6e43b6bb901 /drivers/pci
parentbf164410d08dc83df416e3a6a43ab29bf88890ed (diff)
downloadlinux-151fc5dfc87964e85a1cbbb9cc2c0703c017c2ed.tar.gz
linux-151fc5dfc87964e85a1cbbb9cc2c0703c017c2ed.tar.bz2
linux-151fc5dfc87964e85a1cbbb9cc2c0703c017c2ed.zip
PCI: drivers/pci/pci-sysfs.c: Add missing pci_dev_put
There should be a pci_dev_put when breaking out of a loop that iterates over calls to pci_get_device and similar functions. This was fixed using the following semantic patch. // <smpl> @@ identifier d; type T; expression e; iterator for_each_pci_dev; @@ T *d; ... for_each_pci_dev(d) {... when != pci_dev_put(d) when != e = d ( return d; | + pci_dev_put(d); ? return ...; ) ...} // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/pci')
-rw-r--r--drivers/pci/pci-sysfs.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c
index 1b7b2812bf2d..7d1877341aad 100644
--- a/drivers/pci/pci-sysfs.c
+++ b/drivers/pci/pci-sysfs.c
@@ -702,8 +702,10 @@ static int __init pci_sysfs_init(void)
sysfs_initialized = 1;
for_each_pci_dev(pdev) {
retval = pci_create_sysfs_dev_files(pdev);
- if (retval)
+ if (retval) {
+ pci_dev_put(pdev);
return retval;
+ }
}
return 0;