You are viewing a plain text version of this content. The canonical link for it is here.
Posted to surefire-commits@maven.apache.org by br...@apache.org on 2006/03/31 06:25:27 UTC
svn commit: r390323 -
/maven/surefire/branches/surefire-testng/surefire-api/src/main/java/org/apache/maven/surefire/report/XMLReporter.java
Author: brett
Date: Thu Mar 30 20:25:25 2006
New Revision: 390323
URL: http://svn.apache.org/viewcvs?rev=390323&view=rev
Log:
merge r390296 from trunk
Modified:
maven/surefire/branches/surefire-testng/surefire-api/src/main/java/org/apache/maven/surefire/report/XMLReporter.java
Modified: maven/surefire/branches/surefire-testng/surefire-api/src/main/java/org/apache/maven/surefire/report/XMLReporter.java
URL: http://svn.apache.org/viewcvs/maven/surefire/branches/surefire-testng/surefire-api/src/main/java/org/apache/maven/surefire/report/XMLReporter.java?rev=390323&r1=390322&r2=390323&view=diff
==============================================================================
--- maven/surefire/branches/surefire-testng/surefire-api/src/main/java/org/apache/maven/surefire/report/XMLReporter.java (original)
+++ maven/surefire/branches/surefire-testng/surefire-api/src/main/java/org/apache/maven/surefire/report/XMLReporter.java Thu Mar 30 20:25:25 2006
@@ -191,7 +191,7 @@
if ( message != null && message.trim().length() > 0 )
{
- element.setAttribute( "message", message );
+ element.setAttribute( "message", escapeAttribute( message ));
element.setAttribute( "type", stackTrace.substring( 0, stackTrace.indexOf( ":" ) ) );
}
@@ -257,10 +257,15 @@
property.setAttribute( "name", key );
- property.setAttribute( "value", value );
+ property.setAttribute( "value", escapeAttribute( value ) );
}
}
}
+ private static String escapeAttribute( String attribute )
+ {
+ // Shouldn't Xpp3Dom do this itself?
+ return attribute.replaceAll( "<", "<" ).replaceAll( ">", ">" );
+ }
}