db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kathey Marsden (JIRA)" <j...@apache.org>
Subject [jira] Updated: (DERBY-4350) javaException is created but not thrown or logged in DRDAConnThread.closeStream(InputStream stream)
Date Mon, 14 Feb 2011 22:22:57 GMT

     [ https://issues.apache.org/jira/browse/DERBY-4350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Kathey Marsden updated DERBY-4350:
----------------------------------

    Issue & fix info: [Newcomer]
             Urgency: Normal
              Labels: derby_triage10_8  (was: )

Triage for 10.8.  In the comments I indicated that it should be fixed in context with DERBY-1191,
but no need for it to be tied to that fix.  It can be fixed independently.


> javaException is created but not thrown or logged  in DRDAConnThread.closeStream(InputStream
stream)
> ----------------------------------------------------------------------------------------------------
>
>                 Key: DERBY-4350
>                 URL: https://issues.apache.org/jira/browse/DERBY-4350
>             Project: Derby
>          Issue Type: Bug
>          Components: Network Server
>    Affects Versions: 10.5.3.1, 10.6.1.0
>            Reporter: Kathey Marsden
>            Priority: Minor
>              Labels: derby_triage10_8
>
> I noticed this code in DRDAConnThread.closesStream(InputStream stream)
>    private static void closeStream(InputStream stream){
> 	
> 	try{
> 	    if (stream != null)
> 		stream.close();
> 	    
> 	} catch (IOException e) {
> 	    Util.javaException(e);
> 	    
> 	}
> 	
>     }
>     
> I think the javaException won't be thrown or logged and the underlying IOException will
be lost.  I have not investigated whether it should be thrown or just logged, but should be
addressed with the DERBY-1191 fix.   

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message