Re: Crash due to missing synchronization on 'gconf_client' in 'jdk/src/solaris/native/sun/net/spi/DefaultProxySelector.c'

[Christian Schulte]

Am 08/02/12 17:27, schrieb Chris Hegarty:
Thanks for the cross post David. Yes, net-dev is the best place to
discuss this issue ( bcc'ing off jdk7u-dev ).
Thanks for reporting this issue Christian. I haven't looked at the
proposed patch yet, but I agree we may want to simplify this if possible
( handling the synchronization at the Java level ). We also need to add
support for gnome3 system proxies in the near future. We need to ensure
we don't further complicate that.

The attached patch moves synchronization to the Java level for all 
platforms and also solves the issue here.

Regards,

--
Christian
--- jdk/src/share/classes/sun/net/spi/DefaultProxySelector.java.orig    Fri Aug 
 3 06:05:21 2012
+++ jdk/src/share/classes/sun/net/spi/DefaultProxySelector.java Fri Aug  3 
06:05:37 2012
@@ -339,6 +339,6 @@ public class DefaultProxySelector extends ProxySelecto
         }
     }
 
-    private native static boolean init();
-    private native Proxy getSystemProxy(String protocol, String host);
+    private synchronized native static boolean init();
+    private synchronized native Proxy getSystemProxy(String protocol, String 
host);
 }