On Thu, Aug 02, 2007 at 01:02:40AM +0200, Sebastian Siewior wrote: > > I noticed this, after my code did not work properly. The reason was, > that my private ctx was not retrieved with > crypto_ablkcipher_ctx_aligned() (attached) but with > crypto_ablkcipher_ctx() (and it was not properly aligned anymore). > My fault right?
Yeah if your ctx needs to be aligned you need to use crypto_tfm_ctx_aligned. > Shouldn't the behavior of crypto_blkcipher_ctx_aligned() (which is > currently unused) be the default one? Most of our existing algorithms don't care about alignment. > --- a/include/crypto/algapi.h > +++ b/include/crypto/algapi.h > @@ -160,6 +160,11 @@ static inline void *crypto_ablkcipher_ct > return crypto_tfm_ctx(&tfm->base); > } > > +static inline void *crypto_ablkcipher_ctx_aligned(struct crypto_ablkcipher > *tfm) > +{ > + return crypto_tfm_ctx_aligned(&tfm->base); > +} > + > static inline struct crypto_blkcipher *crypto_spawn_blkcipher( > struct crypto_spawn *spawn) Looks good. Could you add a description and sign-off? Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]> 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html