Hi Linus,
On 7/13/2018 12:46 AM, Linus Walleij wrote:
On Wed, Jul 11, 2018 at 2:25 PM Wei Yongjun <weiyongjun1@xxxxxxxxxx> wrote:
platform_get_resource() may fail and return NULL, so we should
better check it's return value to avoid a NULL pointer dereference
a bit later in the code.
This is detected by Coccinelle semantic patch.
@@
expression pdev, res, n, t, e, e1, e2;
@@
res = platform_get_resource(pdev, t, n);
+ if (!res)
+ return -EINVAL;
... when != res == NULL
e = devm_ioremap_nocache(e1, res->start, e2);
Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx>
Patch applied with Ray's ACK.
Would be nice to add the following Fixes tag:
Fixes: cc4fa83f66e9 ("pinctrl: nsp: add pinmux driver support for
Broadcom NSP SoC")
Thanks,
Ray
Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
