> 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. 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. 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 > >
