Author: trustin
Date: Wed Feb 20 03:44:50 2008
New Revision: 629429
URL: http://svn.apache.org/viewvc?rev=629429&view=rev
Log:
Fixed issue: DIRMINA-533 (NPE in OrderedThreadPoolExecutor and
UnorderedThreadPoolExecutor)
* Added null check to both executor
Modified:
mina/trunk/core/src/main/java/org/apache/mina/filter/executor/OrderedThreadPoolExecutor.java
mina/trunk/core/src/main/java/org/apache/mina/filter/executor/UnorderedThreadPoolExecutor.java
Modified:
mina/trunk/core/src/main/java/org/apache/mina/filter/executor/OrderedThreadPoolExecutor.java
URL:
http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/filter/executor/OrderedThreadPoolExecutor.java?rev=629429&r1=629428&r2=629429&view=diff
==============================================================================
---
mina/trunk/core/src/main/java/org/apache/mina/filter/executor/OrderedThreadPoolExecutor.java
(original)
+++
mina/trunk/core/src/main/java/org/apache/mina/filter/executor/OrderedThreadPoolExecutor.java
Wed Feb 20 03:44:50 2008
@@ -489,7 +489,9 @@
}
try {
- runTasks(getSessionBuffer(session));
+ if (session != null) {
+ runTasks(getSessionBuffer(session));
+ }
} finally {
idleWorkers.incrementAndGet();
}
Modified:
mina/trunk/core/src/main/java/org/apache/mina/filter/executor/UnorderedThreadPoolExecutor.java
URL:
http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/filter/executor/UnorderedThreadPoolExecutor.java?rev=629429&r1=629428&r2=629429&view=diff
==============================================================================
---
mina/trunk/core/src/main/java/org/apache/mina/filter/executor/UnorderedThreadPoolExecutor.java
(original)
+++
mina/trunk/core/src/main/java/org/apache/mina/filter/executor/UnorderedThreadPoolExecutor.java
Wed Feb 20 03:44:50 2008
@@ -430,9 +430,11 @@
break;
}
- queueHandler.polled(UnorderedThreadPoolExecutor.this,
(IoEvent) task);
try {
- runTask(task);
+ if (task != null) {
+
queueHandler.polled(UnorderedThreadPoolExecutor.this, (IoEvent) task);
+ runTask(task);
+ }
} finally {
idleWorkers.incrementAndGet();
}