Added check for permissions in UpdateManager
This commit is contained in:
parent
4723404ff1
commit
b59292ca57
@ -210,6 +210,8 @@ class UpdateManager implements CallbackListener, CommandListener {
|
||||
$this->maniaControl->chat->sendError('Update is currently not possible!', $player->login);
|
||||
return;
|
||||
}
|
||||
|
||||
if($this->checkPermissions($player)) {
|
||||
$this->maniaControl->chat->sendInformation("Starting Update to Version v{$updateData->version}...", $player->login);
|
||||
$this->maniaControl->log("Starting Update to Version v{$updateData->version}...");
|
||||
$performBackup = $this->maniaControl->settingManager->getSetting($this, self::SETTING_PERFORM_BACKUPS);
|
||||
@ -225,6 +227,44 @@ class UpdateManager implements CallbackListener, CommandListener {
|
||||
|
||||
$this->maniaControl->restart();
|
||||
}
|
||||
}
|
||||
|
||||
private function checkPermissions(Player $player) {
|
||||
$writableDirectories = array('core/', 'plugins/');
|
||||
$readableDirectories = array('configs/');
|
||||
$ignore = array('.', '..');
|
||||
$path = str_replace('core', '', realpath(dirname(__FILE__)));
|
||||
|
||||
try {
|
||||
foreach($writableDirectories as $writableDirecotry) {
|
||||
$files = scandir($path.$writableDirecotry);
|
||||
foreach($files as $file) {
|
||||
if(!in_array($file, $ignore)) {
|
||||
if(!is_writable($path.$writableDirecotry.$file)) {
|
||||
throw new \Exception('"'.$path.$writableDirecotry.$file.'" is not writable!');
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
foreach($readableDirectories as $readableDirectory) {
|
||||
$files = scandir($path.$readableDirectory);
|
||||
foreach($files as $file) {
|
||||
if(!in_array($file, $ignore)) {
|
||||
if(!is_readable($path.$readableDirectory.$file)) {
|
||||
throw new \Exception('"'.$path.$readableDirectory.$file.'" is not readable!');
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
} catch(\Exception $e) {
|
||||
$this->maniaControl->log('Cannot update: '.$e->getMessage());
|
||||
$this->maniaControl->chat->sendError('Cannot update: '.$e->getMessage(), $player->login);
|
||||
return false;
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
/**
|
||||
* Check given Plugin Class for Update
|
||||
|
Loading…
Reference in New Issue
Block a user