From: Pablo Neira Ayuso > Sent: 13 November 2016 22:25 > Allocate memory with kmalloc() rather than kzalloc(): the string > is immediately initialized so it is unnecessary to zero out > the allocated memory area. > > Ported from a patch proposed by Sergey Popovich <popovich_sergei@xxxxxxx>. > > Suggested-by: Sergey Popovich <popovich_sergei@xxxxxxx> > Signed-off-by: Jozsef Kadlecsik <kadlec@xxxxxxxxxxxxxxxxx> > --- > include/linux/netfilter/ipset/ip_set_comment.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/netfilter/ipset/ip_set_comment.h > b/include/linux/netfilter/ipset/ip_set_comment.h > index bae5c7609be2..5444b1bbe656 100644 > --- a/include/linux/netfilter/ipset/ip_set_comment.h > +++ b/include/linux/netfilter/ipset/ip_set_comment.h > @@ -34,7 +34,7 @@ ip_set_init_comment(struct ip_set_comment *comment, > return; > if (unlikely(len > IPSET_MAX_COMMENT_SIZE)) > len = IPSET_MAX_COMMENT_SIZE; > - c = kzalloc(sizeof(*c) + len + 1, GFP_ATOMIC); > + c = kmalloc(sizeof(*c) + len + 1, GFP_ATOMIC); > if (unlikely(!c)) > return; > strlcpy(c->str, ext->comment, len + 1); I think I'd do a memcpy() and zero the last byte. Note that any other parts of 'c->' are no longer zeroed by this code fragment. David -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
