tests: policy: avoid using caller's variable

Fix unintended use of caller's variable. Use the corresponding function
parameter instead. ShellCheck:

warning: policy_settings_dir is referenced but not assigned. [SC2154]

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
This commit is contained in:
Dan Mihai 2025-03-19 00:13:46 +00:00
parent 59a70a2b28
commit d83b8349a2

View File

@ -258,9 +258,9 @@ add_copy_from_host_to_policy_settings() {
local -r genpolicy_settings_dir="$1"
local exec_command=(test -d /tmp)
add_exec_to_policy_settings "${policy_settings_dir}" "${exec_command[@]}"
add_exec_to_policy_settings "${genpolicy_settings_dir}" "${exec_command[@]}"
exec_command=(tar -xmf - -C /tmp)
add_exec_to_policy_settings "${policy_settings_dir}" "${exec_command[@]}"
add_exec_to_policy_settings "${genpolicy_settings_dir}" "${exec_command[@]}"
}
# Change genpolicy settings to allow executing on the Guest VM the commands
@ -270,7 +270,7 @@ add_copy_from_guest_to_policy_settings() {
local -r copied_file="$2"
exec_command=(tar cf - "${copied_file}")
add_exec_to_policy_settings "${policy_settings_dir}" "${exec_command[@]}"
add_exec_to_policy_settings "${genpolicy_settings_dir}" "${exec_command[@]}"
}
hard_coded_policy_tests_enabled() {