diff options
| author | Sarah Boyce <42296566+sarahboyce@users.noreply.github.com> | 2025-02-25 09:40:54 +0100 |
|---|---|---|
| committer | Sarah Boyce <42296566+sarahboyce@users.noreply.github.com> | 2025-03-06 10:01:44 +0100 |
| commit | e88f7376fe68dbf4ebaf11fad1513ce700b45860 (patch) | |
| tree | 1aa1a23e92b654c00d65679c555a36508393cc50 /django/utils | |
| parent | 348e46a3e0cd80718d237a3b6d96a1c84bff317d (diff) | |
[4.2.x] Fixed CVE-2025-26699 -- Mitigated potential DoS in wordwrap template filter.
Thanks sw0rd1ight for the report.
Backport of 55d89e25f4115c5674cdd9b9bcba2bb2bb6d820b from main.
Diffstat (limited to 'django/utils')
| -rw-r--r-- | django/utils/text.py | 28 |
1 files changed, 10 insertions, 18 deletions
diff --git a/django/utils/text.py b/django/utils/text.py index e1b835e0e2..81ae88dc76 100644 --- a/django/utils/text.py +++ b/django/utils/text.py @@ -1,6 +1,7 @@ import gzip import re import secrets +import textwrap import unicodedata from gzip import GzipFile from gzip import compress as gzip_compress @@ -97,24 +98,15 @@ def wrap(text, width): ``width``. """ - def _generator(): - for line in text.splitlines(True): # True keeps trailing linebreaks - max_width = min((line.endswith("\n") and width + 1 or width), width) - while len(line) > max_width: - space = line[: max_width + 1].rfind(" ") + 1 - if space == 0: - space = line.find(" ") + 1 - if space == 0: - yield line - line = "" - break - yield "%s\n" % line[: space - 1] - line = line[space:] - max_width = min((line.endswith("\n") and width + 1 or width), width) - if line: - yield line - - return "".join(_generator()) + wrapper = textwrap.TextWrapper( + width=width, + break_long_words=False, + break_on_hyphens=False, + ) + result = [] + for line in text.splitlines(True): + result.extend(wrapper.wrap(line)) + return "\n".join(result) class Truncator(SimpleLazyObject): |
