You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mina.apache.org by el...@apache.org on 2011/04/05 13:26:44 UTC

svn commit: r1088981 - /mina/branches/2.0.3/mina-core/src/main/java/org/apache/mina/core/service/SimpleIoProcessorPool.java

Author: elecharny
Date: Tue Apr  5 11:26:44 2011
New Revision: 1088981

URL: http://svn.apache.org/viewvc?rev=1088981&view=rev
Log:
Throwing a real exception instead of a NPE

Modified:
    mina/branches/2.0.3/mina-core/src/main/java/org/apache/mina/core/service/SimpleIoProcessorPool.java

Modified: mina/branches/2.0.3/mina-core/src/main/java/org/apache/mina/core/service/SimpleIoProcessorPool.java
URL: http://svn.apache.org/viewvc/mina/branches/2.0.3/mina-core/src/main/java/org/apache/mina/core/service/SimpleIoProcessorPool.java?rev=1088981&r1=1088980&r2=1088981&view=diff
==============================================================================
--- mina/branches/2.0.3/mina-core/src/main/java/org/apache/mina/core/service/SimpleIoProcessorPool.java (original)
+++ mina/branches/2.0.3/mina-core/src/main/java/org/apache/mina/core/service/SimpleIoProcessorPool.java Tue Apr  5 11:26:44 2011
@@ -326,6 +326,11 @@ public class SimpleIoProcessorPool<S ext
             }
 
             processor = pool[Math.abs((int) session.getId()) % pool.length];
+            
+            if (processor == null) {
+                throw new IllegalStateException("A disposed processor cannot be accessed.");
+            }
+            
             session.setAttributeIfAbsent(PROCESSOR, processor);
         }