Re: com php-src: fix bug #61807 - Buffer Overflow in apache_request_headers: NEWS sapi/cgi/cgi_main.c sapi/cgi/tests/apache_request_headers.phpt
| From: | Felipe Pena | Date: | Tue, 08 May 2012 15:38:05 +0000 |
| Subject: | Re: com php-src: fix bug #61807 - Buffer Overflow in apache_request_headers: NEWS sapi/cgi/cgi_main.c sapi/cgi/tests/apache_request_headers.phpt | ||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-68997@lists.php.net to get a copy of this message | ||
2012/5/7 Stanislav Malyshev <stas@php.net>:
> Commit: Â Â eb8f3b025b0a6dbbf6b44bf51d8cf345437b7354
> Author: Â Â Stanislav Malyshev <stas@php.net> Â Â Â Â Mon, 7 May 2012 12:24:22 -0700
> Parents: Â fc3ba0552fd5c2d7b5870f3e2fec0a9a2d2996f4
> Branches: Â PHP-5.4.3
>
> Link: Â Â Â
> http://git.php.net/?p=php-src.git;a=commitdiff;h=eb8f3b025b0a6dbbf6b44bf51d8cf345437b7354íSˆçs 0s
> ÙiŸŸqÇ
>
> Log:
> fix bug #61807 - Buffer Overflow in apache_request_headers
>
> Bugs:
> https://bugs.php.net/61807
>
> Changed paths:
> Â M Â NEWS
> Â M Â sapi/cgi/cgi_main.c
> Â A Â sapi/cgi/tests/apache_request_headers.phpt
>
>
> Diff:
> diff --git a/NEWS b/NEWS
> index a41a5d1..7603cfb 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -5,6 +5,7 @@ PHP Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â
> Â Â Â Â Â Â Â Â Â Â Â NEWS
> Â - CGI
> Â . Re-Fix PHP-CGI query string parameter vulnerability, CVE-2012-1823.
> Â Â (Stas)
> + Â . Fix bug #61807 - Buffer Overflow in apache_request_headers.
>
> Â 03 May 2012, PHP 5.4.2
>
> diff --git a/sapi/cgi/cgi_main.c b/sapi/cgi/cgi_main.c
> index 71404a4..a1690b1 100644
> --- a/sapi/cgi/cgi_main.c
> +++ b/sapi/cgi/cgi_main.c
> @@ -1614,15 +1614,21 @@ PHP_FUNCTION(apache_request_headers) /* {{{ */
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â p = var + 5;
>
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â var = q = t;
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â // First char keep uppercase
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â *q++ = *p++;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â while (*p) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (*p == '_') {
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (*p == '=') {
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â // End of name
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â break;
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â } else if (*p == '_') {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â *q++ = '-';
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â p++;
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (*p) {
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â // First char after -
> keep uppercase
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (*p &&
> *p!='=' && *p!='_') {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â *q++ = *p++;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â }
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â } else if (*p >= 'A'
> && *p <= 'Z') {
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â // lowercase
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â *q++ = (*p++ -
> 'A' + 'a');
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â } else {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â *q++ = *p++;
I see C++ comments.
--
Regards,
Felipe Pena