]> granicus.if.org Git - php/commitdiff
Merge branch 'PHP-5.5' into PHP-5.6
authorAdam Harvey <aharvey@php.net>
Mon, 7 Jul 2014 20:45:01 +0000 (20:45 +0000)
committerAdam Harvey <aharvey@php.net>
Mon, 7 Jul 2014 20:45:01 +0000 (20:45 +0000)
* PHP-5.5:
  Fixed bug #66830 (Empty header causes PHP built-in web server to hang).

1  2 
NEWS
sapi/cli/php_cli_server.c
sapi/cli/tests/bug66830.phpt

diff --cc NEWS
index a717429605af7ff4ce3ec0633ce6d092dff9161d,afe5bb91ce701316cf67b8d882c3efaf55cf53de..b5c9654117c08e76e27e4c5959bf0d8e463b2f56
--- 1/NEWS
--- 2/NEWS
+++ b/NEWS
@@@ -7,8 -13,17 +7,12 @@@ PH
      generator). (Nikita)
    . Fixed bug #67151 (strtr with empty array crashes). (Nikita)
  
 -  . Implemented FR #67429 (CLI server is missing some new HTTP response codes).
 -    (Adam)
+ - CLI server:
 -- FPM:
 -  . Fixed bug #67531 (syslog cannot be set in pool configuration). (Remi)
 -
+   . Fixed bug #66830 (Empty header causes PHP built-in web server to hang).
+     (Adam)
  - Intl:
 -  . Fixed bug #66921 (Wrong argument type hint for function 
 +  . Fixed bug #66921 (Wrong argument type hint for function
      intltz_from_date_time_zone). (Stas)
    . Fixed bug #67052 (NumberFormatter::parse() resets LC_NUMERIC setting).
      (Stas)
Simple merge
index 0000000000000000000000000000000000000000,68fcf21c11a5997c8d33081991724422674d0378..58c07e031ab7fb965ce42ec7b25c49ed5c7a843e
mode 000000,100644..100644
--- /dev/null
@@@ -1,0 -1,43 +1,43 @@@
 -Content-type: text/html
+ --TEST--
+ Bug #66830 (Empty header causes PHP built-in web server to hang)
+ --SKIPIF--
+ <?php
+ include "skipif.inc";
+ ?>
+ --FILE--
+ <?php
+ include "php_cli_server.inc";
+ php_cli_server_start(<<<'PHP'
+ header(' ');
+ PHP
+ );
+ list($host, $port) = explode(':', PHP_CLI_SERVER_ADDRESS);
+ $port = intval($port)?:80;
+ $fp = fsockopen($host, $port, $errno, $errstr, 0.5);
+ if (!$fp) {
+   die("connect failed");
+ }
+ if(fwrite($fp, <<<HEADER
+ GET / HTTP/1.1
+ Host: {$host}
+ HEADER
+ )) {
+     while (!feof($fp)) {
+         echo fgets($fp);
+     }
+ }
+ fclose($fp);
+ ?>
+ --EXPECTF--
+ HTTP/1.1 200 OK
+ Host: %s
+ Connection: close
+ X-Powered-By: %s
++Content-type: text/html; charset=UTF-8