[Cache] Make sure we get the correct number of values from redis::mget()

thePanz · nicolas-grekas · commit 685c36c3d269 · 2019-11-27T19:56:02.000+01:00
diff --git a/src/Symfony/Component/Cache/Traits/RedisTrait.php b/src/Symfony/Component/Cache/Traits/RedisTrait.php
@@ -179,7 +179,13 @@ protected function doFetch(array $ids)
                 }
             });
         } else {
-            $values = array_combine($ids, $this->redis->mget($ids));
+            $values = $this->redis->mget($ids);
+
+            if (!\is_array($values) || \count($values) !== \count($ids)) {
+                return [];
+            }
+
+            $values = array_combine($ids, $values);
         }
 
         foreach ($values as $id => $v) {

Original file line number	Diff line number	Diff line change
`@@ -179,7 +179,13 @@ protected function doFetch(array $ids)`
`179`	`179`	`}`
`180`	`180`	`});`
`181`	`181`	`} else {`
`182`		`- $values = array_combine($ids, $this->redis->mget($ids));`
	`182`	`+ $values = $this->redis->mget($ids);`
	`183`	`+`
	`184`	`+ if (!\is_array($values) \|\| \count($values) !== \count($ids)) {`
	`185`	`+ return [];`
	`186`	`+ }`
	`187`	`+`
	`188`	`+ $values = array_combine($ids, $values);`
`183`	`189`	`}`
`184`	`190`
`185`	`191`	`foreach ($values as $id => $v) {`