Merge branch '3.4' into 4.4

fabpot · fabpot · commit 6dae4692ac91 · 2020-03-18T08:15:43.000+01:00
* 3.4:
  [VarDumper] fix side-effect by not using mt_rand()
diff --git a/Cloner/VarCloner.php b/Cloner/VarCloner.php
@@ -46,7 +46,7 @@ protected function doClone($var)
                                         // or null if the original value is used directly
 
         if (!$gid = self::$gid) {
-            $gid = self::$gid = uniqid(mt_rand(), true); // Unique string used to detect the special $GLOBALS variable
+            $gid = self::$gid = md5(random_bytes(6)); // Unique string used to detect the special $GLOBALS variable
         }
         $arrayStub = new Stub();
         $arrayStub->type = Stub::TYPE_ARRAY;

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ protected function doClone($var)`
`46`	`46`	`// or null if the original value is used directly`
`47`	`47`
`48`	`48`	`if (!$gid = self::$gid) {`
`49`		`- $gid = self::$gid = uniqid(mt_rand(), true); // Unique string used to detect the special $GLOBALS variable`
	`49`	`+ $gid = self::$gid = md5(random_bytes(6)); // Unique string used to detect the special $GLOBALS variable`
`50`	`50`	`}`
`51`	`51`	`$arrayStub = new Stub();`
`52`	`52`	`$arrayStub->type = Stub::TYPE_ARRAY;`