]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
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)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 23 Mar 2011 19:50:30 +0000 (12:50 -0700)
commit2de4380602b42871a073bead7153b5c662e3289a
treee50519cc59c3cf26256194ce64f6351699cf99f7
parentf18cf02bf53c19a2a79d2f5a0d063af1c10947c3
PCI: sysfs: Fix failure path for addition of "vpd" attribute

commit 0f12a4e29368a9476076515881d9ef4e5876c6e2 upstream.

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.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/pci/pci-sysfs.c