You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by ng...@apache.org on 2008/07/13 12:12:10 UTC
svn commit: r676293 -
/mina/trunk/core/src/main/java/org/apache/mina/core/DefaultExceptionMonitor.java
Author: ngn
Date: Sun Jul 13 03:12:10 2008
New Revision: 676293
URL: http://svn.apache.org/viewvc?rev=676293&view=rev
Log:
Making sure the DefaultExceptionMonitor does not catch and swallow Errors, such as OOM
Modified:
mina/trunk/core/src/main/java/org/apache/mina/core/DefaultExceptionMonitor.java
Modified: mina/trunk/core/src/main/java/org/apache/mina/core/DefaultExceptionMonitor.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/core/DefaultExceptionMonitor.java?rev=676293&r1=676292&r2=676293&view=diff
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/core/DefaultExceptionMonitor.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/core/DefaultExceptionMonitor.java Sun Jul 13 03:12:10 2008
@@ -39,7 +39,9 @@
@Override
public void exceptionCaught(Throwable cause) {
- if (log.isWarnEnabled()) {
+ if (cause instanceof Error) {
+ throw (Error) cause;
+ } else {
log.warn("Unexpected exception.", cause);
}
}