Cache list of all config files for unlock, in case one was removed ht…

…tps://forum.virtualmin.com/t/virtualmin-gpl-just-delete-it-because-it-automatically-creates-index-html/109870/8
virtualmin · Mar 20, 2021 · b14e728 · b14e728
1 parent 5aa43b4
commit b14e728
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/virtualmin-nginx-lib.pl b/virtualmin-nginx-lib.pl
@@ -384,7 +384,8 @@ sub flush_config_file_lines
 sub lock_all_config_files
 {
 my ($parent) = @_;
-foreach my $f (&get_all_config_files($parent)) {
+@lock_all_config_files_cache = &get_all_config_files($parent);
+foreach my $f (@lock_all_config_files_cache) {
 	&lock_file($f);
 	}
 }
@@ -394,9 +395,10 @@ sub lock_all_config_files
 sub unlock_all_config_files
 {
 my ($parent) = @_;
-foreach my $f (reverse(&get_all_config_files($parent))) {
+foreach my $f (reverse(@lock_all_config_files_cache)) {
 	&unlock_file($f);
 	}
+@lock_all_config_files_cache = ();
 }
 
 # get_all_config_files([&parent])