lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 10 Aug 2014 22:26:47 +0200 From: Daniel Mack <zonque@...il.com> To: Himangi Saraogi <himangi774@...il.com>, "David S. Miller" <davem@...emloft.net>, Olof Johansson <olof@...om.net>, Dan Carpenter <dan.carpenter@...cle.com>, Felipe Balbi <balbi@...com>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org CC: Julia Lawall <julia.lawall@...6.fr> Subject: Re: [PATCH] drivers: net: davinci_cpdma: drop devm_kfree of devm_kzalloc'd data On 08/10/2014 09:58 PM, Himangi Saraogi wrote: > devm_kfree should not have to be explicitly used. > > The Coccinelle semantic patch that fixes this problem is as follows: > > // <smpl> > @@ > expression x,d; > @@ > > x = devm_kzalloc(...) > ... > ?-devm_kfree(d,x); > // </smpl> > > Signed-off-by: Himangi Saraogi <himangi774@...il.com> > Acked-by: Julia Lawall <julia.lawall@...6.fr> Acked-by: Daniel Mack <zonque@...il.com> However, you need to resend this patch after 3.17-rc1 has been released and the net-next branch was opened again. Thanks, Daniel > --- > drivers/net/ethernet/ti/davinci_cpdma.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/net/ethernet/ti/davinci_cpdma.c b/drivers/net/ethernet/ti/davinci_cpdma.c > index 4a000f6..af673f5 100644 > --- a/drivers/net/ethernet/ti/davinci_cpdma.c > +++ b/drivers/net/ethernet/ti/davinci_cpdma.c > @@ -512,7 +512,6 @@ struct cpdma_chan *cpdma_chan_create(struct cpdma_ctlr *ctlr, int chan_num, > spin_lock_irqsave(&ctlr->lock, flags); > if (ctlr->channels[chan_num]) { > spin_unlock_irqrestore(&ctlr->lock, flags); > - devm_kfree(ctlr->dev, chan); > return ERR_PTR(-EBUSY); > } > > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists