public/nixpkgs
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

stdenv: Harden hook runners

Browse Source 
Instead of eval, use a "nameref" to get the name of the array and
iterate with that. Also, make the for-loop parameter a local variable,
too.
This commit is contained in:
John Ericson 2017-07-06 19:29:48 -04:00
parent 74f55017d2
commit a14cf06182
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
pkgs/stdenv/generic/setup.sh
View File

@ -17,8 +17,9 @@ runHook() {
shift shift
local var="$hookName" local var="$hookName"
if [[ "$hookName" =~ Hook$ ]]; then var+=s; else var+=Hooks; fi if [[ "$hookName" =~ Hook$ ]]; then var+=s; else var+=Hooks; fi
eval "local -a dummy=(\"\${$var[@]}\")" local -n var
for hook in "_callImplicitHook 0 $hookName" "${dummy[@]}"; do local hook
for hook in "_callImplicitHook 0 $hookName" "${var[@]}"; do
_eval "$hook" "$@" _eval "$hook" "$@"
done done
return 0 return 0
@ -32,8 +33,9 @@ runOneHook() {
shift shift
local var="$hookName" local var="$hookName"
if [[ "$hookName" =~ Hook$ ]]; then var+=s; else var+=Hooks; fi if [[ "$hookName" =~ Hook$ ]]; then var+=s; else var+=Hooks; fi
eval "local -a dummy=(\"\${$var[@]}\")" local -n var
for hook in "_callImplicitHook 1 $hookName" "${dummy[@]}"; do local hook
for hook in "_callImplicitHook 1 $hookName" "${var[@]}"; do
if _eval "$hook" "$@"; then if _eval "$hook" "$@"; then
return 0 return 0
fi fi