From: Greg Beaver <cellog@php.net>
Date: Tue, 22 Apr 2008 17:11:30 +0000 (+0000)
Subject: fix PATH_INFO munging, add to test case
X-Git-Tag: RELEASE_2_0_0b1~268
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cb8f158cd5fe7769ced391c7d3b49b3c0baaedc4;p=php

fix PATH_INFO munging, add to test case
---

diff --git a/ext/phar/phar_object.c b/ext/phar/phar_object.c
index d582f453b3..8599809279 100755
--- a/ext/phar/phar_object.c
+++ b/ext/phar/phar_object.c
@@ -91,7 +91,7 @@ static void phar_mung_server_vars(char *fname, char *entry, int entry_len, char
 		path_info = Z_STRVAL_PP(stuff);
 		code = Z_STRLEN_PP(stuff);
 		if (Z_STRLEN_PP(stuff) > entry_len && !memcmp(Z_STRVAL_PP(stuff), entry, entry_len)) {
-			ZVAL_STRINGL(*stuff, Z_STRVAL_PP(stuff) + entry_len, Z_STRLEN_PP(stuff) - entry_len - request_uri_len, 1);
+			ZVAL_STRINGL(*stuff, Z_STRVAL_PP(stuff) + entry_len, request_uri_len, 1);
 
 			MAKE_STD_ZVAL(temp); 
 			ZVAL_STRINGL(temp, path_info, code, 0);
diff --git a/ext/phar/tests/files/frontcontroller8.phar b/ext/phar/tests/files/frontcontroller8.phar
index 17648c208b..9061ec74fe 100644
Binary files a/ext/phar/tests/files/frontcontroller8.phar and b/ext/phar/tests/files/frontcontroller8.phar differ
diff --git a/ext/phar/tests/files/frontcontroller8.phar.inc b/ext/phar/tests/files/frontcontroller8.phar.inc
index 70bd421d97..b4ac8d3ffa 100644
--- a/ext/phar/tests/files/frontcontroller8.phar.inc
+++ b/ext/phar/tests/files/frontcontroller8.phar.inc
@@ -2,7 +2,7 @@
 @unlink(dirname(__FILE__) . '/frontcontroller8.phar');
 $a = new Phar(dirname(__FILE__) . '/frontcontroller8.phar');
 $a['a.phps'] = 'hio1';
-$a['a1.phps'] = '<?php var_dump($_SERVER["REQUEST_URI"]);';
+$a['a1.phps'] = '<?php var_dump($_SERVER["REQUEST_URI"], $_SERVER["PATH_INFO"]);';
 $a['a.jpg'] = 'hio2';
 $a['a.php'] = '<?php function hio(){}';
 $a['fronk.gronk'] = 'hio3';
diff --git a/ext/phar/tests/frontcontroller25.phpt b/ext/phar/tests/frontcontroller25.phpt
index 0ac7bac488..4d0475b078 100644
--- a/ext/phar/tests/frontcontroller25.phpt
+++ b/ext/phar/tests/frontcontroller25.phpt
@@ -11,4 +11,5 @@ files/frontcontroller8.phar
 --EXPECTHEADERS--
 Content-type: text/html
 --EXPECTF--
-string(42) "/frontcontroller25.php/a1.phps/extra/stuff"
\ No newline at end of file
+string(42) "/frontcontroller25.php/a1.phps/extra/stuff"
+string(12) "/extra/stuff"
\ No newline at end of file