On 6/20/2014 3:30 PM, Valerie Peng wrote:
>> Webrev is updated at:
>http://cr.openjdk.java.net/~valeriep/8043406/webrev.01
Thanks Valerie. Good to see the security providers granting only the
permissions it requires.
Looks okay to me.
> Sure, I will file a bug after Mandy's confirmation.
Yes please file a bug and it does look like a bug in the loadLibrary.
I'll look into it. It'd be helpful if you can include a stack trace in
the JBS issue without granting FilePermission to access all files.
Thanks
Mandy
> Thanks,
> Valerie
>> On 6/20/2014 8:46 AM, Sean Mullan wrote:
>> 36 // Needed by Runtime.loadLibrary(String) call
>> 37 permission java.io.FilePermission "<<ALL FILES>>", "read";
>>>> It seems like this is due to a bug in Runtime.loadLibrary, since you
>> have already granted the provider the permission to load the library.
>> I think possibly the call to ClassLoader.loadLibrary should be inside
>> a doPrivileged. The workaround is ok for now, but can you file a
>> separate bug for this?
>>>> --Sean
>>>> On 06/18/2014 06:51 PM, Valerie Peng wrote:
>>> Sean,
>>>>>> Not sure if you can get to reviewing this before your vacation.
>>> If not, I will find someone else to help...
>>>>>> Webrev: http://cr.openjdk.java.net/~valeriep/8043406/webrev.00/>>>>>> Thanks,
>>> Valerie