]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
PCI: ibmphp_pci.c: fix NULL dereference
authorAdrian Bunk <bunk@stusta.de>
Fri, 17 Nov 2006 01:19:25 +0000 (02:19 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 1 Dec 2006 22:37:00 +0000 (14:37 -0800)
The correct order is: NULL check before dereference

Spotted by the Coverity checker.

Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/pci/hotplug/ibmphp_pci.c

index d87a9e3eaeeb3ea92e684251a7cfbb3528900daf..d8f05d7a3c7203d5ca838ed26928f586b7feba72 100644 (file)
@@ -1371,12 +1371,12 @@ static int unconfigure_boot_bridge (u8 busno, u8 device, u8 function)
        }
 
        bus = ibmphp_find_res_bus (sec_number);
-       debug ("bus->busno is %x\n", bus->busno);
-       debug ("sec_number is %x\n", sec_number);
        if (!bus) {
                err ("cannot find Bus structure for the bridged device\n");
                return -EINVAL;
        }
+       debug("bus->busno is %x\n", bus->busno);
+       debug("sec_number is %x\n", sec_number);
 
        ibmphp_remove_bus (bus, busno);