Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CVE-2021-23814 Unrestricted Upload of File with Dangerous Type #1113

Closed
bashgeek opened this issue Feb 1, 2022 · 4 comments
Closed

CVE-2021-23814 Unrestricted Upload of File with Dangerous Type #1113

bashgeek opened this issue Feb 1, 2022 · 4 comments
Labels
security Exploits, attacks, dangerous leaks.

Comments

@bashgeek
Copy link

bashgeek commented Feb 1, 2022

GHSA-f8x6-m9f5-ffp8

Surprisingly can't find anything about this in the issues here yet, is this something that's gonna be addressed soon?

Thanks a lot!

@MaxKorlaar
Copy link

Yeah, I noticed it before: Roave/SecurityAdvisories#89 but it's not linked to the repo apparently.

@streamtw streamtw added the security Exploits, attacks, dangerous leaks. label Mar 16, 2022
@streamtw streamtw added the WIP Working in progress. label May 15, 2022
@streamtw
Copy link
Member

v2.5.1 has been released to fix this issue.

@deanzod
Copy link

deanzod commented Jun 24, 2023

I am still able to do this in 2.6. It needs to check file extensions too. Something like....

function hasAllowedExtension($allowed_extensions)
    {
        $extension = $this->file->getClientOriginalExtension();

        if (!in_array($extension, $allowed_extensions)) {
            throw new ExcutableFileException($extension);
        }

        return $this;
    }

@streamtw
Copy link
Member

streamtw commented Nov 15, 2023

Thanks @deanzod . v2.6.2 has been released to check file extension.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security Exploits, attacks, dangerous leaks.
Projects
None yet
Development

No branches or pull requests

4 participants