Summary: | IllegalStateException in SmapStratum on empty JSP | ||
---|---|---|---|
Product: | Tomcat 9 | Reporter: | stef.an <apache> |
Component: | Jasper | Assignee: | Tomcat Developers Mailing List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | P2 | ||
Version: | 9.0.1 | ||
Target Milestone: | ----- | ||
Hardware: | PC | ||
OS: | All |
Description
stef.an
2017-10-10 14:34:37 UTC
Sorry, of course lineData is not null at that point, it is the empty array causing the exception being thrown. // print our StratumSection, FileSection, and LineSections if (fileNameList.size() == 0 || lineData.size() == 0) { throw new IllegalStateException(); } So this was done during the (major) 49176 enhancement in r1800201. Switching back to returning null will likely cause some NPEs in some cases, but this used to be the case before from what I can see. Going back to the SMAP spec, it is legal for there to be zero LineInfo lines so the lineData.size() == 0 test can be removed. I'll double-check the rest of the spec while I am at it. Thanks for the report. This has been fixed for 9.0.2 onwards. |