diff --git a/ql/src/Security/CWE-601/BadRedirectCheck.qhelp b/ql/src/Security/CWE-601/BadRedirectCheck.qhelp index 54ca1ca78fa..b6036f4399b 100644 --- a/ql/src/Security/CWE-601/BadRedirectCheck.qhelp +++ b/ql/src/Security/CWE-601/BadRedirectCheck.qhelp @@ -14,14 +14,14 @@ also check the second character of redirect URLs.

-Also disallow the patterns //* and /\* when checking redirect URLs. +Also disallow redirect URLs starting with // or /\.

The following function validates a (presumably untrusted) redirect URL redir. If it -does not begin with /, the harmless placeholder redirect URL, / is +does not begin with /, the harmless placeholder redirect URL / is returned to prevent an open redirect; otherwise redir itself is returned.

diff --git a/ql/src/Security/CWE-601/BadRedirectCheck.ql b/ql/src/Security/CWE-601/BadRedirectCheck.ql index c3365ffba0d..4d464ab8467 100644 --- a/ql/src/Security/CWE-601/BadRedirectCheck.ql +++ b/ql/src/Security/CWE-601/BadRedirectCheck.ql @@ -1,7 +1,7 @@ /** * @name Bad redirect check * @description A redirect check that checks for a leading slash but not two - * leading slashes or a leading slash then backslash is + * leading slashes or a leading slash followed by a backslash is * incomplete. * @kind problem * @problem.severity warning