Merge branch '5.4' into 6.0

* 5.4:
  Throw exception if incompatible version of psr/simple-cache is used
  [DependencyInjection] copy synthetic status when resolving child definitions
  [HttpClient] Fix Failed to open stream: Too many open files
  [Console] use STDOUT/ERR in ConsoleOutput to save opening too many file descriptors
  Fix for "Implicit conversion from float <float_number> to int loses precision"
  Fix typo in UPGRADE-6.0.md
  [Cache] Set mtime of cache files 1 year into future if they do not expire
  [DependencyInjection] remove arbitratry limitation to exclude inline services from bindings

Author: fabpot

Internal/CurlClientState.php

@@ -23,8 +23,9 @@
  */
 final class CurlClientState extends ClientState
 {
-    public \CurlMultiHandle $handle;
-    public \CurlShareHandle $share;
+    public ?\CurlMultiHandle $handle;
+    public ?\CurlShareHandle $share;
+
     /** @var PushedResponse[] */
     public array $pushedResponses = [];
     public DnsCache $dnsCache;
@@ -64,8 +65,17 @@ public function __construct(int $maxHostConnections, int $maxPendingPushes)
             return;
         }
 
-        curl_multi_setopt($this->handle, \CURLMOPT_PUSHFUNCTION, function ($parent, $pushed, array $requestHeaders) use ($maxPendingPushes) {
-            return $this->handlePush($parent, $pushed, $requestHeaders, $maxPendingPushes);
+        // Clone to prevent a circular reference
+        $multi = clone $this;
+        $multi->handle = null;
+        $multi->share = null;
+        $multi->pushedResponses = &$this->pushedResponses;
+        $multi->logger = &$this->logger;
+        $multi->handlesActivity = &$this->handlesActivity;
+        $multi->openHandles = &$this->openHandles;
+
+        curl_multi_setopt($this->handle, \CURLMOPT_PUSHFUNCTION, static function ($parent, $pushed, array $requestHeaders) use ($multi, $maxPendingPushes) {
+            return $multi->handlePush($parent, $pushed, $requestHeaders, $maxPendingPushes);
         });
     }