From: Po Liu Date: Mon, 29 Aug 2016 07:28:01 +0000 (+0800) Subject: PCI: Check for pci_setup_device() failure in pci_iov_add_virtfn() X-Git-Tag: v5.15~12665^2~5^2~3 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=156c55325d30261d250e88ed3a39f22008f4ca16;p=platform%2Fkernel%2Flinux-starfive.git PCI: Check for pci_setup_device() failure in pci_iov_add_virtfn() If pci_setup_device() returns failure, we must return failure from pci_iov_add_virtfn(). If we ignore the failure and continue with an uninitialized pci_dev for virtfn, we crash later when we try to use those uninitialized parts. Signed-off-by: Po Liu Signed-off-by: Bjorn Helgaas --- diff --git a/drivers/pci/iov.c b/drivers/pci/iov.c index 2194b44..e30f05c 100644 --- a/drivers/pci/iov.c +++ b/drivers/pci/iov.c @@ -136,7 +136,10 @@ int pci_iov_add_virtfn(struct pci_dev *dev, int id, int reset) virtfn->devfn = pci_iov_virtfn_devfn(dev, id); virtfn->vendor = dev->vendor; pci_read_config_word(dev, iov->pos + PCI_SRIOV_VF_DID, &virtfn->device); - pci_setup_device(virtfn); + rc = pci_setup_device(virtfn); + if (rc) + goto failed0; + virtfn->dev.parent = dev->dev.parent; virtfn->physfn = pci_dev_get(dev); virtfn->is_virtfn = 1;