-
Notifications
You must be signed in to change notification settings - Fork 384
/
echoed-request.yaml
62 lines (62 loc) · 1.77 KB
/
echoed-request.yaml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
rules:
- id: echoed-request
mode: taint
message: >-
`Echo`ing user input risks cross-site scripting vulnerability.
You should use `htmlentities()` when showing data to users.
languages: [php]
severity: ERROR
pattern-sources:
- pattern: $_REQUEST
- pattern: $_GET
- pattern: $_POST
pattern-sinks:
- pattern: echo $...VARS;
pattern-sanitizers:
- pattern: htmlentities(...)
- pattern: htmlspecialchars(...)
- pattern: strip_tags(...)
- pattern: isset(...)
- pattern: empty(...)
# Wordpress Escapes
- pattern: esc_html(...)
- pattern: esc_attr(...)
- pattern: wp_kses(...)
# Laravel Escapes
- pattern: e(...)
# Symfony Escapes
- pattern: twig_escape_filter(...)
# CodeIgniter Escapes
- pattern: xss_clean(...)
- pattern: html_escape(...)
# Drupal Escapes
- pattern: Html::escape(...)
- pattern: Xss::filter(...)
# Magento Escapes
- pattern: escapeHtml(...)
# Laminas Escapes
- pattern: escapeHtml(...)
- pattern: escapeHtmlAttr(...)
fix: echo htmlentities($...VARS);
metadata:
technology:
- php
cwe:
- "CWE-79: Improper Neutralization of Input During Web Page Generation ('Cross-site Scripting')"
owasp:
- A07:2017 - Cross-Site Scripting (XSS)
- A03:2021 - Injection
category: security
references:
- https://www.php.net/manual/en/function.htmlentities.php
- https://www.php.net/manual/en/reserved.variables.request.php
- https://www.php.net/manual/en/reserved.variables.post.php
- https://www.php.net/manual/en/reserved.variables.get.php
- https://cheatsheetseries.owasp.org/cheatsheets/Cross_Site_Scripting_Prevention_Cheat_Sheet.html
cwe2022-top25: true
cwe2021-top25: true
subcategory:
- vuln
likelihood: MEDIUM
impact: MEDIUM
confidence: MEDIUM