diff options
| author |  SZEDER Gábor <szeder@ira.uka.de> | 2011-11-09 11:02:50 +0100 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2011-11-09 05:44:05 -0800 |
| commit | 4804d437916d08068143dfc265cc7b5e2f4883e6 (patch) | |
| tree | 6fbe88da4d83f1641c0625034a8c604632290f9c /contrib/completion | |
| parent | completion: fix issue with process substitution not working on Git for Windows (diff) | |
| download | tgif-4804d437916d08068143dfc265cc7b5e2f4883e6.tar.xz | |
completion: don't leak variable from the prompt into environment
Commit e5b8eebc (completion: fix issue with process substitution not
working on Git for Windows, 2011-10-26) introduced a new variable in
__git_ps1_show_upstream(), but didn't declare it as local to prevent it
from leaking into the environment.
We may want to rewrite it like the following, but that can wait until the
next cycle.
while read key value
do
...
done <<-EOF
$(git config -z --get-regexp ...)
EOF
Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib/completion')
| -rwxr-xr-x | contrib/completion/git-completion.bash | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash index fc3d0b1a06..32df141f91 100755 --- a/contrib/completion/git-completion.bash +++ b/contrib/completion/git-completion.bash @@ -110,7 +110,7 @@ __git_ps1_show_upstream () local upstream=git legacy="" verbose="" # get some config options from git-config - output="$(git config -z --get-regexp '^(svn-remote\..*\.url|bash\.showupstream)$' 2>/dev/null | tr '\0\n' '\n ')" + local output="$(git config -z --get-regexp '^(svn-remote\..*\.url|bash\.showupstream)$' 2>/dev/null | tr '\0\n' '\n ')" while read key value; do case "$key" in bash.showupstream) |