On Mon, 7 Sep 2020 11:20:16 +0100
Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx> wrote:
> On Fri, Sep 04, 2020 at 03:09:04PM +0100, Lorenzo Pieralisi wrote:
> > The root bus checks rework in:
> >
> > commit d84c572de1a3 ("PCI: rockchip: Use pci_is_root_bus() to check
> > if bus is root bus")
> >
> > caused a regression whereby in rockchip_pcie_valid_device() if
> > the bus parameter is the root bus and the dev value == 0 the
> > function should return 1 (ie true) without checking if the
> > bus->parent pointer is a root bus because that triggers a NULL
> > pointer dereference.
> >
> > Fix this by streamlining the root bus detection.
> >
> > Fixes: d84c572de1a3 ("PCI: rockchip: Use pci_is_root_bus() to check
> > if bus is root bus") Reported-by: Samuel Dionne-Riel
> > <samuel@xxxxxxxxxxxxxxx> Signed-off-by: Lorenzo Pieralisi
> > <lorenzo.pieralisi@xxxxxxx> Cc: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
> > Cc: Rob Herring <robh@xxxxxxxxxx>
> > Cc: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx>
> > ---
> > drivers/pci/controller/pcie-rockchip-host.c | 11 ++++-------
> > 1 file changed, 4 insertions(+), 7 deletions(-)
>
> Hi Samuel,
>
> I would kindly ask you please to test it since I changed the code,
> I need your Tested-by before asking Bjorn to merge it.
>
Hi,
I'm sorry, I had tested it, but didn't reply back as it worked. Not
being familiar with the customs of the mailing list.
Again, just in case, verified to work and fix the issue on top of
v5.9-rc3.
--
Samuel Dionne-Riel
