From 71c74b1b572f3bdf00ba99009cc23f8f48072a1c Mon Sep 17 00:00:00 2001 From: Till Brehm <tbrehm@ispconfig.org> Date: Sat, 08 Mar 2014 11:58:49 -0500 Subject: [PATCH] Fixed: FS#3372 - Web browser password autofill function overwrites settings --- interface/lib/classes/remoting.inc.php | 154 ++++++++++++++++++++++++++++++++++++++++++++++++--- 1 files changed, 145 insertions(+), 9 deletions(-) diff --git a/interface/lib/classes/remoting.inc.php b/interface/lib/classes/remoting.inc.php index cc76bc7..3fe307d 100644 --- a/interface/lib/classes/remoting.inc.php +++ b/interface/lib/classes/remoting.inc.php @@ -126,8 +126,11 @@ $session_id = $app->db->quote($session_id); $sql = "DELETE FROM remote_session WHERE remote_session = '$session_id'"; - $app->db->query($sql); - return $app->db->affectedRows() == 1; + if($app->db->query($sql) != false) { + return true; + } else { + return false; + } } @@ -136,20 +139,54 @@ @param int session id @param int server id @param string section of the config field in the server table. Could be 'web', 'dns', 'mail', 'dns', 'cron', etc - @author Julio Montoya <gugli100@gmail.com> BeezNest 2010 + @author Julio Montoya <gugli100@gmail.com> BeezNest 2010, extended by M. Cramer <m.cramer@pixcept.de> 2014 */ - public function server_get($session_id, $server_id, $section ='') { + public function server_get($session_id, $server_id = null, $section ='') { global $app; if(!$this->checkPerm($session_id, 'server_get')) { $this->server->fault('permission_denied', 'You do not have the permissions to access this function.'); return false; } - if (!empty($session_id) && !empty($server_id)) { + if (!empty($session_id)) { $app->uses('remoting_lib , getconf'); - $section_config = $app->getconf->get_server_config($server_id, $section); - return $section_config; + if(!empty($server_id)) { + $section_config = $app->getconf->get_server_config($server_id, $section); + return $section_config; + } else { + $servers = array(); + $sql = "SELECT server_id FROM server WHERE 1"; + $all = $app->db->queryAllRecords($sql); + foreach($all as $s) { + $servers[$s['server_id']] = $app->getconf->get_server_config($s['server_id'], $section); + } + unset($all); + unset($s); + return $servers; + } + } else { + return false; + } + } + + /** + Gets a list of all servers + @param int session_id + @param int server_name + @author Marius Cramer <m.cramer@pixcept.de> 2014 + */ + public function server_get_all($session_id) + { + global $app; + if(!$this->checkPerm($session_id, 'server_get')) { + $this->server->fault('permission_denied', 'You do not have the permissions to access this function.'); + return false; + } + if (!empty($session_id)) { + $sql = "SELECT server_id, server_name FROM server WHERE 1"; + $servers = $app->db->queryAllRecords($sql); + return $servers; } else { return false; } @@ -200,6 +237,69 @@ } /** + * set record permissions in any table + * @param string session_id + * @param string index_field + * @param string index_value + * @param array permissions + * @author "ispcomm", improved by M. Cramer <m.cramer@pixcept.de> + */ + public function update_record_permissions($tablename, $index_field, $index_value, $permissions) { + global $app; + + if(!$this->checkPerm($session_id, 'admin_record_permissions')) { + $this->server->fault('permission_denied', 'You do not have the permissions to access this function.'); + return false; + } + + foreach($permissions as $key => $value) { // make sure only sys_ fields are updated + switch($key) { + case 'sys_userid': + // check if userid is valid + $check = $app->db->queryOneRecord('SELECT userid FROM sys_user WHERE userid = ' . $app->functions->intval($value)); + if(!$check || !$check['userid']) { + $this->server->fault('invalid parameters', $value . ' is no valid sys_userid.'); + return false; + } + $permissions[$key] = $app->functions->intval($value); + break; + case 'sys_groupid': + // check if groupid is valid + $check = $app->db->queryOneRecord('SELECT groupid FROM sys_group WHERE groupid = ' . $app->functions->intval($value)); + if(!$check || !$check['groupid']) { + $this->server->fault('invalid parameters', $value . ' is no valid sys_groupid.'); + return false; + } + $permissions[$key] = $app->functions->intval($value); + break; + case 'sys_perm_user': + case 'sys_perm_group': + // check if permissions are valid + $value = strtolower($value); + if(!preg_match('/^[riud]+$/', $value)) { + $this->server->fault('invalid parameters', $value . ' is no valid permission string.'); + return false; + } + + $newvalue = ''; + if(strpos($value, 'r') !== false) $newvalue .= 'r'; + if(strpos($value, 'i') !== false) $newvalue .= 'i'; + if(strpos($value, 'u') !== false) $newvalue .= 'u'; + if(strpos($value, 'd') !== false) $newvalue .= 'd'; + $permissions[$key] = $newvalue; + unset($newvalue); + + break; + default: + $this->server->fault('invalid parameters', 'Only sys_userid, sys_groupid, sys_perm_user and sys_perm_group parameters can be changed with this function.'); + break; + } + } + + return $app->db->datalogUpdate( $tablename, $permissions, $index_field, $index_value ) ; + } + + /** Gets the ISPconfig version of the server @param int session_id @author Sascha Bay <info@space2place.de> TheCry 2013 @@ -231,6 +331,20 @@ return $all; } + //* Get server ips + public function server_ip_get($session_id, $primary_id) + { + global $app; + + if(!$this->checkPerm($session_id, 'server_ip_get')) { + $this->server->fault('permission_denied', 'You do not have the permissions to access this function.'); + return false; + } + $app->uses('remoting_lib'); + $app->remoting_lib->loadFormDef('../admin/form/server_ip.tform.php'); + return $app->remoting_lib->getDataRecord($primary_id); + } + //* Add a IP address record public function server_ip_add($session_id, $client_id, $params) { @@ -1649,7 +1763,18 @@ $this->dataRecord = $params; $app->sites_database_plugin->processDatabaseInsert($this); - return $this->insertQueryExecute($sql, $params); + $retval = $this->insertQueryExecute($sql, $params); + + // set correct values for backup_interval and backup_copies + if(isset($params['backup_interval']) || isset($params['backup_copies'])){ + $sql_set = array(); + if(isset($params['backup_interval'])) $sql_set[] = "backup_interval = '".$app->db->quote($params['backup_interval'])."'"; + if(isset($params['backup_copies'])) $sql_set[] = "backup_copies = ".$app->functions->intval($params['backup_copies']); + //$app->db->query("UPDATE web_database SET ".implode(', ', $sql_set)." WHERE database_id = ".$retval); + $this->updateQueryExecute("UPDATE web_database SET ".implode(', ', $sql_set)." WHERE database_id = ".$retval, $retval, $params); + } + + return $retval; } return false; @@ -1672,7 +1797,18 @@ $this->id = $primary_id; $this->dataRecord = $params; $app->sites_database_plugin->processDatabaseUpdate($this); - return $this->updateQueryExecute($sql, $primary_id, $params); + $retval = $this->updateQueryExecute($sql, $primary_id, $params); + + // set correct values for backup_interval and backup_copies + if(isset($params['backup_interval']) || isset($params['backup_copies'])){ + $sql_set = array(); + if(isset($params['backup_interval'])) $sql_set[] = "backup_interval = '".$app->db->quote($params['backup_interval'])."'"; + if(isset($params['backup_copies'])) $sql_set[] = "backup_copies = ".$app->functions->intval($params['backup_copies']); + //$app->db->query("UPDATE web_database SET ".implode(', ', $sql_set)." WHERE database_id = ".$primary_id); + $this->updateQueryExecute("UPDATE web_database SET ".implode(', ', $sql_set)." WHERE database_id = ".$primary_id, $primary_id, $params); + } + + return $retval; } return false; -- Gitblit v1.9.1