e107inc/e107

Avoid too many return statements within this method.

Open

        return PASSWORD_VALID;

Avoid too many return statements within this method.

Open

                return PASSWORD_INVALID;

Avoid too many return statements within this method.

Open

        if ($stored_hash != $pwHash) return PASSWORD_INVALID;

Returning type true but CheckPassword() is declared to return string

Open

        return PASSWORD_VALID;

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but CheckPassword() is declared to return string

Open

        if ($stored_hash != $pwHash) return PASSWORD_INVALID;

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but CheckPassword() is declared to return string

Open

                    return PASSWORD_INVALID;

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but CheckPassword() is declared to return string

Open

                return PASSWORD_INVALID;

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but CheckPassword() is declared to return string

Open

                    return PASSWORD_INVALID;

• Exclude checks
  • Disable engine
  • Disable check

Variable $setting is undeclared

Open

        $output = substr($setting, 0, 12);        // Identifier, shift count and salt - total 12 chars

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but CheckPassword() is declared to return string

Open

                if (strlen($stored_hash) != 40) return PASSWORD_INVALID;

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but CheckPassword() is declared to return string

Open

                if (strlen($stored_hash) != 34) return PASSWORD_INVALID;

• Exclude checks
  • Disable engine
  • Disable check