[9] RFR 8154191: Deprivilege java.smartcardio module

Sean Mullan sean.mullan at oracle.com
Wed Jun 15 11:33:14 UTC 2016


Looks fine.

--Sean

On 06/14/2016 08:12 PM, Valerie Peng wrote:
> Re-ran passed.
> Webrev updated at:
> http://cr.openjdk.java.net/~valeriep/8154191/jdk/webrev.01/
> Regards,
> Valerie
>
> On 6/14/2016 4:05 PM, Valerie Peng wrote:
>> That sounds better. I will update the test and re-run it.
>> Thanks,
>> Valerie
>>
>> On 6/14/2016 2:28 PM, Sean Mullan wrote:
>>> I don't think you need to make changes to
>>> test/javax/smartcardio/policy. Instead you can change the tests that
>>> use this policy file to use the new jtreg @run
>>> /java.security.policy=policy which extends the default system policy.
>>>
>>> --Sean
>>>
>>>
>>> On 06/13/2016 08:10 PM, Valerie Peng wrote:
>>>> Sean,
>>>>
>>>> Can you please review the changes for deprivileging the
>>>> java.smartcardio
>>>> module?
>>>> I have to update one makefile in the top-level workspace besides the
>>>> java.policy file in the jdk workspace.
>>>> One java.smartcardio regression test uses custom policy file which
>>>> needs
>>>> to be updated as well.
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8154191
>>>> Webrevs:
>>>> top-level=> http://cr.openjdk.java.net/~valeriep/8154191/webrev.00/
>>>> jdk=> http://cr.openjdk.java.net/~valeriep/8154191/jdk/webrev.00/
>>>>
>>>> Thanks,
>>>> Valerie



More information about the security-dev mailing list