On Wed, Mar 09, 2016 at 02:45:36AM +0200, Vladimir Zapolskiy 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>
Acked-by: Shawn Guo <shawnguo@xxxxxxxxxx>
--
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: Shawn Guo <shawnguo@xxxxxxxxxx>
- Date: Tue, 15 Mar 2016 17:00:04 +0800
- In-reply-to: <1457484336-8928-1-git-send-email-vz@mleia.com>
- User-agent: Mutt/1.5.21 (2010-09-15)
- 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] pinctrl: pistachio: fix mfio84-89 function description and pinmux.
- Next by Date: Re: [RFC] pinctrl: sh-pfc: only use dummy states for non-DT platforms
- Previous by thread: [PATCH] pinctrl: freescale: imx: fix bogus check of of_iomap() return value
- Next by thread: Re: [PATCH] pinctrl: freescale: imx: fix bogus check of of_iomap() return value
- Index(es):
 |