On 1/11/2017 9:27 AM, Claes Redestad wrote:
Hi Adam,

On 01/11/2017 02:34 PM, Adam Petcher wrote:
Please review the following bug fix:

http://cr.openjdk.java.net/~apetcher/8168075/webrev.00/

This fixes a bug in which a permission check would try to load resources while the system class loader is being initialized. Resources cannot be loaded at this time, so this change ensures that the resources are loaded earlier.


couldn't this be done in System.setSecurityManager rather than in a static block
in SecurityManager?

http://cr.openjdk.java.net/~redestad/scratch/8168075.alt/

The provided EarlyLoad test still pass with this approach, and this would avoid loading a few classes and a resource bundle when not installing a security manager (the SecurityManager
class is always loaded on bootstrap).
That would work, but I was trying to avoid cluttering up System.java with this code. What if we put it in the SecurityManager constructor? Would that accomplish the same goal?

Thanks!

/Claes

Reply via email to