tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 52730] New: Another incomplete fix for the resource leak bug in CometConnectionManagerValve.java
Date Tue, 21 Feb 2012 12:10:20 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=52730

             Bug #: 52730
           Summary: Another incomplete fix for the resource leak bug in
                    CometConnectionManagerValve.java
           Product: Tomcat 7
           Version: trunk
          Platform: PC
        OS/Version: Mac System 7
            Status: NEW
          Severity: critical
          Priority: P2
         Component: Catalina
        AssignedTo: dev@tomcat.apache.org
        ReportedBy: lianggt08@sei.pku.edu.cn
    Classification: Unclassified


The fix revision 730178 was aimed to remove an resource leak bug on the
FileInputStream object "fis " (created in line 451), the Socket object "socket"
(created in line 474) and the OutputStream object "stream" (created in line
475) in the method "stopServer()" of the file
"/tomcat/trunk/java/org/apache/catalina/valves/CometConnectionManagerValve.java"
, but it is incomplete. 

There are some problems: 
1. when the statements at lines 452-454 throw some exception, the "fis" can not
be closed as expected. 
2. when the statements at lines 475-480 throw some exception, the "socket" and
the "stream" can not be closed as expected. 

The best way to close such resource objects is putting such close operations in
the finaly block of a try-catch-finally structure.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message