From d87f76019fc231ec20d95126a7fee0487e7be5f0 Mon Sep 17 00:00:00 2001 From: tbrehm <t.brehm@ispconfig.org> Date: Tue, 14 Aug 2012 10:56:20 -0400 Subject: [PATCH] - Added new web folder named private to web folder layout. The folder is intended to store data that shall not be visible in the web directory, it is owned by the user of the web. - Changed ownership of web root directory to root user in all security modes to prevent symlink attacks. - Apache log files are now owned by user root. - Improved functions in system library. --- server/plugins-available/network_settings_plugin.inc.php | 161 ++++++++++++++++++++++++++++------------------------- 1 files changed, 85 insertions(+), 76 deletions(-) diff --git a/server/plugins-available/network_settings_plugin.inc.php b/server/plugins-available/network_settings_plugin.inc.php index 961e2cd..b711139 100644 --- a/server/plugins-available/network_settings_plugin.inc.php +++ b/server/plugins-available/network_settings_plugin.inc.php @@ -76,11 +76,12 @@ global $app, $conf; // get the config - $app->uses("getconf"); - $server_config = $app->getconf->get_server_config($conf["server_id"], 'server'); + $app->uses('getconf'); + $server_config = $app->getconf->get_server_config($conf['server_id'], 'server'); - // Configure the debian network card settings - if($server_config['auto_network_configuration'] == 'y') { + //* Configure the debian network card settings. + //* Dont configure + if($server_config['auto_network_configuration'] == 'y' && $data['mirrored'] == false) { if (is_file('/etc/debian_version')) { @@ -89,15 +90,15 @@ $app->load('tpl'); $network_tpl = new tpl(); - $network_tpl->newTemplate("debian_network_interfaces.master"); + $network_tpl->newTemplate('debian_network_interfaces.master'); - $network_tpl->setVar('ip_address',$server_config["ip_address"]); - $network_tpl->setVar('netmask',$server_config["netmask"]); - $network_tpl->setVar('gateway',$server_config["gateway"]); - $network_tpl->setVar('broadcast',$this->broadcast($server_config["ip_address"],$server_config["netmask"])); - $network_tpl->setVar('network',$this->network($server_config["ip_address"],$server_config["netmask"])); + $network_tpl->setVar('ip_address',$server_config['ip_address']); + $network_tpl->setVar('netmask',$server_config['netmask']); + $network_tpl->setVar('gateway',$server_config['gateway']); + $network_tpl->setVar('broadcast',$this->broadcast($server_config['ip_address'],$server_config['netmask'])); + $network_tpl->setVar('network',$this->network($server_config['ip_address'],$server_config['netmask'])); - $records = $app->db->queryAllRecords("SELECT ip_address FROM server_ip WHERE server_id = ".intval($conf["server_id"]) . " order by ip_address"); + $records = $app->db->queryAllRecords("SELECT ip_address FROM server_ip WHERE server_id = ".intval($conf['server_id']) . ' ORDER BY server_ip_id ASC'); $ip_records = array(); $additionl_ip_records = 0; $n = 0; @@ -106,15 +107,15 @@ /* * don't insert the main-ip again! */ - if ($rec['ip_address'] != $server_config["ip_address"]) + if ($rec['ip_address'] != $server_config['ip_address']) { $ip_records[$n] = array( 'id' => $n, 'ip_address' => $rec['ip_address'], - 'netmask' => $server_config["netmask"], - 'gateway' => $server_config["gateway"], - 'broadcast' => $this->broadcast($rec['ip_address'],$server_config["netmask"]), - 'network' => $this->network($rec['ip_address'],$server_config["netmask"]) + 'netmask' => $server_config['netmask'], + 'gateway' => $server_config['gateway'], + 'broadcast' => $this->broadcast($rec['ip_address'],$server_config['netmask']), + 'network' => $this->network($rec['ip_address'],$server_config['netmask']) ); $additionl_ip_records = 1; $n++; @@ -129,33 +130,36 @@ * main-server) because the LAST IP in the list is the IP mysql uses * to determine the host, the user is logging in from. */ + /* + // Disabled this part as it causes problems on multiserver setups if ($additionl_ip_records != 0) { - $swap["ip_address"] = $ip_records[$n-1]["ip_address"]; - $swap["netmask"] = $ip_records[$n-1]["netmask"]; - $swap["gateway"] = $ip_records[$n-1]["gateway"]; + $swap['ip_address'] = $ip_records[$n-1]['ip_address']; + $swap['netmask'] = $ip_records[$n-1]['netmask']; + $swap['gateway'] = $ip_records[$n-1]['gateway']; $ip_records[$n-1] = array( 'id' => $n-1, 'ip_address' => $server_config['ip_address'], - 'netmask' => $server_config["netmask"], - 'gateway' => $server_config["gateway"], - 'broadcast' => $this->broadcast($server_config['ip_address'],$server_config["netmask"]), - 'network' => $this->network($server_config['ip_address'],$server_config["netmask"]) + 'netmask' => $server_config['netmask'], + 'gateway' => $server_config['gateway'], + 'broadcast' => $this->broadcast($server_config['ip_address'],$server_config['netmask']), + 'network' => $this->network($server_config['ip_address'],$server_config['netmask']) ); - $network_tpl->setVar('ip_address',$swap["ip_address"]); - $network_tpl->setVar('netmask',$swap["netmask"]); - $network_tpl->setVar('gateway',$swap["gateway"]); - $network_tpl->setVar('broadcast',$this->broadcast($swap["ip_address"],$swap["netmask"])); - $network_tpl->setVar('network',$this->network($swap["ip_address"],$swap["netmask"])); + $network_tpl->setVar('ip_address',$swap['ip_address']); + $network_tpl->setVar('netmask',$swap['netmask']); + $network_tpl->setVar('gateway',$swap['gateway']); + $network_tpl->setVar('broadcast',$this->broadcast($swap['ip_address'],$swap['netmask'])); + $network_tpl->setVar('network',$this->network($swap['ip_address'],$swap['netmask'])); } + */ $network_tpl->setVar('additionl_ip_records',$additionl_ip_records); $network_tpl->setLoop('interfaces',$ip_records); file_put_contents('/etc/network/interfaces',$network_tpl->grab()); unset($network_tpl); - $app->log("Changed Network settings",LOGLEVEL_DEBUG); + $app->log('Changed Network settings',LOGLEVEL_DEBUG); exec($conf['init_scripts'] . '/' . 'networking force-reload'); } elseif (is_file('/etc/gentoo-release')) @@ -165,14 +169,14 @@ $app->load('tpl'); $network_tpl = new tpl(); - $network_tpl->newTemplate("gentoo_network_interfaces.master"); + $network_tpl->newTemplate('gentoo_network_interfaces.master'); - $network_tpl->setVar('ip_address',$server_config["ip_address"]); - $network_tpl->setVar('netmask',$server_config["netmask"]); - $network_tpl->setVar('gateway',$server_config["gateway"]); - $network_tpl->setVar('broadcast',$this->broadcast($server_config["ip_address"],$server_config["netmask"])); + $network_tpl->setVar('ip_address',$server_config['ip_address']); + $network_tpl->setVar('netmask',$server_config['netmask']); + $network_tpl->setVar('gateway',$server_config['gateway']); + $network_tpl->setVar('broadcast',$this->broadcast($server_config['ip_address'],$server_config['netmask'])); - $records = $app->db->queryAllRecords("SELECT ip_address FROM server_ip WHERE server_id = ".intval($conf["server_id"]) . " order by ip_address"); + $records = $app->db->queryAllRecords("SELECT ip_address FROM server_ip WHERE server_id = ".intval($conf['server_id']) . " order by ip_address"); $ip_records = array(); $additionl_ip_records = 0; $n = 0; @@ -181,14 +185,14 @@ /* * don't insert the main-ip again! */ - if ($rec['ip_address'] != $server_config["ip_address"]) + if ($rec['ip_address'] != $server_config['ip_address']) { $ip_records[$n] = array( 'id' => $n, 'ip_address' => $rec['ip_address'], - 'netmask' => $server_config["netmask"], - 'gateway' => $server_config["gateway"], - 'broadcast' => $this->broadcast($rec['ip_address'],$server_config["netmask"]) + 'netmask' => $server_config['netmask'], + 'gateway' => $server_config['gateway'], + 'broadcast' => $this->broadcast($rec['ip_address'],$server_config['netmask']) ); $additionl_ip_records = 1; $n++; @@ -205,21 +209,21 @@ */ if ($additionl_ip_records != 0) { - $swap["ip_address"] = $ip_records[$n-1]["ip_address"]; - $swap["netmask"] = $ip_records[$n-1]["netmask"]; - $swap["gateway"] = $ip_records[$n-1]["gateway"]; + $swap['ip_address'] = $ip_records[$n-1]['ip_address']; + $swap['netmask'] = $ip_records[$n-1]['netmask']; + $swap['gateway'] = $ip_records[$n-1]['gateway']; $ip_records[$n-1] = array( 'id' => $n-1, 'ip_address' => $server_config['ip_address'], - 'netmask' => $server_config["netmask"], - 'gateway' => $server_config["gateway"], - 'broadcast' => $this->broadcast($server_config['ip_address'],$server_config["netmask"]) + 'netmask' => $server_config['netmask'], + 'gateway' => $server_config['gateway'], + 'broadcast' => $this->broadcast($server_config['ip_address'],$server_config['netmask']) ); - $network_tpl->setVar('ip_address',$swap["ip_address"]); - $network_tpl->setVar('netmask',$swap["netmask"]); - $network_tpl->setVar('gateway',$swap["gateway"]); - $network_tpl->setVar('broadcast',$this->broadcast($swap["ip_address"],$swap["netmask"])); + $network_tpl->setVar('ip_address',$swap['ip_address']); + $network_tpl->setVar('netmask',$swap['netmask']); + $network_tpl->setVar('gateway',$swap['gateway']); + $network_tpl->setVar('broadcast',$this->broadcast($swap['ip_address'],$swap['netmask'])); } $network_tpl->setVar('additionl_ip_records',$additionl_ip_records); @@ -227,59 +231,64 @@ file_put_contents('/etc/conf.d/net',$network_tpl->grab()); unset($network_tpl); - $app->log("Changed Network settings",LOGLEVEL_DEBUG); + $app->log('Changed Network settings',LOGLEVEL_DEBUG); exec($conf['init_scripts'] . '/' . 'net.eth0 restart'); } else { - $app->log("Network configuration not available for this Linux distribution.",LOGLEVEL_DEBUG); + $app->log('Network configuration not available for this Linux distribution.',LOGLEVEL_DEBUG); } } else { - $app->log("Network configuration disabled in server settings.",LOGLEVEL_WARN); + if($data['mirrored'] == true) { + $app->log('Skipping network config request. IP addresses from amster are not configured on the mirror.',LOGLEVEL_DEBUG); + } + if($server_config['auto_network_configuration'] == 'n') { + $app->log('Network configuration disabled in server settings.',LOGLEVEL_DEBUG); + } } } function network($ip, $netmask){ $netmask = $this->netmask($netmask); - list($f1,$f2,$f3,$f4) = explode(".", $netmask); - $netmask_bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT); - list($f1,$f2,$f3,$f4) = explode(".", $ip); - $ip_bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT); + list($f1,$f2,$f3,$f4) = explode('.', $netmask); + $netmask_bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT); + list($f1,$f2,$f3,$f4) = explode('.', $ip); + $ip_bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT); for($i=0;$i<32;$i++){ $network_bin .= substr($netmask_bin,$i,1) * substr($ip_bin,$i,1); } - $network_bin = wordwrap($network_bin, 8, ".", 1); - list($f1,$f2,$f3,$f4) = explode(".", trim($network_bin)); - return bindec($f1).".".bindec($f2).".".bindec($f3).".".bindec($f4); + $network_bin = wordwrap($network_bin, 8, '.', 1); + list($f1,$f2,$f3,$f4) = explode('.', trim($network_bin)); + return bindec($f1).'.'.bindec($f2).'.'.bindec($f3).'.'.bindec($f4); } function broadcast($ip, $netmask){ $netmask = $this->netmask($netmask); $binary_netmask = $this->binary_netmask($netmask); - list($f1,$f2,$f3,$f4) = explode(".", $ip); - $ip_bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT); - $broadcast_bin = str_pad(substr($ip_bin, 0, $binary_netmask),32,"1",STR_PAD_RIGHT); - $broadcast_bin = wordwrap($broadcast_bin, 8, ".", 1); - list($f1,$f2,$f3,$f4) = explode(".", trim($broadcast_bin)); - return bindec($f1).".".bindec($f2).".".bindec($f3).".".bindec($f4); + list($f1,$f2,$f3,$f4) = explode('.', $ip); + $ip_bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT); + $broadcast_bin = str_pad(substr($ip_bin, 0, $binary_netmask),32,'1',STR_PAD_RIGHT); + $broadcast_bin = wordwrap($broadcast_bin, 8, '.', 1); + list($f1,$f2,$f3,$f4) = explode('.', trim($broadcast_bin)); + return bindec($f1).'.'.bindec($f2).'.'.bindec($f3).'.'.bindec($f4); } function netmask($netmask){ - list($f1,$f2,$f3,$f4) = explode(".", trim($netmask)); - $bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT); - $parts = explode("0", $bin); - $bin = str_pad($parts[0], 32, "0", STR_PAD_RIGHT); - $bin = wordwrap($bin, 8, ".", 1); - list($f1,$f2,$f3,$f4) = explode(".", trim($bin)); - return bindec($f1).".".bindec($f2).".".bindec($f3).".".bindec($f4); + list($f1,$f2,$f3,$f4) = explode('.', trim($netmask)); + $bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT); + $parts = explode('0', $bin); + $bin = str_pad($parts[0], 32, '0', STR_PAD_RIGHT); + $bin = wordwrap($bin, 8, '.', 1); + list($f1,$f2,$f3,$f4) = explode('.', trim($bin)); + return bindec($f1).'.'.bindec($f2).'.'.bindec($f3).'.'.bindec($f4); } function binary_netmask($netmask){ - list($f1,$f2,$f3,$f4) = explode(".", trim($netmask)); - $bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT); - $parts = explode("0", $bin); - return substr_count($parts[0], "1"); + list($f1,$f2,$f3,$f4) = explode('.', trim($netmask)); + $bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT); + $parts = explode('0', $bin); + return substr_count($parts[0], '1'); } } // end class -- Gitblit v1.9.1