aboutsummaryrefslogtreecommitdiffstats
path: root/phpBB/phpbb/passwords
diff options
context:
space:
mode:
Diffstat (limited to 'phpBB/phpbb/passwords')
-rw-r--r--phpBB/phpbb/passwords/driver/salted_md5.php3
-rw-r--r--phpBB/phpbb/passwords/helper.php1
-rw-r--r--phpBB/phpbb/passwords/manager.php3
3 files changed, 1 insertions, 6 deletions
diff --git a/phpBB/phpbb/passwords/driver/salted_md5.php b/phpBB/phpbb/passwords/driver/salted_md5.php
index bccf61dc07..96dd298789 100644
--- a/phpBB/phpbb/passwords/driver/salted_md5.php
+++ b/phpBB/phpbb/passwords/driver/salted_md5.php
@@ -105,7 +105,6 @@ class salted_md5 extends \phpbb\passwords\driver\base
{
return (md5($password) === $hash) ? true : false;
}
- // No need to check prefix, already did that in manage
if ($hash === $this->hash($password, $hash))
{
@@ -121,8 +120,6 @@ class salted_md5 extends \phpbb\passwords\driver\base
*/
protected function generate_salt()
{
- $salt = '';
- $random = '';
$count = 6;
$random = $this->helper->get_random_salt($count);
diff --git a/phpBB/phpbb/passwords/helper.php b/phpBB/phpbb/passwords/helper.php
index 28a13d2224..0a0bf1d6b1 100644
--- a/phpBB/phpbb/passwords/helper.php
+++ b/phpBB/phpbb/passwords/helper.php
@@ -119,7 +119,6 @@ class helper
*/
public function check_combined_hash($password, $stored_hash_type, $hash)
{
- $cur_hash = '';
$i = 0;
$data = array(
'prefix' => '$',
diff --git a/phpBB/phpbb/passwords/manager.php b/phpBB/phpbb/passwords/manager.php
index 0b41d3a8c3..dde81a9818 100644
--- a/phpBB/phpbb/passwords/manager.php
+++ b/phpBB/phpbb/passwords/manager.php
@@ -164,11 +164,10 @@ class manager
return false;
}
- $return_ary[$type] = $this->get_algorithm("\${$type}\$");
+ $return_ary[$type] = $this->get_algorithm('$' . $type . '$');
if (empty($return_ary[$type]))
{
-
return false;
}
}