diff options
author | Aleksander Machniak <alec@alec.pl> | 2012-12-04 09:17:08 +0100 |
---|---|---|
committer | Aleksander Machniak <alec@alec.pl> | 2012-12-04 09:17:08 +0100 |
commit | 74cd0a9b62f11bc07c5a1d3ba0098b54883eb0ba (patch) | |
tree | 005192c324221b18ed83994de18c10b14ec42eff /tests | |
parent | 0fa54df638a0b0f514d1bfba3cefb93e38991a35 (diff) |
- Fix XSS vulnerability in vbscript: and data:text links handling (#1488850)
Diffstat (limited to 'tests')
-rw-r--r-- | tests/MailFunc.php | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/tests/MailFunc.php b/tests/MailFunc.php index 967277c2a..4d4250c22 100644 --- a/tests/MailFunc.php +++ b/tests/MailFunc.php @@ -97,6 +97,20 @@ class MailFunc extends PHPUnit_Framework_TestCase } /** + * Test the elimination of some XSS vulnerabilities + */ + function test_html_xss3() + { + // #1488850 + $html = '<p><a href="data:text/html,<script>alert(document.cookie)</script>">Firefox</a>' + .'<a href="vbscript:alert(document.cookie)">Internet Explorer</a></p>'; + $washed = rcmail_wash_html($html, array('safe' => true), array()); + + $this->assertNotRegExp('/data:text/', $washed, "Remove data:text/html links"); + $this->assertNotRegExp('/vbscript:/', $washed, "Remove vbscript: links"); + } + + /** * Test washtml class on non-unicode characters (#1487813) */ function test_washtml_utf8() |