From: Dylan Hardison <dylan@mozilla.com>
Date: Tue, 22 Dec 2015 18:34:38 +0000 (-0500)
Subject: Bug 1232785 - [SECURITY] Buglists in CSV format can be parsed as valid javascript... 
X-Git-Tag: release-5.1.1~122
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=34f8910f2b22ada382d191052f70a7427c88f95a;p=thirdparty%2Fbugzilla.git

Bug 1232785 - [SECURITY] Buglists in CSV format can be parsed as valid javascript in some browsers
r=dkl,a=dkl
---

diff --git a/Bugzilla/Template.pm b/Bugzilla/Template.pm
index ef45cb313d..04abe82001 100644
--- a/Bugzilla/Template.pm
+++ b/Bugzilla/Template.pm
@@ -859,6 +859,9 @@ sub create {
             {
                 my ($var) = @_;
                 $var = ' ' . $var if substr($var, 0, 1) eq '=';
+                # backslash is not special to CSV, but it can be used to confuse some browsers...
+                # so we do not allow it to happen. We only do this for logged-in users.
+                $var =~ s/\\/\x{FF3C}/g if Bugzilla->user->id;
                 $var =~ s/\"/\"\"/g;
                 if ($var !~ /^-?(\d+\.)?\d*$/) {
                     $var = "\"$var\"";