Messages in this thread | | | Subject | Re: [PATCH] drm/tegra: sor: add NULL check on of_match_device() return value | From | Jon Hunter <> | Date | Fri, 7 Jul 2017 08:42:09 +0100 |
| |
On 07/07/17 07:11, Gustavo A. R. Silva wrote: > Check return value from call to of_match_device() > in order to prevent a NULL pointer dereference. > > In case of NULL print error message and return. > > Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com> > --- > drivers/gpu/drm/tegra/sor.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c > index a8f5289..ca3d9b1 100644 > --- a/drivers/gpu/drm/tegra/sor.c > +++ b/drivers/gpu/drm/tegra/sor.c > @@ -2538,6 +2538,10 @@ static int tegra_sor_probe(struct platform_device *pdev) > int err; > > match = of_match_device(tegra_sor_of_match, &pdev->dev); > + if (!match) { > + dev_err(&pdev->dev, "failed to match device\n"); > + return -ENODEV; > + }
Given that Tegra always uses device-tree, I believe that this cannot happen and so this additional check is not needed.
Jon
-- nvpublic
| |