Author: markt
Date: Sat Nov 17 00:01:35 2012
New Revision: 1410636
URL: http://svn.apache.org/viewvc?rev=1410636&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=54144
Add required behaviour for Reader objects when used with c:out
Modified:
tomcat/tc7.0.x/trunk/ (props changed)
tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/Util.java
tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
Propchange: tomcat/tc7.0.x/trunk/
------------------------------------------------------------------------------
Merged /tomcat/trunk:r1410632
Modified: tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/Util.java
URL:
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/Util.java?rev=1410636&r1=1410635&r2=1410636&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/Util.java
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/Util.java Sat
Nov 17 00:01:35 2012
@@ -178,11 +178,18 @@ public class Util {
*
* taken from org.apache.taglibs.standard.tag.common.core.Util
*/
- @SuppressWarnings("null") // escapedBuffer cannot be null
public static String escapeXml(String buffer) {
+ String result = escapeXml(buffer.toCharArray(), buffer.length());
+ if (result == null) {
+ return buffer;
+ } else {
+ return result;
+ }
+ }
+
+ @SuppressWarnings("null") // escapedBuffer cannot be null
+ public static String escapeXml(char[] arrayBuffer, int length) {
int start = 0;
- int length = buffer.length();
- char[] arrayBuffer = buffer.toCharArray();
StringBuilder escapedBuffer = null;
for (int i = 0; i < length; i++) {
@@ -206,7 +213,7 @@ public class Util {
}
// no xml escaping was necessary
if (start == 0) {
- return buffer;
+ return null;
}
// add rest of unescaped portion
if (start < length) {
Modified:
tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java
URL:
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java?rev=1410636&r1=1410635&r2=1410636&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java
Sat Nov 17 00:01:35 2012
@@ -19,11 +19,13 @@
package org.apache.jasper.tagplugins.jstl.core;
import java.io.IOException;
+import java.io.Reader;
import javax.servlet.jsp.JspWriter;
import org.apache.jasper.compiler.tagplugin.TagPlugin;
import org.apache.jasper.compiler.tagplugin.TagPluginContext;
+import org.apache.jasper.tagplugins.jstl.Util;
public final class Out implements TagPlugin {
@@ -39,24 +41,26 @@ public final class Out implements TagPlu
//strValName, strEscapeXmlName & strDefName are two variables' name
//standing for value, escapeXml and default attribute
+ String strObjectName = ctxt.getTemporaryVariableName();
String strValName = ctxt.getTemporaryVariableName();
String strDefName = ctxt.getTemporaryVariableName();
String strEscapeXmlName = ctxt.getTemporaryVariableName();
String strSkipBodyName = ctxt.getTemporaryVariableName();
//according to the tag file, the value attribute is mandatory.
- ctxt.generateJavaSource("String " + strValName + " = null;");
- ctxt.generateJavaSource("if(");
+ ctxt.generateJavaSource("Object " + strObjectName + "=");
ctxt.generateAttribute("value");
- ctxt.generateJavaSource("!=null){");
- ctxt.generateJavaSource(" " + strValName + " = (");
- ctxt.generateAttribute("value");
- ctxt.generateJavaSource(").toString();");
+ ctxt.generateJavaSource(";");
+ ctxt.generateJavaSource("String " + strValName + "=null;");
+ ctxt.generateJavaSource("if(!(" + strObjectName +
+ " instanceof Reader) && "+ strObjectName + " != null){");
+ ctxt.generateJavaSource(
+ strValName + " = " + strObjectName + ".toString();");
ctxt.generateJavaSource("}");
//initiate the strDefName with null.
//if the default has been specified, then assign the value to it;
- ctxt.generateJavaSource("String " + strDefName + " = null;\n");
+ ctxt.generateJavaSource("String " + strDefName + " = null;");
if(hasDefault){
ctxt.generateJavaSource("if(");
ctxt.generateAttribute("default");
@@ -80,24 +84,45 @@ public final class Out implements TagPlu
ctxt.generateJavaSource(
"boolean " + strSkipBodyName + " = " +
"org.apache.jasper.tagplugins.jstl.core.Out.output(out, " +
- strValName + ", " + strDefName + ", " + strEscapeXmlName +
- ");");
+ strObjectName + ", " + strValName + ", " + strDefName + ", " +
+ strEscapeXmlName + ");");
ctxt.generateJavaSource("if(!" + strSkipBodyName + ") {");
ctxt.generateBody();
ctxt.generateJavaSource("}");
}
- public static boolean output(JspWriter out, String value, String
defaultValue,
- boolean escapeXml) throws IOException {
- String v = value != null ? value : defaultValue;
- if (v != null) {
- if(escapeXml){
- v = org.apache.jasper.tagplugins.jstl.Util.escapeXml(v);
+ public static boolean output(JspWriter out, Object input, String value,
+ String defaultValue, boolean escapeXml) throws IOException {
+ if (input instanceof Reader) {
+ char[] buffer = new char[8096];
+ int read = 0;
+ while (read != -1) {
+ read = ((Reader) input).read(buffer);
+ if (read != -1) {
+ if (escapeXml) {
+ String escaped = Util.escapeXml(buffer, read);
+ if (escaped == null) {
+ out.write(buffer, 0, read);
+ } else {
+ out.print(escaped);
+ }
+ } else {
+ out.write(buffer, 0, read);
+ }
+ }
}
- out.write(v);
return true;
} else {
- return false;
+ String v = value != null ? value : defaultValue;
+ if (v != null) {
+ if(escapeXml){
+ v = Util.escapeXml(v);
+ }
+ out.write(v);
+ return true;
+ } else {
+ return false;
+ }
}
}
}
Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL:
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1410636&r1=1410635&r2=1410636&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Sat Nov 17 00:01:35 2012
@@ -219,6 +219,12 @@
<bug>54017</bug>: Simplify coercion of <code>String</code> instances to
<code>Object</code>. (markt)
</scode>
+ <fix>
+ <bug>54144</bug>: Fix a bug in the tag plug-in for
+ <code><c:out></code> that meant that if the value of the tag
+ evaluated to a <code>java.io.Reader</code> object then it was not
+ correctly handled. (markt)
+ </fix>
</changelog>
</subsection>
<subsection name="Cluster">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
