Use wfEscapeShellArg() so that the script works correctly when paths contain whitespaces
authorAlexandre Emsenhuber <ialex@users.mediawiki.org>
Sat, 12 Sep 2009 10:19:56 +0000 (10:19 +0000)
committerAlexandre Emsenhuber <ialex@users.mediawiki.org>
Sat, 12 Sep 2009 10:19:56 +0000 (10:19 +0000)
maintenance/syntaxChecker.php

index 387b92a..90d1f87 100644 (file)
@@ -125,7 +125,7 @@ class SyntaxChecker extends Maintenance {
         * @return boolean
         */
        private function checkFileWithCli( $file ) {
-               $res = exec( 'php -l ' . $file ); 
+               $res = exec( 'php -l ' . wfEscapeShellArg( $file ) ); 
                if( strpos( $res, 'No syntax errors detected' ) === false ) {
                        $this->mFailures[$file] = $res;
                        $this->output( $res . "\n" );