From: Byron Jones <glob@glob.com.au>
Date: Tue, 2 Nov 2010 23:23:50 +0000 (+0100)
Subject: Bug 600464: (CVE-2010-3172) [SECURITY] Content/Header injection due to non-random... 
X-Git-Tag: bugzilla-3.2.9~1
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b4ced4368cc5d779915e140e3c003463e8a95fef;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 1799786d33..2c32c5d611 100644
--- a/Bugzilla/CGI.pm
+++ b/Bugzilla/CGI.pm
@@ -172,7 +172,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";