Corrections of phpt files

From: Date: Tue, 23 Oct 2007 21:33:17 +0000
Subject: Corrections of phpt files
Groups: php.qa 
Request: Send a blank email to php-qa+get-51172@lists.php.net to get a copy of this message
I've downloaded the last php release candidate and I've made 'make test' (result : http://news.php.net/php.qa/51142) In the bug report I sent, I found two lines that should not correspond to PHP bugs : dl() filename length checks (CVE-2007-4887) [ext/standard/tests/general_functions/dl-cve-2007-4887.phpt] phpinfo() [ext/standard/tests/general_functions/phpinfo.phpt] I suppose that errors are in the .phpt files. That's the reason of this email. I want to submit here possible corrections. For the first non-bug (ext/standard/tests/general_functions/dl-cve-2007-4887.phpt), I've added those two lines in the .phpt file in order to pass the test-case : --INI-- enable_dl=1 For the second non-bug (ext/standard/tests/general_functions/phpinfo.phpt), I've changed the line 27 : Loaded Configuration File => %a I use %a instead of %s to handle optional additional lines (like those I have in my report). I have run again the 'make test' after that. Those corrections seems to work well. Hope that helps. -Mazzu

« previous php.qa (#51172) next »