Changeset 48322 for trunk/src/wp-includes/IXR/class-IXR-message.php

Timestamp:

07/05/2020 03:20:16 PM (4 years ago)

Author:

SergeyBiryukov

Message:

XML-RPC: Explicitly unset the reference to the parser resource after calling xml_parser_free() to avoid memory leaks in PHP 7.0.0 or higher.

Props procifer, ayeshrajans.
Fixes #49700.

File:

• trunk/src/wp-includes/IXR/class-IXR-message.php (modified) (1 diff)

trunk/src/wp-includes/IXR/class-IXR-message.php

@@ -111 +111 @@
 
         $final = false;
+
         do {
             if (strlen($this->message) <= $chunk_size) {
                 $final = true;
             }
+
             $part = substr($this->message, 0, $chunk_size);
             $this->message = substr($this->message, $chunk_size);
+
             if (!xml_parse($this->_parser, $part, $final)) {
+                xml_parser_free($this->_parser);
+                unset($this->_parser);
                 return false;
             }
+
             if ($final) {
                 break;
             }
         } while (true);
+
         xml_parser_free($this->_parser);
+        unset($this->_parser);
 
         // Grab the error messages, if any