From: Byron Jones <glob@glob.com.au>
Date: Tue, 2 Nov 2010 23:21:42 +0000 (+0100)
Subject: Bug 600464: (CVE-2010-3172) [SECURITY] Content/Header injection due to non-random... 
X-Git-Tag: bugzilla-3.6.3~1
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=32000411b5bfee0560766aa909c0085f53a3fc13;p=thirdparty%2Fbugzilla.git

Bug 600464: (CVE-2010-3172) [SECURITY] Content/Header injection due to non-random multipart/x-mixed-replace boundary
r=mkanat a=LpSolit
---

diff --git a/Bugzilla/CGI.pm b/Bugzilla/CGI.pm
index eec5df3745..0188099469 100644
--- a/Bugzilla/CGI.pm
+++ b/Bugzilla/CGI.pm
@@ -221,7 +221,8 @@ sub multipart_init {
     }
 
     # Set the MIME boundary and content-type
-    my $boundary = $param{'-boundary'} || '------- =_aaaaaaaaaa0';
+    my $boundary = $param{'-boundary'}
+        || '------- =_' . generate_random_password(16);
     delete $param{'-boundary'};
     $self->{'separator'} = "\r\n--$boundary\r\n";
     $self->{'final_separator'} = "\r\n--$boundary--\r\n";