cvs: php-src(PHP_5_0) / NEWS /sapi/isapi php5isapi.c

sniper Sat Apr 16 23:39:14 2005 EDT Modified files: (Branch: PHP_5_0) /php-src NEWS /php-src/sapi/isapi php5isapi.c Log: MFH: - Fixed bug #31887 (ISAPI: Custom 5xx error does not return correct HTTP response message) http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1760.2.326&r2=1.1760.2.327&ty=u Index: php-src/NEWS diff -u php-src/NEWS:1.1760.2.326 php-src/NEWS:1.1760.2.327 --- php-src/NEWS:1.1760.2.326 Sat Apr 16 08:26:51 2005 +++ php-src/NEWS Sat Apr 16 23:39:11 2005 @@ -25,6 +25,8 @@ - Fixed bug #31583 (php_std_date() uses short day names in non-y2k_compliance mode). (mike at php dot net) - Fixed bug #32282 (Segfault in mysqli_fetch_array on 64-bit) (Georg). +- Fixed bug #31887 (ISAPI: Custom 5xx error does not return correct HTTP + response message). (Jani) - Fixed bug #31502 (Wrong deserialization from session when using WDDX serializer). (Dmitry) - Fixed bug #31363 (broken non-blocking flock()). ian at snork dot net http://cvs.php.net/diff.php/php-src/sapi/isapi/php5isapi.c?r1=1.3.2.2&r2=1.3.2.3&ty=u Index: php-src/sapi/isapi/php5isapi.c diff -u php-src/sapi/isapi/php5isapi.c:1.3.2.2 php-src/sapi/isapi/php5isapi.c:1.3.2.3 --- php-src/sapi/isapi/php5isapi.c:1.3.2.2 Fri Dec 17 22:14:52 2004 +++ php-src/sapi/isapi/php5isapi.c Sat Apr 16 23:39:14 2005 @@ -16,7 +16,7 @@ | Ben Mansell <ben@zeus.com> (Zeus Support) | +----------------------------------------------------------------------+ */ -/* $Id: php5isapi.c,v 1.3.2.2 2004/12/18 03:14:52 sniper Exp $ */ +/* $Id: php5isapi.c,v 1.3.2.3 2005/04/17 03:39:14 sniper Exp $ */ #include "php.h" #include <httpext.h> @@ -57,7 +57,7 @@ #endif */ -#define MAX_STATUS_LENGTH sizeof("xxxx LONGEST STATUS DESCRIPTION") +#define MAX_STATUS_LENGTH sizeof("xxxx LONGEST POSSIBLE STATUS DESCRIPTION") #define ISAPI_SERVER_VAR_BUF_SIZE 1024 #define ISAPI_POST_DATA_BUF 1024 @@ -245,8 +245,8 @@ char *combined_headers, *combined_headers_ptr; LPEXTENSION_CONTROL_BLOCK lpECB = (LPEXTENSION_CONTROL_BLOCK) SG(server_context); HSE_SEND_HEADER_EX_INFO header_info; - char status_buf[MAX_STATUS_LENGTH]; sapi_header_struct default_content_type; + char *status_buf = NULL; /* Obtain headers length */ if (SG(sapi_headers).send_default_content_type) { @@ -277,10 +277,21 @@ case 401: header_info.pszStatus = "401 Authorization Required"; break; - default: - snprintf(status_buf, MAX_STATUS_LENGTH, "%d Undescribed", SG(sapi_headers).http_response_code); + default: { + const char *sline = SG(sapi_headers).http_status_line; + + status_buf = emalloc(MAX_STATUS_LENGTH + 1); + + /* httpd requires that r->status_line is set to the first digit of + * the status-code: */ + if (sline && strlen(sline) > 12 && strncmp(sline, "HTTP/1.", 7) == 0 && sline[8] == ' ') { + status_buf = estrndup(sline + 9, MAX_STATUS_LENGTH); + } else { + snprintf(status_buf, MAX_STATUS_LENGTH, "%d Undescribed", SG(sapi_headers).http_response_code); + } header_info.pszStatus = status_buf; break; + } } header_info.cchStatus = strlen(header_info.pszStatus); header_info.pszHeader = combined_headers; @@ -291,6 +302,9 @@ lpECB->ServerSupportFunction(lpECB->ConnID, HSE_REQ_SEND_RESPONSE_HEADER_EX, &header_info, NULL, NULL); efree(combined_headers); + if (status_buf) { + efree(status_buf); + } return SAPI_HEADER_SENT_SUCCESSFULLY; } @@ -306,7 +320,6 @@ } - static int sapi_isapi_read_post(char *buffer, uint count_bytes TSRMLS_DC) { LPEXTENSION_CONTROL_BLOCK lpECB = (LPEXTENSION_CONTROL_BLOCK) SG(server_context);