On Mon, Jun 15, 2015 at 08:36:06PM -0700, Tadeusz Struk wrote: > On 06/15/2015 08:25 PM, Herbert Xu wrote: > > The current parse_key function requires all three number to be > > present, n, e, and d, no? > > No, it will handle whatever it will find. So if a public key will be passed it will only set "n" and "e". > If a private key will be passed it will set all three "n", "e", and "d". > Then during operation I check if there is everything that's required. AFAICS the ASN1 parser will call all three functions and bomb out if any one of them fails. If you did make them all optional then you'd need to check to ensure that at least n and e are present. Also all your test vectors contain private keys. Please add at least one that contains a public key only to test this. Cheers, -- Email: Herbert Xu <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