On Fri, Apr 21, 2017 at 06:35:07PM +0200, Stephan Müller wrote: > > After checking again, IMHO that is no unreleated cleanup or even a cleanup at > all. > > void *private used to be struct crypto_aead and is now struct aead_tfm. struct > crypto_aead is found in private->aead. Hence, the patch assigned private to > tfm and then obtained the struct crypto_aead pointer. As this was not > necessary before, it is a required extension IMHO. Fair enough. But what about the change in aead_sock_destruct? Can you explain why it is no longer possible to obtain the tfm from ctx->aead_req? Thanks, -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt