On Wed, Jul 22, 2009 at 03:02:59PM +0800, Herbert Xu wrote: > On Tue, Jul 21, 2009 at 11:02:17AM +0200, Steffen Klassert wrote: > > > > + err = crypto_ahash_finup(ahreq); > > + if (err == -EINPROGRESS) > > + return; > > This is redundant. All completion functions must expect to be > called with EINPROGRESS followed by the actual completion later.
I copied this from crypto_authenc_encrypt_done(), so this is probaply redundant too. If so, I'd send a patch to remove the already existing checks too and put the updated ahash convertion patch on top of it. -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
