Fix a bug found by vrana in audit
Summary: seen https://secure.phabricator.com/rPf5e71eb88920d4afb5590b0682eda66f09b2c024 Test Plan: i believe in vrana. Reviewers: vrana Reviewed By: vrana CC: aran, epriestley Differential Revision: https://secure.phabricator.com/D3588
This commit is contained in:
		@@ -83,6 +83,6 @@ PhutilTranslator::getInstance()
 | 
			
		||||
$paths = PhabricatorEnv::getEnvConfig('environment.append-paths');
 | 
			
		||||
if (!empty($paths)) {
 | 
			
		||||
  $current_env_path = getenv('PATH');
 | 
			
		||||
  $new_env_paths = implode(':', $paths);
 | 
			
		||||
  putenv('PATH='.$current_env_path.':'.$new_env_paths);
 | 
			
		||||
  $new_env_paths = implode(PATH_SEPARATOR, $paths);
 | 
			
		||||
  putenv('PATH='.$current_env_path.PATH_SEPARATOR.$new_env_paths);
 | 
			
		||||
}
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user