From 1c4806c515b365e39485a48f366ac9ecb7caa77c Mon Sep 17 00:00:00 2001 From: Ilia Alshanetsky Date: Wed, 13 Dec 2006 00:41:33 +0000 Subject: Fixed bug #39201 (Possible crash in Apache 2 with 413 ErrorHandler). --- sapi/apache2handler/sapi_apache2.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'sapi/apache2handler') diff --git a/sapi/apache2handler/sapi_apache2.c b/sapi/apache2handler/sapi_apache2.c index 87b1861531..a9deab7854 100644 --- a/sapi/apache2handler/sapi_apache2.c +++ b/sapi/apache2handler/sapi_apache2.c @@ -587,8 +587,13 @@ zend_first_try { } } - /* check if comming due to ErrorDocument */ - if (parent_req && parent_req->status != HTTP_OK && strcmp(r->protocol, "INCLUDED")) { + /* + * check if comming due to ErrorDocument + * We make a special exception of 413 (Invalid POST request) as the invalidity of the request occurs + * during processing of the request by PHP during POST processing. Therefor we need to re-use the exiting + * PHP instance to handle the request rather then creating a new one. + */ + if (parent_req && parent_req->status != HTTP_OK && parent_req->status != 413 && strcmp(r->protocol, "INCLUDED")) { parent_req = NULL; goto normal; } -- cgit v1.2.1