diff options
author | Joel Holdsworth <jholdsworth@nvidia.com> | 2022-01-06 21:40:34 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-01-06 14:55:12 -0800 |
commit | 8a470599f340e2e2bcf96f0b267199de321edef2 (patch) | |
tree | 4424c1edc1f9d90d4952b15d57170c8dbd38f2fe /t/t7811-grep-open.sh | |
parent | git-p4: don't select shell mode using the type of the command argument (diff) | |
download | tgif-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/t7811-grep-open.sh')
0 files changed, 0 insertions, 0 deletions