revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-22 06:42:41 +01:00
Code Issues Releases Wiki Activity

Fix dynamic string usage as safe input

Browse source 
Summary: I somehow missed it.

Test Plan: Sent this diff by different copy of Arcanist.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D4800
This commit is contained in:
vrana 2013-02-03 15:00:23 -08:00
parent a9e316bf9c
commit 95b2c4587d
1 changed files with 11 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
scripts/arcanist.php
View file

@ -555,12 +555,17 @@ function reenter_if_this_is_arcanist_or_libphutil(
$libphutil_path = dirname(phutil_get_library_root('phutil')); $libphutil_path = dirname(phutil_get_library_root('phutil'));
} }
$err = phutil_passthru( if (phutil_is_windows()) {
phutil_is_windows() $err = phutil_passthru(
? 'set ARC_PHUTIL_PATH=%s & %Ls' 'set ARC_PHUTIL_PATH=%s & %Ls',
: 'ARC_PHUTIL_PATH=%s %Ls', $libphutil_path,
$libphutil_path, $original_argv);
$original_argv); } else {
$err = phutil_passthru(
'ARC_PHUTIL_PATH=%s %Ls',
$libphutil_path,
$original_argv);
}
exit($err); exit($err);
} }