minor #34159 [Security] Fix merge (chalasr)

chalasr · chalasr · commit 913c485efda1 · 2019-10-28T23:51:27.000+01:00
This PR was merged into the 4.4 branch. Discussion ---------- [Security] Fix merge | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - Commits ------- fd0ed42 [Security] Fix merge
diff --git a/src/Symfony/Component/Security/Core/Encoder/NativePasswordEncoder.php b/src/Symfony/Component/Security/Core/Encoder/NativePasswordEncoder.php
@@ -48,7 +48,7 @@ public function __construct(int $opsLimit = null, int $memLimit = null, int $cos
             throw new \InvalidArgumentException('$cost must be in the range of 4-31.');
         }
 
-        $this->algo = (string) ($algo ?? \defined('PASSWORD_ARGON2ID') ? PASSWORD_ARGON2ID : (\defined('PASSWORD_ARGON2I') ? PASSWORD_ARGON2I : PASSWORD_BCRYPT));
+        $this->algo = (string) ($algo ?? (\defined('PASSWORD_ARGON2ID') ? PASSWORD_ARGON2ID : (\defined('PASSWORD_ARGON2I') ? PASSWORD_ARGON2I : PASSWORD_BCRYPT)));
         $this->options = [
             'cost' => $cost,
             'time_cost' => $opsLimit,

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public function __construct(int $opsLimit = null, int $memLimit = null, int $cos`
`48`	`48`	`throw new \InvalidArgumentException('$cost must be in the range of 4-31.');`
`49`	`49`	`}`
`50`	`50`
`51`		`- $this->algo = (string) ($algo ?? \defined('PASSWORD_ARGON2ID') ? PASSWORD_ARGON2ID : (\defined('PASSWORD_ARGON2I') ? PASSWORD_ARGON2I : PASSWORD_BCRYPT));`
	`51`	`+ $this->algo = (string) ($algo ?? (\defined('PASSWORD_ARGON2ID') ? PASSWORD_ARGON2ID : (\defined('PASSWORD_ARGON2I') ? PASSWORD_ARGON2I : PASSWORD_BCRYPT)));`
`52`	`52`	`$this->options = [`
`53`	`53`	`'cost' => $cost,`
`54`	`54`	`'time_cost' => $opsLimit,`