On Wed, Mar 9, 2016 at 1:45 AM, Vladimir Zapolskiy <vz@xxxxxxxxx> wrote:
> On error path of_iomap() returns NULL, hence IS_ERR() check is invalid
> and may cause a NULL pointer dereference, the change fixes this
> problem.
>
> While we are here invert a device node check to simplify the code.
>
> Fixes: 26d8cde5260b ("pinctrl: freescale: imx: add shared input select reg support")
> Signed-off-by: Vladimir Zapolskiy <vz@xxxxxxxxx>
Patch applied with Shawn's ACK.
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
Re: [PATCH] pinctrl: freescale: imx: fix bogus check of of_iomap() return value
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH] pinctrl: freescale: imx: fix bogus check of of_iomap() return value
- From: Linus Walleij <linus.walleij@xxxxxxxxxx>
- Date: Wed, 16 Mar 2016 13:57:09 +0100
- In-reply-to: <1457484336-8928-1-git-send-email-vz@mleia.com>
- References:
- [PATCH] pinctrl: freescale: imx: fix bogus check of of_iomap() return value
- From: Vladimir Zapolskiy
- [PATCH] pinctrl: freescale: imx: fix bogus check of of_iomap() return value
- Prev by Date: Re: [PATCH 1/1] gpio: mcp23s08: Fix clearing of interrupt.
- Next by Date: Re: [PATCH 00/50] pinctrl: Add and use devm_ apis for pinctrl_{register, unregister}
- Previous by thread: Re: [PATCH] pinctrl: freescale: imx: fix bogus check of of_iomap() return value
- Next by thread: [PATCH] pinctrl: pxa2xx: export symbols
- Index(es):
 |