mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

bug 475133 - Crash [@ nsStyleContext::GetRuleNode] with <xul:progressmeter>, DOMAttrModified event 

r=enn, sr=roc
Don't forget to block scripts when manually calling nsProgressMeterFrame::AttributeChanged in progressmeter initialization to prevent evil mutation event listeners from crashing us.
This commit is contained in:
Nickolay Ponomarev 2009-02-23 17:15:31 +03:00
Родитель a094eff461
Коммит 207aca6ddf
3 изменённых файлов: 24 добавлений и 0 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

20
layout/xul/base/src/crashtests/475133.html Normal file
Просмотреть файл

@ -0,0 +1,20 @@
<!DOCTYPE html>
<html>
<head>
<script type="text/javascript">
function x()
{
document.removeEventListener("DOMAttrModified", x, false);
document.removeChild(document.documentElement);
}
function boom()
{
var p = document.createElementNS("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul", "progressmeter");
document.addEventListener("DOMAttrModified", x, false);
document.documentElement.appendChild(p);
}
</script>
</head>
<body onload="boom();"></body>
</html>

1
layout/xul/base/src/crashtests/crashtests.list
Просмотреть файл

@ -41,3 +41,4 @@ load 434458-1.xul
load 467481-1.xul
load 470063-1.html
load 472189.xul
load 475133.html

3
layout/xul/base/src/nsProgressMeterFrame.cpp
Просмотреть файл

@ -82,6 +82,7 @@ public:
PRBool shouldFlush = PR_FALSE;
nsIFrame* frame = mWeakFrame.GetFrame();
if (frame) {
nsAutoScriptBlocker scriptBlocker;
frame->AttributeChanged(kNameSpaceID_None, nsGkAtoms::value, 0);
shouldFlush = PR_TRUE;
}
@ -115,6 +116,8 @@ nsProgressMeterFrame::AttributeChanged(PRInt32 aNameSpaceID,
nsIAtom* aAttribute,
PRInt32 aModType)
{
NS_ASSERTION(!nsContentUtils::IsSafeToRunScript(),
"Scripts not blocked in nsProgressMeterFrame::AttributeChanged!");
nsresult rv = nsBoxFrame::AttributeChanged(aNameSpaceID, aAttribute,
aModType);
if (NS_OK != rv) {