You should probably update the keytool doc to say that if duplicate extensions are specified, only the last one is added.

Also, unrelated to you fix but can you break up that really long line (1213) in KeyToolTest?

Looks fine otherwise.

--Sean

On 02/16/2015 02:56 AM, Weijun Wang wrote:
Please review the fix at

    http://cr.openjdk.java.net/~weijun/8073182/webrev.00/

This fix stores extensions inside CertificateExtensions using OID as key
so there will never be a type duplicate.

Existing test updated.

Thanks
Max

Reply via email to