tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject DO NOT REPLY [Bug 52800] New: Please do not catch Throwable!
Date Thu, 01 Mar 2012 00:37:17 GMT

             Bug #: 52800
           Summary: Please do not catch Throwable!
           Product: Tomcat 7
           Version: 7.0.26
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Catalina
    Classification: Unclassified

See also bug 48644.

I see "catch (Throwable ..." occurring 311 times in the Tomcat source:

$ grep -r "catch.*(.*Throwable" * | wc -l

Please at the very least, change these to "catch (Exception ...". Otherwise you
end up catching OutOfMemoryErrors.

IMHO, and I haven't grepped the mailing list for any controversy surrounding
this, when Tomcat (or anything!!!) receives an OOME, it should exit to the OS
to be restarted by whatever script or mechanism is managing it. You cannot
recover from an OOME, and I have to resort to SIGKILL to stop the JVM.

Configure bugmail:
------- You are receiving this mail because: -------
You are the assignee for the bug.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message