Summary: | Wasted work in ErrorReportValve.getPartialServletStackTrace | ||
---|---|---|---|
Product: | Tomcat 7 | Reporter: | Adrian Nistor <nistor1> |
Component: | Catalina | Assignee: | Tomcat Developers Mailing List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | nistor1 |
Priority: | P2 | Keywords: | PatchAvailable |
Version: | 7.0.41 | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Linux | ||
Attachments: |
patch
patchShort |
Description
Adrian Nistor
2013-06-29 19:12:03 UTC
Created attachment 30500 [details]
patchShort
(In reply to Adrian Nistor from comment #0) > The above fix (in patch.diff) is certainly correct (it's easy to see > through code inspection), but I think we can have an even shorter > patch (one line, in patchShort.diff): just break as soon as "pos" is > set, without reversion the loop order. patchShort.diff is correct > only if there can be only one "elements[i]" with class name > "org.apache.catalina.core.ApplicationFilterChain" and method name > "internalDoFilter" or if it doesn't matter which such "elements[i]" is > detected (the last, like in the original code, or the first, like in > patchShort.diff). We need the last element and not the first one. So the first patch is the proper one. Regards Violeta Thanks for the report. The first patch was applied in trunk and 7.0.x and will be included in 7.0.42 onwards. |