From: Jesper Juhl Subject: Re: [PATCH] rfc4106, Intel, AES-NI: Don't leak memory in rfc4106_set_hash_subkey(). Date: Thu, 10 Feb 2011 19:47:26 +0100 (CET) Message-ID: References: <4d502ccb.p+rKcuFjWCnBlXY9%tadeusz.struk@intel.com> <20110207210901.GA21941@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: tadeusz.struk@intel.com, linux-kernel@vger.kernel.org, linux-crypto@vger.kernel.org, aidan.o.mahony@intel.com, gabriele.paoloni@intel.com, adrian.hoban@intel.com To: Herbert Xu Return-path: Received: from swampdragon.chaosbits.net ([90.184.90.115]:23541 "EHLO swampdragon.chaosbits.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750917Ab1BJSsf (ORCPT ); Thu, 10 Feb 2011 13:48:35 -0500 In-Reply-To: <20110207210901.GA21941@gondor.apana.org.au> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Tue, 8 Feb 2011, Herbert Xu wrote: > On Mon, Feb 07, 2011 at 05:32:59PM +0000, tadeusz.struk@intel.com wrote: > > > > Here goes. > > > > Fix up previous patch that attempted to fix a mem leak in > > rfc4106_set_hash_subkey. The previous patch was flawed in that the 'goto > > out' would still leak. > > Sign-off? > Since it's identical to the patch I posted except for the moving of the kfree() which (IMHO) is a minor detail, I hope you can use my sign-off so we can move along and get the patch merged.. Signed-off-by: Jesper Juhl -- Jesper Juhl http://www.chaosbits.net/ Plain text mails only, please. Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html