Coordinated Disclosure Timeline
- 2021-08-31: Followed the instructions in their security policy and entered the details into this form.
- 2021-08-31: Received automated email from notifications@erpnext.com after completing the form. Issue is tracked as
VUL-283348
. - 2021-11-30: Deadline expired.
- 2021-12-06: Posted a pull request to fix the bug: https://github.com/frappe/frappe/pull/15201
Summary
Frappe contains a regular expression that is vulnerable to ReDoS (Regular Expression Denial of Service).
Product
Frappe
Tested Version
Details
ReDoS
ReDoS, or Regular Expression Denial of Service, is a vulnerability affecting inefficient regular expressions which can perform extremely badly when run on a crafted input string.
This vulnerability was found using a CodeQL query which identifies inefficient regular expressions.
Vulnerability
The vulnerable regular expression is here.
To see that the regular expression is vulnerable, copy-paste it into a separate file as shown below:
- Run the code below with
python3
:
import re
re.match(r"^[\w][\w\'\-]*([ \w][\w\'\-]+)*$", "a'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0'0}")
Impact
This issue may lead to a denial of service.
Credit
This issue was discovered by GitHub team members @erik-krogh (Erik Krogh Kristensen) and @yoff (Rasmus Petersen).
Contact
You can contact the GHSL team at securitylab@github.com
, please include a reference to GHSL-2021-122
in any communication regarding this issue.