Skip to content
This repository was archived by the owner on Apr 11, 2025. It is now read-only.
This repository was archived by the owner on Apr 11, 2025. It is now read-only.

[SECURITY] phpsec/user.php - Passwords converted to lower case before hashing #118

@AndrewCarterUK

Description

@AndrewCarterUK

Note: I have been asked to log all the security issues that I have found by an OWASP contributor to support the decision to abandon this project. My personal opinion is that all of the code in this repository is beyond repair and that it is not wise to attempt to fix it.

This line of code gives a huge number of equivalent passwords by converting the password before hashing it. This means that in the eyes of the auth library 'paSsWord' is equal to 'password'.

return hash($algo, strtolower($dynamicSalt . $pass . BasicPasswordManagement::getStaticSalt()));

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions