diff options
| author | Pravin Kamble <iampbkamble@gmail.com> | 2025-12-09 11:57:52 +0530 |
|---|---|---|
| committer | Jacob Walls <jacobtylerwalls@gmail.com> | 2025-12-11 12:38:04 -0500 |
| commit | dae08cf55b83caef5e8ee39b16417692e8565278 (patch) | |
| tree | 92a4b3b3b82c59e19195cb1ee9767adac0267be7 /django/core | |
| parent | 37eb8909699d261b9b879e6f04dbfeec75b5e080 (diff) | |
Fixed #36769 -- Avoided visiting deeply nested nodes in XML deserializer.
Only children at one level of depth need to be visited.
Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
Diffstat (limited to 'django/core')
| -rw-r--r-- | django/core/serializers/xml_serializer.py | 28 |
1 files changed, 9 insertions, 19 deletions
diff --git a/django/core/serializers/xml_serializer.py b/django/core/serializers/xml_serializer.py index 910c489a37..a9fe3cf01d 100644 --- a/django/core/serializers/xml_serializer.py +++ b/django/core/serializers/xml_serializer.py @@ -416,27 +416,17 @@ class Deserializer(base.Deserializer): def getInnerText(node): - """Get all the inner text of a DOM node (recursively).""" - inner_text_list = getInnerTextList(node) - return "".join(inner_text_list) - - -def getInnerTextList(node): - """Return a list of the inner texts of a DOM node (recursively).""" + """Get the inner text of a DOM node and any children one level deep.""" # inspired by # https://mail.python.org/pipermail/xml-sig/2005-March/011022.html - result = [] - for child in node.childNodes: - if ( - child.nodeType == child.TEXT_NODE - or child.nodeType == child.CDATA_SECTION_NODE - ): - result.append(child.data) - elif child.nodeType == child.ELEMENT_NODE: - result.extend(getInnerTextList(child)) - else: - pass - return result + return "".join( + [ + element.data + for child in node.childNodes + for element in (child, *child.childNodes) + if element.nodeType in (element.TEXT_NODE, element.CDATA_SECTION_NODE) + ] + ) # Below code based on Christian Heimes' defusedxml |
