summaryrefslogtreecommitdiff
path: root/t/t4034
diff options
context:
space:
mode:
authorLibravatar Joel Holdsworth <jholdsworth@nvidia.com>2022-01-06 21:40:34 +0000
committerLibravatar Junio C Hamano <gitster@pobox.com>2022-01-06 14:55:12 -0800
commit8a470599f340e2e2bcf96f0b267199de321edef2 (patch)
tree4424c1edc1f9d90d4952b15d57170c8dbd38f2fe /t/t4034
parentgit-p4: don't select shell mode using the type of the command argument (diff)
downloadtgif-8a470599f340e2e2bcf96f0b267199de321edef2.tar.xz
git-p4: pass command arguments as lists instead of using shell
In the majority of the subprocess calls where shell=True was used, it was only needed to parse command arguments by spaces. In each of these cases, the commands are now being passed in as lists instead of strings. This change aids the comprehensibility of the code. Constucting commands and arguments using strings risks bugs from unsanitized inputs, and the attendant complexity of properly quoting and escaping command arguments. Signed-off-by: Joel Holdsworth <jholdsworth@nvidia.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4034')
0 files changed, 0 insertions, 0 deletions