On Fri, Feb 19, 2010 at 01:21:41AM +0100, Richard Hartmann wrote: > > Signed-off-by: Richard Hartmann <richih.mailinglist@xxxxxxxxx> > --- > crypto/khazad.c | 71 +++++++++++++++++++++++++++---------------------------- > 1 files changed, 35 insertions(+), 36 deletions(-) > > @@ -810,25 +810,25 @@ static void khazad_crypt(const u64 roundKey[KHAZAD_ROUNDS + 1], > state = be64_to_cpu(*src) ^ roundKey[0]; > > for (r = 1; r < KHAZAD_ROUNDS; r++) { > - state = T0[(int)(state >> 56) ] ^ > - T1[(int)(state >> 48) & 0xff] ^ > - T2[(int)(state >> 40) & 0xff] ^ > - T3[(int)(state >> 32) & 0xff] ^ > - T4[(int)(state >> 24) & 0xff] ^ > - T5[(int)(state >> 16) & 0xff] ^ > - T6[(int)(state >> 8) & 0xff] ^ > - T7[(int)(state ) & 0xff] ^ > + state = T0[(int)(state >> 56)] ^ > + T1[(int)(state >> 48)&0xff] ^ > + T2[(int)(state >> 40)&0xff] ^ > + T3[(int)(state >> 32)&0xff] ^ > + T4[(int)(state >> 24)&0xff] ^ > + T5[(int)(state >> 16)&0xff] ^ > + T6[(int)(state >> 8)&0xff] ^ > + T7[(int)(state )&0xff] ^ > roundKey[r]; > - } > + } This indentation change is bogus. > @@ -865,7 +865,6 @@ static struct crypto_alg khazad_alg = { > static int __init khazad_mod_init(void) > { > int ret = 0; > - > ret = crypto_register_alg(&khazad_alg); > return ret; > } What's this for? Please review the all the rest of the patches for similar errors and resubmit. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <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