On Mon, Feb 22, 2010 at 11:21:04AM +0800, Wang, Shane wrote: > > @@ -474,7 +475,10 @@ static u64 vmac(unsigned char m[], unsig > } > p = be64_to_cpup(out_p + i); > h = vhash(m, mbytes, (u64 *)0, &ctx->__vmac_ctx); > - return p + h; > + r = p + h; > + r = le64_to_cpup(&r); > + > + return r; Is there any reason why we don't simply do return le64_to_cpu(p + h); 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