You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by tr...@apache.org on 2008/01/16 06:03:54 UTC

svn commit: r612355 - /mina/trunk/core/src/main/java/org/apache/mina/common/AbstractPollingIoProcessor.java

Author: trustin
Date: Tue Jan 15 21:03:53 2008
New Revision: 612355

URL: http://svn.apache.org/viewvc?rev=612355&view=rev
Log:
Better error handling in AbstractIoProcessor.addNew

Modified:
    mina/trunk/core/src/main/java/org/apache/mina/common/AbstractPollingIoProcessor.java

Modified: mina/trunk/core/src/main/java/org/apache/mina/common/AbstractPollingIoProcessor.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/common/AbstractPollingIoProcessor.java?rev=612355&r1=612354&r2=612355&view=diff
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/common/AbstractPollingIoProcessor.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/common/AbstractPollingIoProcessor.java Tue Jan 15 21:03:53 2008
@@ -266,7 +266,7 @@
             // in AbstractIoFilterChain.fireSessionOpened().
             ((AbstractIoService) session.getService()).getListeners().fireSessionCreated(session);
             notified = true;
-        } catch (Exception e) {
+        } catch (Throwable e) {
             if (notified) {
                 // Clear the DefaultIoFilterChain.CONNECT_FUTURE attribute
                 // and call ConnectFuture.setException().
@@ -279,6 +279,8 @@
                     destroy(session);
                 } catch (Exception e1) {
                     ExceptionMonitor.getInstance().exceptionCaught(e1);
+                } finally {
+                    registered = false;
                 }
             }
         }