diff --git a/server/plugins-available/getmail_plugin.inc.php b/server/plugins-available/getmail_plugin.inc.php index c6daf42a6c76f4b96f7e753726c2e5f45ab45b63..b7645bbe7417878f87f401c157a09c953518074e 100644 --- a/server/plugins-available/getmail_plugin.inc.php +++ b/server/plugins-available/getmail_plugin.inc.php @@ -140,12 +140,22 @@ class getmail_plugin { function delete($event_name,$data) { global $app, $conf; + // load the server specific configuration options for getmail + $app->uses("getconf"); + $getmail_config = $app->getconf->get_server_config($conf["server_id"], 'getmail'); + $this->getmail_config_dir = $getmail_config["getmail_config_dir"]; + $config_file_path = escapeshellcmd($this->getmail_config_dir.'/'.$data["old"]["source_server"].'_'.$data["old"]["source_username"].'.conf'); if(stristr($config_file_path,"..") || stristr($config_file_path,"|") || stristr($config_file_path,";") || stristr($config_file_path,'$')) { $app->log("Possibly faked path for getmail config file: '$config_file_path'. File is not written.",LOGLEVEL_ERROR); return false; } - if(is_file($config_file_path)) unlink($config_file_path); + if(is_file($config_file_path)) { + unlink($config_file_path); + $app->log("Deleting file: '$config_file_path'.",LOGLEVEL_DEBUG); + } else { + $app->log("Nothing to delete: '$config_file_path'.",LOGLEVEL_DEBUG); + } }