Attachment 30559 Details for Bug 55207 – patch to reject on any element inside <jsp:text>

[patch] patch to reject on any element inside <jsp:text>

55207.patch (text/plain), 1.31 KB, created by Jeremy Boynes on 2013-07-07 01:59:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Boynes

Created: 2013-07-07 01:59:38 UTC

Size: 1.31 KB

patch

obsolete

>Index: test/webapp/WEB-INF/tags/bug55198.tagx
>===================================================================
>--- test/webapp/WEB-INF/tags/bug55198.tagx	(revision 1500346)
>+++ test/webapp/WEB-INF/tags/bug55198.tagx	(working copy)
>@@ -17,9 +17,7 @@
> -->
> <jsp:root xmlns:jsp="http://java.sun.com/JSP/Page" version="2.0">
> <jsp:directive.tag body-content="scriptless" />
>-<jsp:text>
>   <a href="#" onclick="window.alert(&quot;${'foo'}&quot;)">foo</a>
>   <a href="#" onclick="window.alert(&quot;bar&quot;)">bar</a>
>-</jsp:text>
> <jsp:doBody />
> </jsp:root>
>Index: java/org/apache/jasper/compiler/JspDocumentParser.java
>===================================================================
>--- java/org/apache/jasper/compiler/JspDocumentParser.java	(revision 1500346)
>+++ java/org/apache/jasper/compiler/JspDocumentParser.java	(working copy)
>@@ -270,11 +270,8 @@
>             return;
>         }
> 
>-        String currentPrefix = getPrefix(current.getQName());
>-
>         // jsp:text must not have any subelements
>-        if (JSP_URI.equals(uri) && TEXT_ACTION.equals(current.getLocalName())
>-                && "jsp".equals(currentPrefix)) {
>+        if (current instanceof Node.JspText) {
>             throw new SAXParseException(
>                 Localizer.getMessage("jsp.error.text.has_subelement"),
>                 locator);

Actions: View | Diff

Attachments on bug 55207: 30557 | 30558 | 30559