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)
commit0f12a4e29368a9476076515881d9ef4e5876c6e2
treecb58b69f498d132aab3a68f8fb259520ef3c3934
parent100b33c8bd8a3235fd0b7948338d6cbb3db3c63d
PCI: sysfs: Fix failure path for addition of "vpd" attribute

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