You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@santuario.apache.org by mu...@apache.org on 2015/06/18 14:04:23 UTC
svn commit: r1686198 -
/santuario/xml-security-java/trunk/src/main/java/org/apache/xml/security/utils/IgnoreAllErrorHandler.java
Author: mullan
Date: Thu Jun 18 12:04:23 2015
New Revision: 1686198
URL: http://svn.apache.org/r1686198
Log:
Wraps calls to System.getProperty in doPrivileged.
Modified:
santuario/xml-security-java/trunk/src/main/java/org/apache/xml/security/utils/IgnoreAllErrorHandler.java
Modified: santuario/xml-security-java/trunk/src/main/java/org/apache/xml/security/utils/IgnoreAllErrorHandler.java
URL: http://svn.apache.org/viewvc/santuario/xml-security-java/trunk/src/main/java/org/apache/xml/security/utils/IgnoreAllErrorHandler.java?rev=1686198&r1=1686197&r2=1686198&view=diff
==============================================================================
--- santuario/xml-security-java/trunk/src/main/java/org/apache/xml/security/utils/IgnoreAllErrorHandler.java (original)
+++ santuario/xml-security-java/trunk/src/main/java/org/apache/xml/security/utils/IgnoreAllErrorHandler.java Thu Jun 18 12:04:23 2015
@@ -30,19 +30,30 @@ import org.xml.sax.SAXParseException;
*/
public class IgnoreAllErrorHandler implements ErrorHandler {
- private static org.slf4j.Logger log =
+ private static final org.slf4j.Logger log =
org.slf4j.LoggerFactory.getLogger(IgnoreAllErrorHandler.class);
/** Field throwExceptions */
private static final boolean warnOnExceptions =
- System.getProperty("org.apache.xml.security.test.warn.on.exceptions", "false").equals("true");
+ getProperty("org.apache.xml.security.test.warn.on.exceptions");
/** Field throwExceptions */
private static final boolean throwExceptions =
- System.getProperty("org.apache.xml.security.test.throw.exceptions", "false").equals("true");
+ getProperty("org.apache.xml.security.test.throw.exceptions");
+ private static boolean getProperty(final String name) {
+ return java.security.AccessController.doPrivileged(
+ new java.security.PrivilegedAction<Boolean>() {
+
+ @Override
+ public Boolean run() {
+ return Boolean.getBoolean(name);
+ }
+ });
+ }
/** @inheritDoc */
+ @Override
public void warning(SAXParseException ex) throws SAXException {
if (IgnoreAllErrorHandler.warnOnExceptions) {
log.warn("", ex);
@@ -54,6 +65,7 @@ public class IgnoreAllErrorHandler imple
/** @inheritDoc */
+ @Override
public void error(SAXParseException ex) throws SAXException {
if (IgnoreAllErrorHandler.warnOnExceptions) {
log.error("", ex);
@@ -65,6 +77,7 @@ public class IgnoreAllErrorHandler imple
/** @inheritDoc */
+ @Override
public void fatalError(SAXParseException ex) throws SAXException {
if (IgnoreAllErrorHandler.warnOnExceptions) {
log.warn("", ex);