Hi Alexander,
The fix looks fine. I'd also mention the JDK bug id in the comment at
line 436 (no need for a new webrev with this change).
--
best regards,
Anthony
On 7/16/2014 8:56 PM, Alexander Zvegintsev wrote:
Hello AWT team,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/8048289/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8048289
UIManager.getSystemLookAndFeelClassName() calls
UNIXToolkit.isNativeGTKAvailable()
which loads gtk library, checks version, and closes library. Thread
specific data key is created upon gtk dlopen,
but this key is not deleted at dlclose. This produces a crash at thread
termination.
So this fix is a workaround for the glib issue [1], it simply doesn't
close library.
Simple case to reproduce this issue written on C is attached to [1].
[1] https://bugzilla.gnome.org/show_bug.cgi?id=733065