...
The normalization form KC (NFKC) is the most suitable for performing input validation because the input is transformed into an equivalent canonical form that can be safely compared with the required form.
Noncompliant Code Example
This noncompliant code example validates the String
before performing the normalization. Consequently, an attacker can get past the validation logic because the angle brackets being checked for have alternative unicode representations that need to be normalized before any validation can be performed.
Code Block | ||
---|---|---|
| ||
// String s may be user controllable // \uFE64 is normalized to < and \uFE64 is normalized to > using KFKC String s = "\uFE64" + "script" + "\uFE65"; //validate Pattern pattern = Pattern.compile("[<>]"); // check for angle brackets Matcher matcher = pattern.matcher(s); if(matcher.find()) { System.out.println("found black listed tag"); } else { // ... } // normalize s = Normalizer.normalize(s, Form.NFKC); |
Compliant Solution
This compliant solution normalizes the string before validating it. Alternative representations of the string are normalized to the canonical angle brackets. Input validation succeeds and an IllegalStateException
results.
Code Block | ||
---|---|---|
| ||
String s = "\uFE64" + "script" + "\uFE65"; // normalize s = Normalizer.normalize(s, Form.NFKC); //validate Pattern pattern = Pattern.compile("[<>]"); Matcher matcher = pattern.matcher(s); if(matcher.find()) { System.out.println("found black listed tag"); throw new IllegalStateException(); } else { // ... } |
Risk Assessment
Validating input before normalization can allow attackers to bypass filters and other security mechanisms. This can result in the execution of arbitrary code.
Rule | Severity | Likelihood | Remediation Cost | Priority | Level |
---|---|---|---|---|---|
MSC41-J | high | probable | medium | P12 | L1 |
Automated Detection
TODO
Related Vulnerabilities
Search for vulnerabilities resulting from the violation of this rule on the CERT website.
References
Wiki Markup |
---|
\[[API 06|AA. Java References#API 06]\] \[[Unicode 08|AA. Java References#Unicode 08]\] \[[Weber 09|AA. Java References#Weber 09]\] \[[MITRE 09|AA. Java References#MITRE 09]\] [CWE ID 289|http://cwe.mitre.org/data/definitions/289.html] "Authentication Bypass by Alternate Name" and [CWE ID 180|http://cwe.mitre.org/data/definitions/289.html] "Incorrect Behavior Order: Validate Before Canonicalize" |
...