RFR: 8231584: Deadlock with ClassLoader.findLibrary and System.loadLibrary call
Mandy Chung
mandy.chung at oracle.com
Thu Oct 10 05:00:48 UTC 2019
On 10/9/19 9:24 AM, Anton Kozlov wrote:
> Updated webrev with fixes:
> http://cr.openjdk.java.net/~akozlov/8231584/webrev.03/
I like this version. Some minor comments:
2014 static synchronized void initLibraryPaths() { This does not need
synchronized since it's still during phase 1 before other thread can
execute java code.
LoadLibraryTest.java
- please add @bug 8231584
- there are places silently catching the checked exception
e.g. line 62, 97, 117, 140, 157. I suggest throw an unchecked
exception instead to help diagnosis in case the test fails.
121 return defineClass(null, b, 0, b.length);
- it should pass name instead of null
- in fact, TestClassLoader only needs to delegate the parent classloader
as it doesn't have any .class to find. So no need to override findClass.
Thanks
Mandy
More information about the core-libs-dev
mailing list