You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by se...@apache.org on 2011/03/11 11:02:41 UTC
svn commit: r1080502 -
/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractServletController.java
Author: sergeyb
Date: Fri Mar 11 10:02:41 2011
New Revision: 1080502
URL: http://svn.apache.org/viewvc?rev=1080502&view=rev
Log:
Reducing the synchronization scope in AbstractServletController.updateDest
Modified:
cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractServletController.java
Modified: cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractServletController.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractServletController.java?rev=1080502&r1=1080501&r2=1080502&view=diff
==============================================================================
--- cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractServletController.java (original)
+++ cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractServletController.java Fri Mar 11 10:02:41 2011
@@ -120,10 +120,12 @@ public abstract class AbstractServletCon
request.setAttribute("org.apache.cxf.transport.endpoint.address",
base + path);
} else {
- d2.getEndpointInfo().setAddress(base + path);
- if (WSDLLibrary.isAvailable()
- && d2.getEndpointInfo().getExtensor(AddressType.class) != null) {
- d2.getEndpointInfo().getExtensor(AddressType.class).setLocation(base + path);
+ synchronized (d2) {
+ d2.getEndpointInfo().setAddress(base + path);
+ if (WSDLLibrary.isAvailable()
+ && d2.getEndpointInfo().getExtensor(AddressType.class) != null) {
+ d2.getEndpointInfo().getExtensor(AddressType.class).setLocation(base + path);
+ }
}
}
}