diff options
| author | Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> | 2015-02-11 04:58:35 +0000 |
|---|---|---|
| committer | Alex Shi <alex.shi@linaro.org> | 2015-12-16 15:29:25 +0800 |
| commit | c749bcc15a0e9a800f50ae7a6cc9e49881918286 (patch) | |
| tree | bf8c9dcc977c4a7fa9fd4db6eb4025a26a3c29b8 /drivers | |
| parent | 2bab01c9c05f7a2f9c9d9cbdfda56798f482070a (diff) | |
of/pci: Remove duplicate kfree in of_pci_get_host_bridge_resources()
Commit d2be00c0fb5a ("of/pci: Free resources on failure in
of_pci_get_host_bridge_resources()") fixed the error path so it frees
everything on the "resources" list. That list includes the bus_range, so
we should not free it again.
Remove the superfluous free of bus_range.
[bhelgaas: changelog]
Fixes: d2be00c0fb5a ("of/pci: Free resources on failure in of_pci_get_host_bridge_resources()")
Reported-by: Jiang Liu <jiang.liu@linux.intel.com>
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
CC: Rafael J. Wysocki <rjw@rjwysocki.net>
(cherry picked from commit feb28979c137ba3f649ad36fc27c85c64c111f78)
Signed-off-by: Alex Shi <alex.shi@linaro.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/of/of_pci.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/of/of_pci.c b/drivers/of/of_pci.c index ecc5fa5640d2..8a74b1d399a1 100644 --- a/drivers/of/of_pci.c +++ b/drivers/of/of_pci.c @@ -229,7 +229,6 @@ parse_failed: list_for_each_entry(window, resources, list) kfree(window->res); pci_free_resource_list(resources); - kfree(bus_range); return err; } EXPORT_SYMBOL_GPL(of_pci_get_host_bridge_resources); |