From da9bb4ee99c9760ba8d696ff3e789d46a24d4aab Mon Sep 17 00:00:00 2001 From: Jeff Trawick Date: Sat, 15 Dec 2007 13:27:55 +0000 Subject: [PATCH] merge r600645 from trunk: http_protocol: Escape request method in 413 error reporting. Determined to be not generally exploitable, but a flaw in any case. PR 44014 Submitted by: Victor Stinner Reviewed by: wrowe, rpluem, trawick git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.0.x@604425 13f79535-47bb-0310-9956-ffa450edef68 --- CHANGES | 4 ++++ STATUS | 6 ------ modules/http/http_protocol.c | 4 ++-- 3 files changed, 6 insertions(+), 8 deletions(-) diff --git a/CHANGES b/CHANGES index f383f48231b..a44cfe5d4ac 100644 --- a/CHANGES +++ b/CHANGES @@ -1,6 +1,10 @@ -*- coding: utf-8 -*- Changes with Apache 2.0.62 + *) http_protocol: Escape request method in 413 error reporting. + Determined to be not generally exploitable, but a flaw in any case. + PR 44014 [Victor Stinner ] + *) SECURITY: CVE-2007-5000 (cve.mitre.org) mod_imagemap: Fix a cross-site scripting issue. Reported by JPCERT. [Joe Orton] diff --git a/STATUS b/STATUS index 87a4892b454..edddadc25c8 100644 --- a/STATUS +++ b/STATUS @@ -129,12 +129,6 @@ RELEASE SHOWSTOPPERS: PATCHES ACCEPTED TO BACKPORT FROM TRUNK: [ start all new proposals below, under PATCHES PROPOSED. ] - * http_protocol: Escape request method in 413 error reporting. - Determined to be not generally exploitable, but a flaw in any case. - PR 44014 [Victor Stinner ] - http://svn.apache.org/viewvc?view=rev&rev=600645 - +1: wrowe, rpluem, trawick - * http_protocol: Escape request method in 405 error reporting. Trunk: http://svn.apache.org/viewvc?view=rev&revision=603346 diff --git a/modules/http/http_protocol.c b/modules/http/http_protocol.c index d773045c224..d521842b7b6 100644 --- a/modules/http/http_protocol.c +++ b/modules/http/http_protocol.c @@ -2200,7 +2200,7 @@ static const char *get_canned_error_string(int status, case HTTP_LENGTH_REQUIRED: s1 = apr_pstrcat(p, "

A request of the requested method ", - r->method, + ap_escape_html(r->pool, r->method), " requires a valid Content-length.
\n", NULL); return(add_optional_notes(r, s1, "error-notes", "

\n")); @@ -2247,7 +2247,7 @@ static const char *get_canned_error_string(int status, "The requested resource
", ap_escape_html(r->pool, r->uri), "
\n", "does not allow request data with ", - r->method, + ap_escape_html(r->pool, r->method), " requests, or the amount of data provided in\n" "the request exceeds the capacity limit.\n", NULL)); -- 2.47.2