diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[2.6] Fix HTML escaping of to-source links

Browse Source
This commit is contained in:
Nicolas Grekas 2015-05-19 17:38:50 -07:00
parent e9bb160bcc
commit eecd197e6a
2 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/Symfony/Component/Debug/ExceptionHandler.php
View File

@ -380,12 +380,12 @@ EOF;
$file = preg_match('#[^/\\\\]*$#', $path, $file) ? $file[0] : $path; $file = preg_match('#[^/\\\\]*$#', $path, $file) ? $file[0] : $path;
if ($linkFormat = $this->fileLinkFormat) { if ($linkFormat = $this->fileLinkFormat) {
$link = str_replace(array('%f', '%l'), array($path, $line), $linkFormat); $link = strtr($this->escapeHtml($linkFormat), array('%f' => $path, , '%l' => (int) $line));
return sprintf(' in <a href="%s" title="Go to source">%s line %d</a>', htmlspecialchars($link), htmlspecialchars($file), $line); return sprintf(' in <a href="%s" title="Go to source">%s line %d</a>', $link, $file, $line);
} }
return sprintf(' in <a title="%s line %3$d" ondblclick="var f=this.innerHTML;this.innerHTML=this.title;this.title=f;">%s line %d</a>', htmlspecialchars($path), htmlspecialchars($file), $line); return sprintf(' in <a title="%s line %3$d" ondblclick="var f=this.innerHTML;this.innerHTML=this.title;this.title=f;">%s line %d</a>', $path, $file, $line);
} }
/** /**
@ -422,7 +422,7 @@ EOF;
} }
/** /**
* Returns an UTF-8 and HTML encoded string * Returns an UTF-8 and HTML encoded string.
*/ */
protected static function utf8Htmlize($str) protected static function utf8Htmlize($str)
{ {
@ -441,7 +441,7 @@ EOF;
} }
/** /**
* HTML-encodes a string * HTML-encodes a string.
*/ */
private function escapeHtml($str) private function escapeHtml($str)
{ {

6
src/Symfony/Component/HttpKernel/DataCollector/DumpDataCollector.php
View File

@ -100,9 +100,9 @@ class DumpDataCollector extends DataCollector implements DataDumperInterface
$name = $info->getTemplateName(); $name = $info->getTemplateName();
$src = $info->getEnvironment()->getLoader()->getSource($name); $src = $info->getEnvironment()->getLoader()->getSource($name);
$info = $info->getDebugInfo(); $info = $info->getDebugInfo();
if (isset($info[$trace[$i-1]['line']])) { if (isset($info[$trace[$i - 1]['line']])) {
$file = false; $file = false;
$line = $info[$trace[$i-1]['line']]; $line = $info[$trace[$i - 1]['line']];
$src = explode("\n", $src); $src = explode("\n", $src);
$fileExcerpt = array(); $fileExcerpt = array();
@ -259,7 +259,7 @@ class DumpDataCollector extends DataCollector implements DataDumperInterface
$name = strip_tags($this->style('', $name)); $name = strip_tags($this->style('', $name));
$file = strip_tags($this->style('', $file)); $file = strip_tags($this->style('', $file));
if ($fileLinkFormat) { if ($fileLinkFormat) {
$link = strtr($fileLinkFormat, array('%f' => $file, '%l' => (int) $line)); $link = strtr(strip_tags($this->style('', $fileLinkFormat)), array('%f' => $file, '%l' => (int) $line));
$name = sprintf('<a href="%s" title="%s">'.$s.'</a>', $link, $file, $name); $name = sprintf('<a href="%s" title="%s">'.$s.'</a>', $link, $file, $name);
} else { } else {
$name = sprintf('<abbr title="%s">'.$s.'</abbr>', $file, $name); $name = sprintf('<abbr title="%s">'.$s.'</abbr>', $file, $name);