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

Reply via email to