diff options
| author | Lorenzo Pieralisi | 2015-02-11 04:58:35 +0000 |
|---|---|---|
| committer | Bjorn Helgaas | 2015-02-11 15:27:15 -0600 |
| commit | feb28979c137ba3f649ad36fc27c85c64c111f78 (patch) | |
| tree | 72bcf44869bb59a71fe06f0741240229e503aa8a /drivers/of | |
| parent | 872912352c5be930e9568e5f3b6d73107d9f278d (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>
Diffstat (limited to 'drivers/of')
| -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 110fece2ff53..62426d81a4d6 100644 --- a/drivers/of/of_pci.c +++ b/drivers/of/of_pci.c @@ -229,7 +229,6 @@ parse_failed: resource_list_for_each_entry(window, resources) kfree(window->res); pci_free_resource_list(resources); - kfree(bus_range); return err; } EXPORT_SYMBOL_GPL(of_pci_get_host_bridge_resources); |