Bug 40625 - CGIServlet eats root cause
CGIServlet eats root cause
Status: RESOLVED FIXED
Product: Tomcat 5
Classification: Unclassified
Component: Servlets:CGI
5.0.20
Other other
: P2 minor (vote)
: ---
Assigned To: Tomcat Developers Mailing List
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2006-09-28 05:16 UTC by Takayoshi Kimura
Modified: 2006-09-29 16:45 UTC (History)
0 users



Attachments
Proposed patch against /tomcat/container/tc5.5.x (581 bytes, patch)
2006-09-28 05:18 UTC, Takayoshi Kimura
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Takayoshi Kimura 2006-09-28 05:16:58 UTC
The code below doesn't make sense:
<snip>
            catch (IOException e){
                log ("Caught exception " + e);
                throw new IOException (e.toString());
            }
</snip>

It should be "throw e;".

IMHO, It seems no need to log this exception here, so 
the cache block can be removed.
Comment 1 Takayoshi Kimura 2006-09-28 05:18:33 UTC
Created attachment 18925 [details]
Proposed patch against /tomcat/container/tc5.5.x
Comment 2 Takayoshi Kimura 2006-09-28 05:21:38 UTC
> IMHO, It seems no need to log this exception here, so 
> the cache block can be removed.

Ooops, s/cache/catch/ :-P
Comment 3 Mark Thomas 2006-09-29 16:45:30 UTC
Many thanks for the report and patch. This has been fixed in SVN and will be in
5.5.21 onwards.