On Fri, Apr 24, 2009 at 12:27:20PM +0200, Steffen Klassert wrote: > > + alg = crypto_get_attr_alg(tb, algt->type, > + (algt->mask & CRYPTO_ALG_TYPE_MASK) > + | CRYPTO_ALG_PCRYPT); Do you really need to exclude pcrypt algorithms here? If the user wants to added another layer of pcrypt then we should obey. Otherwise this looks pretty good to me. 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