Re: [PATCH] ISP1704 USB Charger: Fix use-after-free error inisp1704_charger_probe()

From: Anton Vorontsov
Date: Sat May 05 2012 - 23:13:44 EST


On Sun, Apr 22, 2012 at 10:34:59PM +0200, Marek Vasut wrote:
[...]
> > -
> > dev_err(&pdev->dev, "failed to register isp1704 with error %d\n", ret);
> > -
> > isp1704_charger_set_power(isp, 0);
> > + kfree(isp);

Dan Carpenter sent a similar fix ~month ago, so it's in already.

> Use devm_kzalloc() and be done with all this goo?

Yeah, that's actually a good idea.

Thanks,

--
Anton Vorontsov
Email: cbouatmailru@xxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/