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