From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH v2] gpio/omap: fix possible memory leak in omap2_gpio_dev_init() Date: Fri, 07 Sep 2012 15:37:39 -0700 Message-ID: <87har9ihn0.fsf@deeprootsystems.com> References: <871uidjxmr.fsf@deeprootsystems.com> <20120907222416.GR1303@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pb0-f46.google.com ([209.85.160.46]:62143 "EHLO mail-pb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757683Ab2IGWhn (ORCPT ); Fri, 7 Sep 2012 18:37:43 -0400 Received: by pbbrr13 with SMTP id rr13so257284pbb.19 for ; Fri, 07 Sep 2012 15:37:43 -0700 (PDT) In-Reply-To: <20120907222416.GR1303@atomide.com> (Tony Lindgren's message of "Fri, 7 Sep 2012 15:24:17 -0700") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Tony Lindgren Cc: Wei Yongjun , linux@arm.linux.org.uk, paul@pwsan.com, yongjun_wei@trendmicro.com.cn, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org Tony Lindgren writes: > * Kevin Hilman [120907 15:07]: >> Wei Yongjun writes: >> >> > From: Wei Yongjun >> > >> > pdata and pdata->regs have been allocated in this function and >> > should be freed before leaving it, and in the other error handling >> > cases too. >> > >> > spatch with a semantic match is used to found this problem. >> > (http://coccinelle.lip6.fr/) >> > >> > Signed-off-by: Wei Yongjun >> >> Acked-by: Kevin Hilman >> >> Tony, can you pick this one up for fixes? > > Sure, is fixes-noncritical OK for this one? Yes. Kevin