Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Message-ID: <1033246955.1255871971262.JavaMail.jira@brutus>
Date: Sun, 18 Oct 2009 06:19:31 -0700 (PDT)
From: "Subhajit DasGupta (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Subject: [jira] Created: (SANDBOX-314) ContinuationClassLoader does not
 close output stream in getClassFromStream(InputStream,String)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

ContinuationClassLoader does not close output stream in getClassFromStream(InputStream,String)
----------------------------------------------------------------------------------------------

                 Key: SANDBOX-314
                 URL: https://issues.apache.org/jira/browse/SANDBOX-314
             Project: Commons Sandbox
          Issue Type: Bug
          Components: Javaflow
         Environment: This is a problem with the way the getClassFromStream method is coded. The problem is independent of the environment.
            Reporter: Subhajit DasGupta
            Priority: Minor


The getClassFromStream(InputStream,String) method creates a ByteArrayOutputStream but fails to close it after it finishes using it. This could lead to memory leaks in long running programs using the "getClassFromStream" method. I am submitting a patch below. This code opens and uses the ByteArrayIOutputStream object "baos" inside a "try", and closes baos in a "finally" block.

Index: ContinuationClassLoader.java
===================================================================
--- ContinuationClassLoader.java	(revision 826428)
+++ ContinuationClassLoader.java	(working copy)
@@ -350,7 +350,9 @@
      */
     private Class getClassFromStream(InputStream stream, String classname)
                 throws IOException, SecurityException {
-        ByteArrayOutputStream baos = new ByteArrayOutputStream();
+        ByteArrayOutputStream baos = null;
+        try{
+        baos = new ByteArrayOutputStream();
         int bytesRead;
         byte[] buffer = new byte[BUFFER_SIZE];
 
@@ -360,6 +362,11 @@
 
         byte[] classData = baos.toByteArray();
         return defineClassFromData(classData, classname);
+        }finally{
+            if ( baos != null ){
+                baos.close();
+            }
+        }
     }
 
     /**


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.