On 08/04/2012 12:00 PM, Julia Lawall wrote: > From: Julia Lawall <Julia.Lawall@xxxxxxx> > > Using kfree to free data allocated with devm_kzalloc causes double frees. > > The semantic patch that fixes this problem is as follows: > (http://coccinelle.lip6.fr/) > > // <smpl> > @@ > expression x; > @@ > > x = devm_kzalloc(...) > ... > ?-kfree(x); > // </smpl> > > Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx> Applied. Thanks, Florian Tobias Schandinat > > --- > drivers/video/auo_k190x.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/video/auo_k190x.c b/drivers/video/auo_k190x.c > index 77da6a2..c03ecdd 100644 > --- a/drivers/video/auo_k190x.c > +++ b/drivers/video/auo_k190x.c > @@ -987,7 +987,6 @@ err_regfb: > fb_dealloc_cmap(&info->cmap); > err_cmap: > fb_deferred_io_cleanup(info); > - kfree(info->fbdefio); > err_defio: > vfree((void *)info->screen_base); > err_irq: > @@ -1022,7 +1021,6 @@ int __devexit auok190x_common_remove(struct platform_device *pdev) > fb_dealloc_cmap(&info->cmap); > > fb_deferred_io_cleanup(info); > - kfree(info->fbdefio); > > vfree((void *)info->screen_base); > > > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
