Am Donnerstag, den 13.05.2010, 22:01 +0200 schrieb Julia Lawall: > From: Julia Lawall <julia@xxxxxxx> > > Use kzalloc rather than the combination of kmalloc and memset. > > The semantic patch that makes this change is as follows: > (http://coccinelle.lip6.fr/) > > // <smpl> > @@ > expression x,size,flags; > statement S; > @@ > > -x = kmalloc(size,flags); > +x = kzalloc(size,flags); > if (x == NULL) S > -memset(x, 0, size); > // </smpl> > > Signed-off-by: Julia Lawall <julia@xxxxxxx> Reviewed-by: Thomas Meyer <thomas@xxxxxxxx> > > --- > drivers/parport/share.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff -u -p a/drivers/parport/share.c b/drivers/parport/share.c > --- a/drivers/parport/share.c > +++ b/drivers/parport/share.c > @@ -282,14 +282,13 @@ struct parport *parport_register_port(un > int device; > char *name; > > - tmp = kmalloc(sizeof(struct parport), GFP_KERNEL); > + tmp = kzalloc(sizeof(struct parport), GFP_KERNEL); > if (!tmp) { > printk(KERN_WARNING "parport: memory squeeze\n"); > return NULL; > } > > /* Init our structure */ > - memset(tmp, 0, sizeof(struct parport)); > tmp->base = base; > tmp->irq = irq; > tmp->dma = dma; -- 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
