PCI: fix upstream P2P bridge checks when enabling OBFF and LTR
authorBjorn Helgaas <bhelgaas@google.com>
Tue, 19 Jun 2012 13:35:34 +0000 (07:35 -0600)
committerBjorn Helgaas <bhelgaas@google.com>
Wed, 20 Jun 2012 23:28:53 +0000 (17:28 -0600)
commit8291550f8479fde2cee571d1b367e6918819f189
tree111ffade25983b6c92dc8b2dc8634f2ed978a533
parentd6d88c832eaea6c6947ddf7b664601930a9f8a14
PCI: fix upstream P2P bridge checks when enabling OBFF and LTR

pci_enable_obff() and pci_enable_ltr() incorrectly check "dev->bus" instead
of "dev->bus->self" to determine whether the upstream device is a P2P
bridge or a host bridge.  For devices on the root bus, the upstream device
is a host bridge, "dev->bus != NULL" and "dev->bus->self == NULL", and we
panic with a null pointer dereference.

These functions should previously have panicked when called on devices
supporting OBFF or LTR, so they should be regarded as untested.

Found by Coverity (CID 143038 and CID 143039).

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/pci.c