[interchange-cvs] [SCM] Interchange branch, master, updated. REL_5_7_2-30-g9004a52

Stefan Hornburg interchange-cvs at icdevgroup.org
Mon Nov 2 11:35:16 UTC 2009


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Interchange".

The branch, master has been updated
       via  9004a52ad56900c48d76f3780019bda9f8efcf97 (commit)
       via  4678ef60b7fe296a0ff0130c109984af0efa2b23 (commit)
      from  286ede3b6ad339107a183e8a52686827eba532b1 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 9004a52ad56900c48d76f3780019bda9f8efcf97
Author: Stefan Hornburg (Racke) <racke at linuxia.de>
Date:   Mon Nov 2 12:31:56 2009 +0100

    Recorded fix for #325.

commit 4678ef60b7fe296a0ff0130c109984af0efa2b23
Author: Stefan Hornburg (Racke) <racke at linuxia.de>
Date:   Mon Nov 2 12:30:58 2009 +0100

    Fix crash with TolerateGet and upload forms (#325).

-----------------------------------------------------------------------

Summary of changes and diff:
 WHATSNEW-5.7       |    2 ++
 lib/Vend/Server.pm |    6 +++---
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/WHATSNEW-5.7 b/WHATSNEW-5.7
index a379231..8277e8b 100644
--- a/WHATSNEW-5.7
+++ b/WHATSNEW-5.7
@@ -20,6 +20,8 @@ Core
 
 * Remove empty mv_arg from more-list links.
 
+* Fix crash with TolerateGet and upload forms (#325).
+ 
 Tags
 ----
 
diff --git a/lib/Vend/Server.pm b/lib/Vend/Server.pm
index c4f95b1..888cff6 100644
--- a/lib/Vend/Server.pm
+++ b/lib/Vend/Server.pm
@@ -252,7 +252,7 @@ EOF
 #::logDebug("content type header: " . $CGI::content_type);
 		## check for valid content type
 		if ($CGI::content_type =~ m{^(?:multipart/form-data|application/x-www-form-urlencoded|application/xml|application/json)\b}i) {
-			parse_post(\$CGI::query_string)
+			parse_post(\$CGI::query_string, 1)
 				if $Global::TolerateGet;
 			parse_post($h->{entity});
 		}
@@ -332,7 +332,7 @@ sub store_cgi_kv {
 }
 
 sub parse_post {
-	my $sref = shift;
+	my ($sref, $is_get) = @_;
 	return unless length $$sref;
 
 	my (@pairs, $pair, $key, $value, $charset);
@@ -346,7 +346,7 @@ sub parse_post {
 
 	$CGI::values{mv_form_charset} = $charset;
 
-	if ($CGI::content_type =~ m{^multipart/}i) {
+	if ($CGI::content_type =~ m{^multipart/}i && ! $is_get) {
 		return parse_multipart($sref) if $CGI::useragent !~ /MSIE\s+5/i;
 		# try and work around an apparent IE5 bug that sends the content type
 		# of the next POST after a multipart/form POST as multipart also -


hooks/post-receive
-- 
Interchange



More information about the interchange-cvs mailing list