From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen Gang Subject: Re: [Suggestion] fs/ext3: memory leak by calling set_qf_name or clear_qf_name, many times. Date: Tue, 08 Jan 2013 11:13:30 +0800 Message-ID: <50EB8EDA.3020709@asianux.com> References: <50DA857B.1050808@asianux.com> <20121231120621.GD7564@quack.suse.cz> <50E66B19.5060907@asianux.com> <20130107195425.GE13659@quack.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Theodore Ts'o , akpm@linux-foundation.org, linux-ext4@vger.kernel.org To: Jan Kara Return-path: Received: from intranet.asianux.com ([58.214.24.6]:28532 "EHLO intranet.asianux.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750969Ab3AHDMi (ORCPT ); Mon, 7 Jan 2013 22:12:38 -0500 In-Reply-To: <20130107195425.GE13659@quack.suse.cz> Sender: linux-ext4-owner@vger.kernel.org List-ID: =E4=BA=8E 2013=E5=B9=B401=E6=9C=8808=E6=97=A5 03:54, Jan Kara =E5=86=99= =E9=81=93: > On Fri 04-01-13 13:39:37, Chen Gang wrote: >> if what I guess is true >> it seems we have to duplicate memory for old_opts in ext3_remoun= t. >> and for clear_qf_name, set_qf_name, ext3_remount: >> use normal memory allocation and free ways instead of current = using. > Yes. Updated patch attached. >=20 at least for me, Jan Kara's patch is ok. thanks. welcome another members (especially Theodore Ts'o) to giving confirma= tion or completion. also welcome to checking this suggestion whether valid for fs/ext4. --=20 Chen Gang Asianux Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html