On Sun, Jan 16, 2011 at 03:38:47PM +0100, Jesper Juhl wrote: > There's a small memory leak in > arch/x86/crypto/aesni-intel_glue.c::rfc4106_set_hash_subkey(). If the call > to kmalloc() fails and returns NULL then the memory allocated previously > by ablkcipher_request_alloc() is not freed when we leave the function. > > I could have just added a call to ablkcipher_request_free() before we > return -ENOMEM, but that started to look too much like the code we > already had at the end of the function, so I chose instead to rework the > code a bit so that there are now a few labels at the end that we goto when > various allocations fail, so we don't have to repeat the same blocks of > code (this also reduces the object code size slightly). > > Please review and consider merging. > > Signed-off-by: Jesper Juhl <jj@xxxxxxxxxxxxx> Patch applied. Thanks a lot! -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
