GitHub Advisoryadvisory · PyPI
GHSA-8423-8fgw-73vq tornado: multipart split() creates huge temp list before max_parts check -> memory amplification DoS (httputil.py:34)
MEDIUM
Official page ↗
- Ecosystem
- PyPI
- Packages
- tornado
- Severity
- Moderate
- CVSS vector
- CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:L/SC:N/SI:N/SA:N
- Weaknesses
- CWE-770
- Reviewed
- 2026-09-01
Affected packages
• PyPI tornado: < 6.5.8 → fixed in 6.5.8
Description
Description
Summary
`parse_multipart_form_data` (httputil.py:34) calls
`data.split(b"--"+boundary+b"\r\n")` before the `max_parts` check (:35).
A 600KB body with 100k parts creates a 100k-element transient list first,
then rejects transient memory amplification (each split element is a copy).
Pre-auth HTTP DoS.
Root cause
```python
parts = data[:final_boundary_index].split(b"--" + boundary + b"\r\n") # :34 huge list first
if len(parts) > config.max_parts: # :35 check after
raise HTTPInputError("multipart/form-data has too many parts")
```
PoC
gist: https://gist.github.com/afldl/649861f25d39b53b7edbe0298e171617
`poc.py` + `output.txt` (100k parts from 600KB transient list).
Fix
Count separators without materializing the list (e.g. `data.count(b"--"+boundary)` first).
Credit
Reported by afldl, 2026-07.
Related
Weakness · CWE (1)
- CWE-770 Allocation of Resources Without Limits or Throttling weakness
References
- https://github.com/tornadoweb/tornado/security/advisories/GHSA-8423-8fgw-73vq web
- https://github.com/tornadoweb/tornado/pull/3704 web
- https://github.com/tornadoweb/tornado/commit/de85b3f87446e323e881bbaa3d5a74f4b76e5f05 web
- https://gist.github.com/afldl/649861f25d39b53b7edbe0298e171617 web
- https://github.com/tornadoweb/tornado package
- https://github.com/tornadoweb/tornado/releases/tag/v6.5.8 web