On Monday, August 11, 2014 5:00 AM, Himangi Saraogi wrote:
>
> Using kfree to free data allocated with devm_kzalloc causes double frees.
>
> The Coccinelle semantic patch that fixes this problem is as follows:
>
> // <smpl>
> @@
> expression x;
> @@
>
> x = devm_kzalloc(...)
> ...
> ?-kfree(x);
> // </smpl>
>
> Signed-off-by: Himangi Saraogi <himangi...@gmail.com>
> Acked-by: Julia Lawall <julia.law...@lip6.fr>
Reviewed-by: Jingoo Han <jg1....@samsung.com>
Best regards,
Jingoo Han
> ---
> drivers/usb/phy/phy-gpio-vbus-usb.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/usb/phy/phy-gpio-vbus-usb.c
> b/drivers/usb/phy/phy-gpio-vbus-usb.c
> index ea9e705..f4b14bd 100644
> --- a/drivers/usb/phy/phy-gpio-vbus-usb.c
> +++ b/drivers/usb/phy/phy-gpio-vbus-usb.c
> @@ -260,10 +260,8 @@ static int gpio_vbus_probe(struct platform_device *pdev)
>
> gpio_vbus->phy.otg = devm_kzalloc(&pdev->dev, sizeof(struct usb_otg),
> GFP_KERNEL);
> - if (!gpio_vbus->phy.otg) {
> - kfree(gpio_vbus);
> + if (!gpio_vbus->phy.otg)
> return -ENOMEM;
> - }
>
> platform_set_drvdata(pdev, gpio_vbus);
> gpio_vbus->dev = &pdev->dev;
> --
> 1.9.1
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
