From e94a9fb5e4b6a2bc07e0bb4cf8ea35fc70c4bbf0 Mon Sep 17 00:00:00 2001 From: tbrehm <t.brehm@ispconfig.org> Date: Thu, 10 May 2012 12:31:27 -0400 Subject: [PATCH] Merged revisions 3049-3051,3053-3054,3058,3070,3074-3077,3082,3086-3088,3091-3092,3094-3095 from stable branch. --- server/plugins-available/shelluser_base_plugin.inc.php | 149 +++++++++++++++++++++++++++++++++++++++++++++---- 1 files changed, 137 insertions(+), 12 deletions(-) diff --git a/server/plugins-available/shelluser_base_plugin.inc.php b/server/plugins-available/shelluser_base_plugin.inc.php index 3f86b42..5c41863 100755 --- a/server/plugins-available/shelluser_base_plugin.inc.php +++ b/server/plugins-available/shelluser_base_plugin.inc.php @@ -72,6 +72,13 @@ $app->uses('system'); + //* Check if the resulting path is inside the docroot + $web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ".intval($data['new']['parent_domain_id'])); + if(substr(realpath($data['new']['dir']),0,strlen($web['document_root'])) != $web['document_root']) { + $app->log('Directory of the shell user is outside of website docroot.',LOGLEVEL_WARN); + return false; + } + if($app->system->is_user($data['new']['puser'])) { // Get the UID of the parent user $uid = intval($app->system->getuid($data['new']['puser'])); @@ -88,15 +95,22 @@ exec($command); $app->log("Executed command: ".$command,LOGLEVEL_DEBUG); $app->log("Added shelluser: ".$data['new']['username'],LOGLEVEL_DEBUG); + + // call the ssh-rsa update function + $app->uses("getconf"); + $this->data = $data; + $this->app = $app; + $this->_setup_ssh_rsa(); //* Create .bash_history file - exec('touch '.escapeshellcmd($data['new']['dir']).'/.bash_history'); - exec('chmod 755 '.escapeshellcmd($data['new']['dir']).'/.bash_history'); - exec('chown '.escapeshellcmd($data['new']['username']).':'.escapeshellcmd($data['new']['pgroup']).' '.escapeshellcmd($data['new']['dir']).'/.bash_history'); + touch(escapeshellcmd($data['new']['dir']).'/.bash_history'); + chmod(escapeshellcmd($data['new']['dir']).'/.bash_history', 0755); + chown(escapeshellcmd($data['new']['dir']).'/.bash_history', escapeshellcmd($data['new']['username'])); + chgrp(escapeshellcmd($data['new']['dir']).'/.bash_history', escapeshellcmd($data['new']['pgroup'])); //* Disable shell user temporarily if we use jailkit if($data['new']['chroot'] == 'jailkit') { - $command = 'usermod -L '.escapeshellcmd($data['new']['username']); + $command = 'usermod -s /bin/false -L '.escapeshellcmd($data['new']['username']); exec($command); $app->log("Disabling shelluser temporarily: ".$command,LOGLEVEL_DEBUG); } @@ -105,7 +119,7 @@ $app->log("UID = $uid for shelluser:".$data['new']['username']." not allowed.",LOGLEVEL_ERROR); } } else { - $app->log("Skippung insert of user:".$data['new']['username'].", parent user ".$data['new']['puser']." does not exist.",LOGLEVEL_WARN); + $app->log("Skipping insertion of user:".$data['new']['username'].", parent user ".$data['new']['puser']." does not exist.",LOGLEVEL_WARN); } } @@ -113,6 +127,13 @@ global $app, $conf; $app->uses('system'); + + //* Check if the resulting path is inside the docroot + $web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ".intval($data['new']['parent_domain_id'])); + if(substr(realpath($data['new']['dir']),0,strlen($web['document_root'])) != $web['document_root']) { + $app->log('Directory of the shell user is outside of website docroot.',LOGLEVEL_WARN); + return false; + } if($app->system->is_user($data['new']['puser'])) { // Get the UID of the parent user @@ -133,13 +154,19 @@ exec($command); $app->log("Executed command: $command ",LOGLEVEL_DEBUG); $app->log("Updated shelluser: ".$data['old']['username'],LOGLEVEL_DEBUG); - + + // call the ssh-rsa update function + $app->uses("getconf"); + $this->data = $data; + $this->app = $app; + $this->_setup_ssh_rsa(); //* Create .bash_history file if(!is_file($data['new']['dir']).'/.bash_history') { - exec('touch '.escapeshellcmd($data['new']['dir']).'/.bash_history'); - exec('chmod 755 '.escapeshellcmd($data['new']['dir']).'/.bash_history'); - exec('chown '.escapeshellcmd($data['new']['username']).':'.escapeshellcmd($data['new']['pgroup']).' '.escapeshellcmd($data['new']['dir']).'/.bash_history'); + touch(escapeshellcmd($data['new']['dir']).'/.bash_history'); + chmod(escapeshellcmd($data['new']['dir']).'/.bash_history', 0755); + chown(escapeshellcmd($data['new']['dir']).'/.bash_history',escapeshellcmd($data['new']['username'])); + chgrp(escapeshellcmd($data['new']['dir']).'/.bash_history',escapeshellcmd($data['new']['pgroup'])); } } else { @@ -150,7 +177,7 @@ $app->log("UID = $uid for shelluser:".$data['new']['username']." not allowed.",LOGLEVEL_ERROR); } } else { - $app->log("Skippung update for user:".$data['new']['username'].", parent user ".$data['new']['puser']." does not exist.",LOGLEVEL_WARN); + $app->log("Skipping update for user:".$data['new']['username'].", parent user ".$data['new']['puser']." does not exist.",LOGLEVEL_WARN); } } @@ -181,9 +208,107 @@ } - + private function _setup_ssh_rsa() { + global $app; + $this->app->log("ssh-rsa setup shelluser_base",LOGLEVEL_DEBUG); + // Get the client ID, username, and the key + $domain_data = $this->app->db->queryOneRecord('SELECT sys_groupid FROM web_domain WHERE web_domain.domain_id = '.intval($this->data['new']['parent_domain_id'])); + $sys_group_data = $this->app->db->queryOneRecord('SELECT * FROM sys_group WHERE sys_group.groupid = '.intval($domain_data['sys_groupid'])); + $id = intval($sys_group_data['client_id']); + $username= $sys_group_data['name']; + $client_data = $this->app->db->queryOneRecord('SELECT * FROM client WHERE client.client_id = '.$id); + $userkey = $client_data['ssh_rsa']; + unset($domain_data); + unset($client_data); + + // ssh-rsa authentication variables + $sshrsa = $this->data['new']['ssh_rsa']; + $usrdir = escapeshellcmd($this->data['new']['dir']); + $sshdir = $usrdir.'/.ssh'; + $sshkeys= $usrdir.'/.ssh/authorized_keys'; + + $app->uses('file'); + $sshrsa = $app->file->unix_nl($sshrsa); + $sshrsa = $app->file->remove_blank_lines($sshrsa,0); + + // If this user has no key yet, generate a pair + if ($userkey == '' && $id > 0){ + //Generate ssh-rsa-keys + exec('ssh-keygen -t rsa -C '.$username.'-rsa-key-'.time().' -f /tmp/id_rsa -N ""'); + + // use the public key that has been generated + $userkey = file_get_contents('/tmp/id_rsa.pub'); + + // save keypair in client table + $this->app->db->query("UPDATE client SET created_at = ".time().", id_rsa = '".file_get_contents('/tmp/id_rsa')."', ssh_rsa = '".$userkey."' WHERE client_id = ".$id); + + exec('rm -f /tmp/id_rsa /tmp/id_rsa.pub'); + $this->app->log("ssh-rsa keypair generated for ".$username,LOGLEVEL_DEBUG); + }; + + if (!file_exists($sshkeys)){ + // add root's key + $app->file->mkdirs($sshdir, '0700'); + if(is_file('/root/.ssh/authorized_keys')) file_put_contents($sshkeys, file_get_contents('/root/.ssh/authorized_keys')); + + // Remove duplicate keys + $existing_keys = @file($sshkeys); + $new_keys = explode("\n", $userkey); + $final_keys_arr = @array_merge($existing_keys, $new_keys); + $new_final_keys_arr = array(); + if(is_array($final_keys_arr) && !empty($final_keys_arr)){ + foreach($final_keys_arr as $key => $val){ + $new_final_keys_arr[$key] = trim($val); + } + } + $final_keys = implode("\n", array_flip(array_flip($new_final_keys_arr))); + + // add the user's key + file_put_contents($sshkeys, $final_keys); + $app->file->remove_blank_lines($sshkeys); + $this->app->log("ssh-rsa authorisation keyfile created in ".$sshkeys,LOGLEVEL_DEBUG); + } + + //* Get the keys + $existing_keys = file($sshkeys); + $new_keys = explode("\n", $sshrsa); + $old_keys = explode("\n",$this->data['old']['ssh_rsa']); + + //* Remove all old keys + if(is_array($old_keys)) { + foreach($old_keys as $key => $val) { + $k = array_search(trim($val),$existing_keys); + unset($existing_keys[$k]); + } + } + + //* merge the remaining keys and the ones fom the ispconfig database. + if(is_array($new_keys)) { + $final_keys_arr = array_merge($existing_keys, $new_keys); + } else { + $final_keys_arr = $existing_keys; + } + + $new_final_keys_arr = array(); + if(is_array($final_keys_arr) && !empty($final_keys_arr)){ + foreach($final_keys_arr as $key => $val){ + $new_final_keys_arr[$key] = trim($val); + } + } + $final_keys = implode("\n", array_flip(array_flip($new_final_keys_arr))); + + // add the custom key + file_put_contents($sshkeys, $final_keys); + $app->file->remove_blank_lines($sshkeys); + $this->app->log("ssh-rsa key updated in ".$sshkeys,LOGLEVEL_DEBUG); + + // set proper file permissions + exec("chown -R ".escapeshellcmd($this->data['new']['puser']).":".escapeshellcmd($this->data['new']['pgroup'])." ".$sshdir); + exec("chmod 600 '$sshkeys'"); + + } } // end class -?> \ No newline at end of file +?> -- Gitblit v1.9.1