From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liam Girdwood Date: Wed, 05 Mar 2014 19:18:29 +0000 Subject: Re: [alsa-devel] [patch] ASoC: Baytrail: fix error handling in sst_byt_dsp_init() Message-Id: <1394047109.2314.29.camel@loki> List-Id: References: <20140305111157.GD16926@elgon.mountain> In-Reply-To: <20140305111157.GD16926@elgon.mountain> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter Cc: Liam Girdwood , alsa-devel@alsa-project.org, Takashi Iwai , kernel-janitors@vger.kernel.org, Mark Brown , Jarkko Nikula On Wed, 2014-03-05 at 14:11 +0300, Dan Carpenter wrote: > Calling "kfree(byt)" is a double free because that was allocated with > devm_kzalloc(). There were a couple places which leak "byt->msg". That > memory is allocated in msg_empty_list_init(). > Thanks, we've just fixed this too but were not quicker than you for upstreaming ;) Acked-by: Liam Girdwood