aboutsummaryrefslogtreecommitdiffstats
path: root/arch/powerpc/kernel/pci_32.c
diff options
context:
space:
mode:
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>2007-12-20 22:53:27 -0500
committerPaul Mackerras <paulus@samba.org>2007-12-21 06:14:07 -0500
commitb1b166b7ea2ec15700a8ba5c6ec99c2f9ecad89e (patch)
tree066e5c4b1ed75eefbd3d443fe8a87f1f61fc9648 /arch/powerpc/kernel/pci_32.c
parent0094f2cdcfb6f2132b2ea3b4e85e0f6899c8595b (diff)
[POWERPC] Fix possible NULL deref in ppc32 PCI
The 32-bit PCI code tests if "bus" is non-NULL after calling pci_scan_bus_parented() in one place but not another before dereferencing it. Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/powerpc/kernel/pci_32.c')
-rw-r--r--arch/powerpc/kernel/pci_32.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/arch/powerpc/kernel/pci_32.c b/arch/powerpc/kernel/pci_32.c
index 37cb7b944319..a9c6cb24aab0 100644
--- a/arch/powerpc/kernel/pci_32.c
+++ b/arch/powerpc/kernel/pci_32.c
@@ -442,9 +442,10 @@ static int __init pcibios_init(void)
442 hose->last_busno = 0xff; 442 hose->last_busno = 0xff;
443 bus = pci_scan_bus_parented(hose->parent, hose->first_busno, 443 bus = pci_scan_bus_parented(hose->parent, hose->first_busno,
444 hose->ops, hose); 444 hose->ops, hose);
445 if (bus) 445 if (bus) {
446 pci_bus_add_devices(bus); 446 pci_bus_add_devices(bus);
447 hose->last_busno = bus->subordinate; 447 hose->last_busno = bus->subordinate;
448 }
448 if (pci_assign_all_buses || next_busno <= hose->last_busno) 449 if (pci_assign_all_buses || next_busno <= hose->last_busno)
449 next_busno = hose->last_busno + pcibios_assign_bus_offset; 450 next_busno = hose->last_busno + pcibios_assign_bus_offset;
450 } 451 }