diff options
author | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2022-06-22 12:44:04 +0200 |
---|---|---|
committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2022-07-04 08:26:57 +0200 |
commit | 0dc9c016fadb71a067e5a42be30164e3f96c0492 (patch) | |
tree | ca672f9aad4b7a05802d27c5a22bede0ddd1c419 | |
parent | a2b88d7be6fc9b443794518d2f75fd2f3d9e677a (diff) | |
download | django-0dc9c016fadb71a067e5a42be30164e3f96c0492.tar.gz |
[4.0.x] Fixed CVE-2022-34265 -- Protected Trunc(kind)/Extract(lookup_name) against SQL injection.
Thanks Takuto Yoshikai (Aeye Security Lab) for the report.
-rw-r--r-- | django/db/backends/base/operations.py | 3 | ||||
-rw-r--r-- | django/db/models/functions/datetime.py | 4 | ||||
-rw-r--r-- | docs/releases/3.2.14.txt | 11 | ||||
-rw-r--r-- | docs/releases/4.0.6.txt | 12 | ||||
-rw-r--r-- | tests/db_functions/datetime/test_extract_trunc.py | 34 |
5 files changed, 61 insertions, 3 deletions
diff --git a/django/db/backends/base/operations.py b/django/db/backends/base/operations.py index 322a89c17a..ac14b0556f 100644 --- a/django/db/backends/base/operations.py +++ b/django/db/backends/base/operations.py @@ -9,6 +9,7 @@ from django.db import NotSupportedError, transaction from django.db.backends import utils from django.utils import timezone from django.utils.encoding import force_str +from django.utils.regex_helper import _lazy_re_compile class BaseDatabaseOperations: @@ -54,6 +55,8 @@ class BaseDatabaseOperations: # Prefix for EXPLAIN queries, or None EXPLAIN isn't supported. explain_prefix = None + extract_trunc_lookup_pattern = _lazy_re_compile(r"[\w\-_()]+") + def __init__(self, connection): self.connection = connection self._cache = None diff --git a/django/db/models/functions/datetime.py b/django/db/models/functions/datetime.py index 2d6ec7089e..5f98e6bba1 100644 --- a/django/db/models/functions/datetime.py +++ b/django/db/models/functions/datetime.py @@ -51,6 +51,8 @@ class Extract(TimezoneMixin, Transform): super().__init__(expression, **extra) def as_sql(self, compiler, connection): + if not connection.ops.extract_trunc_lookup_pattern.fullmatch(self.lookup_name): + raise ValueError("Invalid lookup_name: %s" % self.lookup_name) sql, params = compiler.compile(self.lhs) lhs_output_field = self.lhs.output_field if isinstance(lhs_output_field, DateTimeField): @@ -235,6 +237,8 @@ class TruncBase(TimezoneMixin, Transform): super().__init__(expression, output_field=output_field, **extra) def as_sql(self, compiler, connection): + if not connection.ops.extract_trunc_lookup_pattern.fullmatch(self.kind): + raise ValueError("Invalid kind: %s" % self.kind) inner_sql, inner_params = compiler.compile(self.lhs) tzname = None if isinstance(self.lhs.output_field, DateTimeField): diff --git a/docs/releases/3.2.14.txt b/docs/releases/3.2.14.txt index bde1777376..f5719798c1 100644 --- a/docs/releases/3.2.14.txt +++ b/docs/releases/3.2.14.txt @@ -5,3 +5,14 @@ Django 3.2.14 release notes *July 4, 2022* Django 3.2.14 fixes a security issue with severity "high" in 3.2.13. + +CVE-2022-34265: Potential SQL injection via ``Trunc(kind)`` and ``Extract(lookup_name)`` arguments +================================================================================================== + +:class:`Trunc() <django.db.models.functions.Trunc>` and +:class:`Extract() <django.db.models.functions.Extract>` database functions were +subject to SQL injection if untrusted data was used as a +``kind``/``lookup_name`` value. + +Applications that constrain the lookup name and kind choice to a known safe +list are unaffected. diff --git a/docs/releases/4.0.6.txt b/docs/releases/4.0.6.txt index ee7979161e..d1b6fc6a43 100644 --- a/docs/releases/4.0.6.txt +++ b/docs/releases/4.0.6.txt @@ -6,7 +6,13 @@ Django 4.0.6 release notes Django 4.0.6 fixes a security issue with severity "high" in 4.0.5. -Bugfixes -======== +CVE-2022-34265: Potential SQL injection via ``Trunc(kind)`` and ``Extract(lookup_name)`` arguments +================================================================================================== -* ... +:class:`Trunc() <django.db.models.functions.Trunc>` and +:class:`Extract() <django.db.models.functions.Extract>` database functions were +subject to SQL injection if untrusted data was used as a +``kind``/``lookup_name`` value. + +Applications that constrain the lookup name and kind choice to a known safe +list are unaffected. diff --git a/tests/db_functions/datetime/test_extract_trunc.py b/tests/db_functions/datetime/test_extract_trunc.py index 7d94b39fb3..f4b4a6d6f1 100644 --- a/tests/db_functions/datetime/test_extract_trunc.py +++ b/tests/db_functions/datetime/test_extract_trunc.py @@ -235,6 +235,23 @@ class DateFunctionTests(TestCase): self.assertEqual(qs.count(), 1) self.assertGreaterEqual(str(qs.query).lower().count("extract"), 2) + def test_extract_lookup_name_sql_injection(self): + start_datetime = datetime(2015, 6, 15, 14, 30, 50, 321) + end_datetime = datetime(2016, 6, 15, 14, 10, 50, 123) + if settings.USE_TZ: + start_datetime = timezone.make_aware(start_datetime) + end_datetime = timezone.make_aware(end_datetime) + self.create_model(start_datetime, end_datetime) + self.create_model(end_datetime, start_datetime) + + msg = "Invalid lookup_name: " + with self.assertRaisesMessage(ValueError, msg): + DTModel.objects.filter( + start_datetime__year=Extract( + "start_datetime", "day' FROM start_datetime)) OR 1=1;--" + ) + ).exists() + def test_extract_func(self): start_datetime = datetime(2015, 6, 15, 14, 30, 50, 321) end_datetime = datetime(2016, 6, 15, 14, 10, 50, 123) @@ -899,6 +916,23 @@ class DateFunctionTests(TestCase): 2, ) + def test_trunc_lookup_name_sql_injection(self): + start_datetime = datetime(2015, 6, 15, 14, 30, 50, 321) + end_datetime = datetime(2016, 6, 15, 14, 10, 50, 123) + if settings.USE_TZ: + start_datetime = timezone.make_aware(start_datetime) + end_datetime = timezone.make_aware(end_datetime) + self.create_model(start_datetime, end_datetime) + self.create_model(end_datetime, start_datetime) + msg = "Invalid kind: " + with self.assertRaisesMessage(ValueError, msg): + DTModel.objects.filter( + start_datetime__date=Trunc( + "start_datetime", + "year', start_datetime)) OR 1=1;--", + ) + ).exists() + def test_trunc_func(self): start_datetime = datetime(2015, 6, 15, 14, 30, 50, 321) end_datetime = datetime(2016, 6, 15, 14, 10, 50, 123) |