On 5/3/2016 9:42 AM, Mandy Chung wrote:
On May 3, 2016, at 2:37 AM, Frank Yuan <[email protected]> wrote:
Hi Mandy and Joe
I have re-fixed the bug with your comment, would you like to check again?
http://cr.openjdk.java.net/~fyuan/8155514/webrev.01/
This patch extending the default security policy is a better fix. Looks fine.
Looks good to me too.
I saw that you already pushed the previous version to jdk9/dev. Please create
a new JBS issue for this patch since you can’t push with a duplicated bug ID.
Indeed. And also, the Bug Id in the Subject is incorrect, that's
somebody else's enhancement.
-Joe
Mandy
Thanks,
Frank
-----Original Message-----
From: Mandy Chung [mailto:[email protected]]
Sent: Friday, April 29, 2016 4:00 AM
To: Frank Yuan <[email protected]>
Cc: core-libs-dev <[email protected]>; huizhe wang
<[email protected]>; Xueming Shen
<[email protected]>
Subject: Re: RFR: 8155600: jaxp.library.TestPolicy should extend the default
security policy
Hi Frank,
The fix looks fine.
Thanks for verifying it with my patch to deprivilege jdk.charasets.
Mandy
On Apr 28, 2016, at 1:11 AM, Frank Yuan <[email protected]> wrote:
Hi Mandy, Joe and all
Would you like to review the fix for bug
https://bugs.openjdk.java.net/browse/JDK-8155514?
The webrev is at: http://cr.openjdk.java.net/~fyuan/8155514/webrev.00/.
Thanks,
Frank
