Defect #6969
openLess-than sign in issue description and comments are not escaped
0%
Description
When an issue description or comment contains a less-than sign (<
), this sign is output verbatim in the issue page, instead of being escaped with ampersand-"lt"-semicolon. This causes the issue details page to be invalid XHTML, which is contrary to the page's doctype, and makes it impossible to read the page with an XML parser. I created an issue on the demo site to demonstrate the problem.
To reproduce, run xmllint URL-OF-ISSUE-PAGE
, like this:
$ xmllint http://demo.redmine.org/issues/38181 http://demo.redmine.org/issues/38181:166: parser error : StartTag: invalid element name <p>Hm: <</p> ^ http://demo.redmine.org/issues/38181:241: parser error : StartTag: invalid element name mg alt="Comment" src="/images/comment.png?1286930539" /></a></div><p>And this? < ^ http://demo.redmine.org/issues/38181:330: parser error : Entity 'copy' not defined Powered by <a href="http://www.redmine.org/">Redmine</a> © 2006-2010 Je ^
The third error is a false positive (xmllint doesn't know XHTML entities), but the first two errors are symptoms of this problem.
Files
Related issues
Updated by Go MAEDA about 9 years ago
- File issue6969_test_escaping.diff issue6969_test_escaping.diff added
- Category changed from Issues to Security
- Status changed from New to Confirmed
- Private changed from No to Yes
Thank you for reporting this issue.
Textile formatter in the latest trunk (r14634) is still affected.
Here is a test to catch this issue: issue6969_test_escaping.diff
Updated by Jean-Philippe Lang about 9 years ago
- Category changed from Security to Text formatting
- Status changed from Confirmed to Resolved
- Assignee set to Jean-Philippe Lang
- Private changed from Yes to No
- Resolution set to Fixed
Fixed in r14812.
Updated by Jean-Philippe Lang about 9 years ago
- Status changed from Resolved to Closed
Updated by Jean-Philippe Lang about 9 years ago
- Related to Defect #21202: Left aligned sign in tabular is not worked since applying #6969 added
Updated by Jean-Philippe Lang about 9 years ago
- Status changed from Closed to Reopened
- Target version changed from 2.6.8 to Candidate for next major release
Fix reverted, see #21202.