A couple of comments:

mapPBEParamsToAlgorithm should be a static method.

Instead of catching the IOException and falling back, couldn't you instead look at the algorithm OID (on line 328 & 1954) and determine if it is PBE or PBES2 (and pass that in as an additional parameter)?

--Sean

On 02/11/2013 02:09 PM, Vincent Ryan wrote:
Please review this fix to correct the ASN.1 DER decoding of PBES2
algorithm parameters in PKCS12 keystore.

Webrev: http://cr.openjdk.java.net/~vinnie/8007934/webrev.00/

Thanks.

Reply via email to