BUG - Cleanup website tmp directories (#5434)

florian030 · florian030 · commit a08a29104365 · 2020-01-07T08:11:00.000+01:00
diff --git a/server/lib/classes/cron.d/200-logfiles.inc.php b/server/lib/classes/cron.d/200-logfiles.inc.php
@@ -217,7 +217,7 @@ public function onRunJob() {
 			foreach($records as $rec){
 				$tmp_path = realpath($rec['document_root'].'/tmp');
 				if($tmp_path != '' && strlen($tmp_path) > 10 && is_dir($tmp_path) && $app->system->is_user($rec['system_user'])){
-					exec("cd ?; find . -mtime +1 -name 'sess_*' | grep -v -w .no_delete | xargs rm > /dev/null 2> /dev/null", $tmp_path);
+					$app->system->exec_safe("cd ?; find . -mtime +1 -name 'sess_*' | grep -v -w .no_delete | xargs rm > /dev/null 2> /dev/null", $tmp_path);
 				}
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,7 @@ public function onRunJob() {`
`217`	`217`	`foreach($records as $rec){`
`218`	`218`	`$tmp_path = realpath($rec['document_root'].'/tmp');`
`219`	`219`	`if($tmp_path != '' && strlen($tmp_path) > 10 && is_dir($tmp_path) && $app->system->is_user($rec['system_user'])){`
`220`		`- exec("cd ?; find . -mtime +1 -name 'sess_*' \| grep -v -w .no_delete \| xargs rm > /dev/null 2> /dev/null", $tmp_path);`
	`220`	`+ $app->system->exec_safe("cd ?; find . -mtime +1 -name 'sess_*' \| grep -v -w .no_delete \| xargs rm > /dev/null 2> /dev/null", $tmp_path);`
`221`	`221`	`}`
`222`	`222`	`}`
`223`	`223`	`}`