Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753276AbbGIBlO (ORCPT ); Wed, 8 Jul 2015 21:41:14 -0400 Received: from kirsty.vergenet.net ([202.4.237.240]:38015 "EHLO kirsty.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752511AbbGIBlF (ORCPT ); Wed, 8 Jul 2015 21:41:05 -0400 Date: Thu, 9 Jul 2015 10:41:02 +0900 From: Simon Horman To: SF Markus Elfring Cc: "David S. Miller" , Jozsef Kadlecsik , Julian Anastasov , Pablo Neira Ayuso , Patrick McHardy , Wensong Zhang , netdev@vger.kernel.org, lvs-devel@vger.kernel.org, netfilter-devel@vger.kernel.org, coreteam@netfilter.org, LKML , "kernel-janitors@vger.kernel.org" , Julia Lawall Subject: Re: [PATCH] net-ipvs: Delete an unnecessary check before the function call "module_put" Message-ID: <20150709014102.GB14843@verge.net.au> References: <530CD2C4.4050903@users.sourceforge.net> <530CF8FF.8080600@users.sourceforge.net> <530DD06F.4090703@users.sourceforge.net> <5317A59D.4@users.sourceforge.net> <546BA253.6030100@users.sourceforge.net> <20141119134040.GA12270@salvia> <55955471.9000501@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <55955471.9000501@users.sourceforge.net> Organisation: Horms Solutions Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 652 Lines: 15 On Thu, Jul 02, 2015 at 05:10:41PM +0200, SF Markus Elfring wrote: > From: Markus Elfring > Date: Thu, 2 Jul 2015 17:00:14 +0200 > > The module_put() function tests whether its argument is NULL and then > returns immediately. Thus the test around the call is not needed. > > This issue was detected by using the Coccinelle software. Thanks, applied to ipvs-next for v4.3. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/