You are viewing a plain text version of this content. The canonical link for it is here.
Posted to xmlrpc-auto@ws.apache.org by jo...@apache.org on 2008/11/28 21:46:35 UTC
svn commit: r721558 - in /webservices/xmlrpc/trunk:
common/src/main/java/org/apache/xmlrpc/util/ThreadPool.java pom.xml
src/changes/changes.xml
Author: jochen
Date: Fri Nov 28 12:46:35 2008
New Revision: 721558
URL: http://svn.apache.org/viewvc?rev=721558&view=rev
Log:
Eliminated a possible race condition in the ThreadPool.
PR: XMLRPC-162
Submitted-By: Mark Gertsvolf <ma...@nortel.com>
Modified:
webservices/xmlrpc/trunk/common/src/main/java/org/apache/xmlrpc/util/ThreadPool.java
webservices/xmlrpc/trunk/pom.xml
webservices/xmlrpc/trunk/src/changes/changes.xml
Modified: webservices/xmlrpc/trunk/common/src/main/java/org/apache/xmlrpc/util/ThreadPool.java
URL: http://svn.apache.org/viewvc/webservices/xmlrpc/trunk/common/src/main/java/org/apache/xmlrpc/util/ThreadPool.java?rev=721558&r1=721557&r2=721558&view=diff
==============================================================================
--- webservices/xmlrpc/trunk/common/src/main/java/org/apache/xmlrpc/util/ThreadPool.java (original)
+++ webservices/xmlrpc/trunk/common/src/main/java/org/apache/xmlrpc/util/ThreadPool.java Fri Nov 28 12:46:35 2008
@@ -56,7 +56,9 @@
if (t == null) {
try {
synchronized (this) {
- wait();
+ if (!isShuttingDown() && getTask() == null) {
+ wait();
+ }
}
} catch (InterruptedException e) {
// Do nothing
Modified: webservices/xmlrpc/trunk/pom.xml
URL: http://svn.apache.org/viewvc/webservices/xmlrpc/trunk/pom.xml?rev=721558&r1=721557&r2=721558&view=diff
==============================================================================
--- webservices/xmlrpc/trunk/pom.xml (original)
+++ webservices/xmlrpc/trunk/pom.xml Fri Nov 28 12:46:35 2008
@@ -168,6 +168,10 @@
<email>landonf@threerings.net</email>
</contributor>
<contributor>
+ <name>Mark Gertsvolf</name>
+ <email>markg@nortel.com</email>
+ </contributor>
+ <contributor>
<name>Catalin Hritcu</name>
<email>Catalin.Hritcu@gmail.com</email>
</contributor>
Modified: webservices/xmlrpc/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/webservices/xmlrpc/trunk/src/changes/changes.xml?rev=721558&r1=721557&r2=721558&view=diff
==============================================================================
--- webservices/xmlrpc/trunk/src/changes/changes.xml (original)
+++ webservices/xmlrpc/trunk/src/changes/changes.xml Fri Nov 28 12:46:35 2008
@@ -23,7 +23,14 @@
<title>Changes in Apache XML-RPC</title>
</properties>
<body>
- <release version="3.1.1-SNAPSHOT" date="Not yet released">
+ <release version="3.1.2" date="Not yet released">
+ <action dev="jochen" type="fix" due-to="Mark Gertsvolf" due-to-email="markg@nortel.com"
+ issue="XMLRPC-162">
+ Eliminated a possible race condition in the ThreadPool.
+ </action>
+ </release>
+
+ <release version="3.1.1" date="2008-Aug-10">
<action dev="jochen" type="fix" due-to="Póka Balázs" due-to-email="p.balazs@gmail.com">
The XmlRpcCommonsTransport is now always declaring its entities as repeatable.
</action>