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

Cross Site Scripting Vulnerability on "Import Mail" feature in PHP List #667

Closed
Songohan22 opened this issue May 26, 2020 · 2 comments
Closed

Comments

@Songohan22
Copy link

Songohan22 commented May 26, 2020

Describe the bug
An authenticated malicious user can take advantage of a Stored XSS vulnerability in the "Import Mail" feature.
To Reproduce
Steps to reproduce the behavior:

  1. Log into the panel.
  2. Go to "/admin/?page=import&tk=6adb3efa27ad1baa355bd3b62671c46b"
  3. Click "import by uploading a file with emails"
  4. Insert payload:
    <svg/on<script><script>load=alert('XSS')//</script>
  5. Click "Save "
  6. View the preview to trigger XSS.
  7. View the preview to get in request and such Stored XSS

Expected behavior
The removal of script tags is not sufficient to prevent an XSS attack. You must HTML Entity encode any output that is stored back to the page.

Impact
Commonly include transmitting private data, like cookies or other session information, to the attacker, redirecting the victim to web content controlled by the attacker, or performing other malicious operations on the user’s machine under the guise of the vulnerable site.

Screenshots
image
image
image

Desktop (please complete the following information):

  • OS: Windows
  • Browser: Firefox
  • Version: 76.0.1

PHP List Version 3.5.5

@Songohan22 Songohan22 changed the title Cross Site Scripting Vulnerability on "Import Mail" feature in Lavelite Cross Site Scripting Vulnerability on "Import Mail" feature in PHP List May 26, 2020
@Songohan22
Copy link
Author

Hi @suap
Please review it! Thanks

@michield
Copy link
Member

Resolved with 9c4515c

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants