diff --git a/web/src/main/java/org/springframework/security/web/firewall/StrictHttpFirewall.java b/web/src/main/java/org/springframework/security/web/firewall/StrictHttpFirewall.java index d3049cfbc7..fc051810f2 100644 --- a/web/src/main/java/org/springframework/security/web/firewall/StrictHttpFirewall.java +++ b/web/src/main/java/org/springframework/security/web/firewall/StrictHttpFirewall.java @@ -340,7 +340,7 @@ public class StrictHttpFirewall implements HttpFirewall { return true; } - if (path.indexOf("//") > 0) { + if (path.indexOf("//") > -1) { return false; } diff --git a/web/src/test/java/org/springframework/security/web/firewall/StrictHttpFirewallTests.java b/web/src/test/java/org/springframework/security/web/firewall/StrictHttpFirewallTests.java index c9bbc654a1..5613ad4dbc 100644 --- a/web/src/test/java/org/springframework/security/web/firewall/StrictHttpFirewallTests.java +++ b/web/src/test/java/org/springframework/security/web/firewall/StrictHttpFirewallTests.java @@ -26,7 +26,8 @@ import static org.assertj.core.api.Assertions.fail; */ public class StrictHttpFirewallTests { public String[] unnormalizedPaths = { "/..", "/./path/", "/path/path/.", "/path/path//.", "./path/../path//.", - "./path", ".//path", ".", "/path//" }; + "./path", ".//path", ".", "//path", "//path/path", "//path//path", "/path//path" }; + private StrictHttpFirewall firewall = new StrictHttpFirewall();