Skip to content
This repository has been archived by the owner on May 25, 2023. It is now read-only.

This opens the user to a file upload vulnerability #3514

Closed
wants to merge 1 commit into from
Closed

This opens the user to a file upload vulnerability #3514

wants to merge 1 commit into from

Conversation

lcashdol
Copy link

This package has been included in various other packages and this code included in the projects web accessible path.
It's actively being exploited in the wild. CVE-2018-9206

This package has been included in various other packages and this code included in the projects web accessible path.
It's actively being exploited in the wild.  CVE-2018-9206
@blueimp
Copy link
Owner

blueimp commented Oct 11, 2018

Thanks for your report @lcashdol

Reading the CVE report I can not yet confirm a vulnerability in this project.
If however you have a proof of concept how the default implementations provided by this project can be used for a remote code execution, please contact me via https://blueimp.net

For information about potential security vulnerabilities when misconfiguring file upload handling in relation to this project, please have a look art the following wiki page:
https://github.com/blueimp/jQuery-File-Upload/wiki/Security

A common misconception is that allowing the upload of arbitrary files by itself results in a security vulnerability.
Only in combination with a misconfiguration in how these files are handled can this result in a vulnerability.

@blueimp blueimp closed this in aeb47e5 Oct 13, 2018
@blueimp
Copy link
Owner

blueimp commented Oct 13, 2018

Thanks to @lcashdol's report, the issue could be identified as a combination of the default configuration of Apache v2.3.9+ to disable .htaccess support and the jQuery File Upload PHP implementation relying on its .htaccess file for security.

As security fix, instead of disabling the example implementation completely, only image file types are now allowed by default.

Thanks again @lcashdol!

@lcashdol
Copy link
Author

You're very welcome. :-)

@GildedHonour
Copy link

Only Apache is vulnerable?

@blueimp
Copy link
Owner

blueimp commented Oct 24, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants