Author: fschumacher
Date: Thu Nov 3 18:22:56 2016
New Revision: 1767946
URL: http://svn.apache.org/viewvc?rev=1767946&view=rev
Log:
Check that threadPriority values used in AbstractProtocol are valid.
Followup to r1767328
Modified:
tomcat/trunk/java/org/apache/coyote/AbstractProtocol.java
tomcat/trunk/webapps/docs/changelog.xml
Modified: tomcat/trunk/java/org/apache/coyote/AbstractProtocol.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/AbstractProtocol.java?rev=1767946&r1=1767945&r2=1767946&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/AbstractProtocol.java (original)
+++ tomcat/trunk/java/org/apache/coyote/AbstractProtocol.java Thu Nov 3
18:22:56 2016
@@ -554,7 +554,11 @@ public abstract class AbstractProtocol<S
// Start async timeout thread
asyncTimeout = new AsyncTimeout();
Thread timeoutThread = new Thread(asyncTimeout, getNameInternal() +
"-AsyncTimeout");
- timeoutThread.setPriority(endpoint.getThreadPriority());
+ int priority = endpoint.getThreadPriority();
+ if (priority < Thread.MIN_PRIORITY || priority > Thread.MAX_PRIORITY) {
+ priority = Thread.NORM_PRIORITY;
+ }
+ timeoutThread.setPriority(priority);
timeoutThread.setDaemon(true);
timeoutThread.start();
}
Modified: tomcat/trunk/webapps/docs/changelog.xml
URL:
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1767946&r1=1767945&r2=1767946&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Thu Nov 3 18:22:56 2016
@@ -45,6 +45,14 @@
issues do not "pop up" wrt. others).
-->
<section name="Tomcat 9.0.0.M13" rtext="in development">
+ <subsection name="Coyote">
+ <changelog>
+ <fix>
+ Check that threadPriority values used in AbstractProtocol are valid.
+ (fschumacher)
+ </fix>
+ </changelog>
+ </subsection>
</section>
<section name="Tomcat 9.0.0.M12" rtext="release in progress">
<subsection name="Catalina">
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
