[WebProfilerBundle] Fix infinite loop in JS

nicolas-grekas · nicolas-grekas · commit 68b6c32458d1 · 2016-08-31T17:02:18.000+02:00
diff --git a/src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/toolbar_js.html.twig b/src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/toolbar_js.html.twig
@@ -16,8 +16,11 @@
             function(xhr, el) {
 
                 /* Evaluate embedded scripts inside the toolbar */
-                var rx=/<script>(.*?)<\/script>/g, s;
-                while (s = rx.exec(el.innerHTML)) {eval(s[1]);};
+                var i, scripts = [].slice.call(el.querySelectorAll('script'));
+
+                for (i = 0; i < scripts.length; ++i) {
+                    eval(scripts[i].firstChild.nodeValue);
+                }
 
                 el.style.display = -1 !== xhr.responseText.indexOf('sf-toolbarreset') ? 'block' : 'none';
 
@@ -38,8 +41,8 @@
                 Sfjs.renderAjaxRequests();
 
                 /* Handle toolbar-info position */
-                var toolbarBlocks = document.querySelectorAll('.sf-toolbar-block');
-                for (var i = 0; i < toolbarBlocks.length; i += 1) {
+                var toolbarBlocks = [].slice.call(el.querySelectorAll('.sf-toolbar-block'));
+                for (i = 0; i < toolbarBlocks.length; ++i) {
                     toolbarBlocks[i].onmouseover = function () {
                         var toolbarInfo = this.querySelectorAll('.sf-toolbar-info')[0];
                         var pageWidth = document.body.clientWidth;
