Looks fine although you should update the 8224635 title to match the changeset.

Also, I assume you will file a followon bug for the original issue (JDK-8224256). I think a new bug must be filed instead of re-opening 8224256 because a changeset has already been pushed for it.

--Sean

On 5/23/19 12:08 PM, Arthur Eubanks wrote:
bug: https://bugs.openjdk.java.net/browse/JDK-8224635
webrev: http://cr.openjdk.java.net/~aeubanks/8224635/webrev.00/index.html

Test java/security/SecureClassLoader/DefineClass.java is failing after JDK-8224256
This change reverts the changes in JDK-8224256 and adds back in the test.

Reply via email to