On Fri, Nov 18, 2016 at 12:35 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > Smatch complains that we dereference "map" before testing it for NULL > which is true. We should be testing "*map" instead. Also on the error > path, we should free *map and set it to NULL. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Patch applied with Maxime'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: sunxi: Testing the wrong variable
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [patch] pinctrl: sunxi: Testing the wrong variable
- From: Linus Walleij <linus.walleij@xxxxxxxxxx>
- Date: Tue, 22 Nov 2016 09:56:18 +0100
- In-reply-to: <20161118113557.GA3281@mwanda>
- References: <20161118113557.GA3281@mwanda>
- References:
- [patch] pinctrl: sunxi: Testing the wrong variable
- From: Dan Carpenter
- [patch] pinctrl: sunxi: Testing the wrong variable
- Prev by Date: Re: [PATCH V2 2/2] pinctrl: tegra: Add driver to configure voltage and power of io pads
- Next by Date: Re: [PATCH] pinctrl: stm32: use builtin_platform_driver
- Previous by thread: Re: [patch] pinctrl: sunxi: Testing the wrong variable
- Next by thread: [PATCH] gpio: etraxfs: use builtin_platform_driver
- Index(es):
 |