PCI: sysfs: Fix failure path for addition of "vpd" attribute
authorBen Hutchings <bhutchings@solarflare.com>
Thu, 13 Jan 2011 19:47:56 +0000 (19:47 +0000)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Tue, 8 Feb 2011 18:02:46 +0000 (10:02 -0800)
Commit 280c73d ("PCI: centralize the capabilities code in
pci-sysfs.c") changed the initialisation of the "rom" and "vpd"
attributes, and made the failure path for the "vpd" attribute
incorrect.  We must free the new attribute structure (attr), but
instead we currently free dev->vpd->attr.  That will normally be NULL,
resulting in a memory leak, but it might be a stale pointer, resulting
in a double-free.

Found by inspection; compile-tested only.

Cc: stable@kernel.org
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/pci-sysfs.c

index 8ecaac983923d9c6b22c2e0246907db089b5e43a..cf2b3654b4a81bc8d9753a9dbd8c319638efda2b 100644 (file)
@@ -1087,7 +1087,7 @@ static int pci_create_capabilities_sysfs(struct pci_dev *dev)
                attr->write = write_vpd_attr;
                retval = sysfs_create_bin_file(&dev->dev.kobj, attr);
                if (retval) {
-                       kfree(dev->vpd->attr);
+                       kfree(attr);
                        return retval;
                }
                dev->vpd->attr = attr;