diff --git a/src/opnsense/scripts/systemhealth/clearlog b/src/opnsense/scripts/systemhealth/clearlog index 15b14671c..8d18e83ed 100755 --- a/src/opnsense/scripts/systemhealth/clearlog +++ b/src/opnsense/scripts/systemhealth/clearlog @@ -49,7 +49,6 @@ if (isset($opts['m']) && isset($opts['f'])) { } else { $basename = "/var/log/{$mname}/{$fname}"; } - $filename = "{$basename}.log"; if (is_dir($basename)) { foreach (glob("{$basename}/{$fname}_*.log") as $filename) { @@ -58,17 +57,10 @@ if (isset($opts['m']) && isset($opts['f'])) { system_syslog_start(); } + $filename = "{$basename}.log"; if (is_file($filename)) { - /* preserve file ownership and permissions - XXX: not sure if this is still needed, at least it's not consistent with how directories are reset now. - */ - if (file_exists($filename)) { - $handle = fopen($filename, 'r+'); - if ($handle) { - ftruncate($handle, 0); - fclose($handle); - } - } + // remove legacy clog file + @unlink($filename); system_syslog_start(); // XXX: should probably add some plugin hook for this. if ($fname == 'dhcpd' && $mname == 'core') {