Skip to content
Snippets Groups Projects
Commit 2bdfaf8d authored by Uma's avatar Uma
Browse files

Fixes cleaning up text data for vulnerable code

parent 1c70c7bb
No related branches found
No related tags found
No related merge requests found
......@@ -715,12 +715,16 @@ function vtlib_purify($input, $ignore=false) {
* @return <String>
*/
function purifyHtmlEventAttributes($value,$replaceAll = false){
$htmlEventAttributes = "onerror|onblur|onchange|oncontextmenu|onfocus|oninput|oninvalid|".
"onreset|onsearch|onselect|onsubmit|onkeydown|onkeypress|onkeyup|".
"onclick|ondblclick|ondrag|ondragend|ondragenter|ondragleave|ondragover|".
"ondragstart|ondrop|onmousedown|onmousemove|onmouseout|onmouseover|".
"onmouseup|onmousewheel|onscroll|onwheel|oncopy|oncut|onpaste|onload|".
"onselectionchange|onabort|onselectstart|onstart|onfinish|onloadstart|onshow";
$htmlEventAttributes = "onerror|onblur|onchange|oncontextmenu|onfocus|oninput|oninvalid|onresize|onauxclick|oncancel|oncanplay|oncanplaythrough|".
"onreset|onsearch|onselect|onsubmit|onkeydown|onkeypress|onkeyup|onclose|oncuechange|ondurationchange|onemptied|onended|".
"onclick|ondblclick|ondrag|ondragend|ondragenter|ondragleave|ondragover|ondragexit|onformdata|onloadeddata|onloadedmetadata|".
"ondragstart|ondrop|onmousedown|onmousemove|onmouseout|onmouseover|onmouseenter|onmouseleave|onpause|onplay|onplaying|".
"onmouseup|onmousewheel|onscroll|onwheel|oncopy|oncut|onpaste|onload|onprogress|onratechange|onsecuritypolicyviolation|".
"onselectionchange|onabort|onselectstart|onstart|onfinish|onloadstart|onshow|onreadystatechange|onseeked|onslotchange|".
"onseeking|onstalled|onsubmit|onsuspend|ontimeupdate|ontoggle|onvolumechange|onwaiting|onwebkitanimationend|onstorage|".
"onwebkitanimationiteration|onwebkitanimationstart|onwebkittransitionend|onafterprint|onbeforeprint|onbeforeunload|".
"onhashchange|onlanguagechange|onmessage|onmessageerror|onoffline|ononline|onpagehide|onpageshow|onpopstate|onunload".
"onrejectionhandled|onunhandledrejection|onloadend";
// remove malicious html attributes with its value.
if ($replaceAll) {
......@@ -728,7 +732,7 @@ function purifyHtmlEventAttributes($value,$replaceAll = false){
$value = preg_replace("/\s*(" . $htmlEventAttributes . ")" . $regex, '', $value);
/**
* #131224556::if anchor tag having 'javascript:' string then remove the tag contents.
* If anchor tag having 'javascript:' string then remove the tag contents.
* Right now, we fixed this for anchor tag as we don't see any other such things right now.
* All other event attributes are already handled above. Need to update this if any thing new found
*/
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment