summaryrefslogtreecommitdiff
path: root/src/share/mh/publish_key
diff options
context:
space:
mode:
authorJonas Smedegaard <dr@jones.dk>2013-03-08 19:29:38 +0100
committerDaniel Kahn Gillmor <dkg@fifthhorseman.net>2013-03-13 02:13:38 -0400
commit0c081207c94b0ed338a704fafac10166ce98bba7 (patch)
treeef791d7ea68fd6a51074156bbac97d9c6b5a398a /src/share/mh/publish_key
parented10318d3760b56e57d5e1bef04ab57761ab8bd1 (diff)
Pass only single commands through su wrapper
It is a healthy coding practice to keep each argument separate when executing system calls, i.e. quote each variable separately instead of relying on whitespace to indicate argument separation. Quoting shell-inside-shell is tricky to do right, and gets trickier when more than a single command is wrapped together. This patch simplifies convoluted shell calls to contain only one command each. Example: before: bash -c ". .../common; process_authorized_user_ids -" after: bash -c ".../common process_authorized_user_ids -"
Diffstat (limited to 'src/share/mh/publish_key')
0 files changed, 0 insertions, 0 deletions