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: liangg...@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-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
