From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick McHardy Subject: Re: [PATCH 5/5] net/ipv4 : Correct the size argument to kzalloc Date: Mon, 04 Jan 2010 15:23:23 +0100 Message-ID: <4B41F9DB.1010908@trash.net> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: "David S. Miller" , Alexey Kuznetsov , "Pekka Savola (ipv6)" , James Morris , Hideaki YOSHIFUJI , netfilter-devel@vger.kernel.org, netfilter@vger.kernel.org, coreteam@netfilter.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org To: Julia Lawall Return-path: Received: from stinky.trash.net ([213.144.137.162]:34970 "EHLO stinky.trash.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752909Ab0ADOXZ (ORCPT ); Mon, 4 Jan 2010 09:23:25 -0500 In-Reply-To: Sender: netfilter-devel-owner@vger.kernel.org List-ID: Julia Lawall wrote: > From: Julia Lawall > > obj has type struct snmp_object **, not struct snmp_object *. But indeed > it is not even clear why kmalloc is needed. The memory is freed by the end > of the function, so the local variable of pointer type should be sufficient. Thanks. As this is basically just an unneeded and oversized allocation without any bad consequences, I've applied it to nf-next-2.6.git.