Fix to Issue # 190

karansanwal · karansanwal · commit 7ac1c8415156 · 2014-10-28T00:43:49.000+05:30
diff --git a/src/main/java/org/owasp/esapi/reference/DefaultHTTPUtilities.java b/src/main/java/org/owasp/esapi/reference/DefaultHTTPUtilities.java
@@ -811,7 +811,7 @@ public void sendForward( String location )  throws AccessControlException,Servle
     public void sendRedirect(HttpServletResponse response, String location) throws AccessControlException, IOException {
         if (!ESAPI.validator().isValidRedirectLocation("Redirect", location, false)) {
             logger.fatal(Logger.SECURITY_FAILURE, "Bad redirect location: " + location);
-            throw new IOException("Redirect failed");
+            throw new AccessControlException("Redirect failed");
         }
         response.sendRedirect(location);
     }

Original file line number	Diff line number	Diff line change
`@@ -811,7 +811,7 @@ public void sendForward( String location ) throws AccessControlException,Servle`
`811`	`811`	`public void sendRedirect(HttpServletResponse response, String location) throws AccessControlException, IOException {`
`812`	`812`	`if (!ESAPI.validator().isValidRedirectLocation("Redirect", location, false)) {`
`813`	`813`	`logger.fatal(Logger.SECURITY_FAILURE, "Bad redirect location: " + location);`
`814`		`- throw new IOException("Redirect failed");`
	`814`	`+ throw new AccessControlException("Redirect failed");`
`815`	`815`	`}`
`816`	`816`	`response.sendRedirect(location);`
`817`	`817`	`}`