Summary

Fix a regression with displaying diff viewer file entries.

Review Request #3820 — Created Jan. 30, 2013 and submitted Jan. 30, 2013, 11:10 p.m. — Latest diff uploaded Jan. 30, 2013, 8:29 p.m.

Information

Owner

chipx86

Repository

Review Board

Branch

release-1.7.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Fix a regression with displaying diff viewer file entries.

The "Files Changed" list in the diff viewer broke with Windows-style
paths. A previous change was made to fix a JavaScript injection issue,
but that caused some escaping issues when using backslashes. We now
escape both HTML and JavaScript.

Testing Done

Added some entries with backslashes, and added one with a <script>alert('...')</script>.
All were escaped as expected.