From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Subject: Re: tipc: Delete error messages for failed memory allocations in three functions Date: Tue, 23 May 2017 18:23:17 +0200 Message-ID: References: <7ac9d513-6f9f-a096-e68f-81a722c2723c@users.sourceforge.net> <1495548133.2093.58.camel@perches.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: "David S. Miller" , Jon Maloy , Ying Xue , LKML , kernel-janitors@vger.kernel.org To: Joe Perches , tipc-discussion@lists.sourceforge.net, netdev@vger.kernel.org Return-path: Received: from mout.web.de ([212.227.15.14]:59726 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758912AbdEWQXx (ORCPT ); Tue, 23 May 2017 12:23:53 -0400 In-Reply-To: <1495548133.2093.58.camel@perches.com> Content-Language: en-GB Sender: netdev-owner@vger.kernel.org List-ID: > tipc_subseq_alloc does a kcalloc (memset to 0), > half of which is immediately overwritten. > > In other words, don't just blindly remove stuff, > understand what it does and improve it. Do you suggest another specific source code transformation pattern here? Regards, Markus