From 77c4a04d3db3cf31b057ce34067375f5201af064 Mon Sep 17 00:00:00 2001
From: Daniel Kvist <daniel@danielkvist.net>
Date: Thu, 4 Feb 2016 23:27:31 +0100
Subject: [PATCH] Fixes #195 by adding static initialisation of the maps rather
 then using the constructor

---
 .../java/org/owasp/webgoat/util/HtmlEncoder.java    | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/webgoat-container/src/main/java/org/owasp/webgoat/util/HtmlEncoder.java b/webgoat-container/src/main/java/org/owasp/webgoat/util/HtmlEncoder.java
index 31a56a629..3b7dfa62d 100644
--- a/webgoat-container/src/main/java/org/owasp/webgoat/util/HtmlEncoder.java
+++ b/webgoat-container/src/main/java/org/owasp/webgoat/util/HtmlEncoder.java
@@ -119,14 +119,13 @@ public class HtmlEncoder
 	};
 
 	/**
-	 * <p>Constructor for HtmlEncoder.</p>
+	 * Initialises the mappings between entities and characters
 	 */
-	public HtmlEncoder()
-	{
-		for (int i = 0; i < entities.length; i++)
-			e2i.put((String) entities[i][0], (Integer) entities[i][1]);
-		for (int i = 0; i < entities.length; i++)
-			i2e.put((Integer) entities[i][1], (String) entities[i][0]);
+	static {
+	    for (int i = 0; i < entities.length; i++)
+            e2i.put((String) entities[i][0], (Integer) entities[i][1]);
+        for (int i = 0; i < entities.length; i++)
+            i2e.put((Integer) entities[i][1], (String) entities[i][0]);
 	}
 
 	/**