From 4569cae57f127afd093794310ccd290d2d9fdf36 Mon Sep 17 00:00:00 2001
From: Marius Burkard <m.burkard@pixcept.de>
Date: Wed, 20 Apr 2016 10:58:46 -0400
Subject: [PATCH] Merge branch 'stable-3.1'

---
 server/plugins-available/shelluser_base_plugin.inc.php |   34 +++++++++++++++++++++-------------
 1 files changed, 21 insertions(+), 13 deletions(-)

diff --git a/server/plugins-available/shelluser_base_plugin.inc.php b/server/plugins-available/shelluser_base_plugin.inc.php
index 8a51b10..71275d2 100755
--- a/server/plugins-available/shelluser_base_plugin.inc.php
+++ b/server/plugins-available/shelluser_base_plugin.inc.php
@@ -113,16 +113,20 @@
 					$homedir = $data['new']['dir'].'/home/'.$data['new']['username'];
 				}
 				
+				// Create home base directory if it does not exist
 				if(!is_dir($data['new']['dir'].'/home')){
-					$app->file->mkdirs(escapeshellcmd($data['new']['dir'].'/home'), '0750');
-					$app->system->chown(escapeshellcmd($data['new']['dir'].'/home'),escapeshellcmd($data['new']['puser']));
-					$app->system->chgrp(escapeshellcmd($data['new']['dir'].'/home'),escapeshellcmd($data['new']['pgroup']));
+					$app->file->mkdirs(escapeshellcmd($data['new']['dir'].'/home'), '0755');
 				}
+				
+				// Change ownership of home base dir to root user
+				$app->system->chown(escapeshellcmd($data['new']['dir'].'/home'),'root');
+				$app->system->chgrp(escapeshellcmd($data['new']['dir'].'/home'),'root');
+				$app->system->chmod(escapeshellcmd($data['new']['dir'].'/home'),0755);
 				
 				if(!is_dir($homedir)){
 					$app->file->mkdirs(escapeshellcmd($homedir), '0750');
-					$app->system->chown(escapeshellcmd($homedir),escapeshellcmd($data['new']['puser']));
-					$app->system->chgrp(escapeshellcmd($homedir),escapeshellcmd($data['new']['pgroup']));
+					$app->system->chown(escapeshellcmd($homedir),escapeshellcmd($data['new']['puser']),false);
+					$app->system->chgrp(escapeshellcmd($homedir),escapeshellcmd($data['new']['pgroup']),false);
 				}
 				$command = 'useradd';
 				$command .= ' -d '.escapeshellcmd($homedir);
@@ -136,6 +140,10 @@
 				exec($command);
 				$app->log("Executed command: ".$command, LOGLEVEL_DEBUG);
 				$app->log("Added shelluser: ".$data['new']['username'], LOGLEVEL_DEBUG);
+				
+				$app->system->chown(escapeshellcmd($data['new']['dir']),escapeshellcmd($data['new']['username']),false);
+				$app->system->chgrp(escapeshellcmd($data['new']['dir']),escapeshellcmd($data['new']['pgroup']),false);
+				
 
 				// call the ssh-rsa update function
 				$app->uses("getconf");
@@ -145,7 +153,7 @@
 
 				//* Create .bash_history file
 				$app->system->touch(escapeshellcmd($homedir).'/.bash_history');
-				$app->system->chmod(escapeshellcmd($homedir).'/.bash_history', 0755);
+				$app->system->chmod(escapeshellcmd($homedir).'/.bash_history', 0750);
 				$app->system->chown(escapeshellcmd($homedir).'/.bash_history', $data['new']['username']);
 				$app->system->chgrp(escapeshellcmd($homedir).'/.bash_history', $data['new']['pgroup']);
 
@@ -245,17 +253,17 @@
 							$app->system->chown(escapeshellcmd($data['new']['dir'].'/home'),escapeshellcmd($data['new']['puser']));
 							$app->system->chgrp(escapeshellcmd($data['new']['dir'].'/home'),escapeshellcmd($data['new']['pgroup']));
 						}
-						$app->file->mkdirs(escapeshellcmd($homedir), '0750');
-						$app->system->chown(escapeshellcmd($homedir),escapeshellcmd($data['new']['puser']));
-						$app->system->chgrp(escapeshellcmd($homedir),escapeshellcmd($data['new']['pgroup']));
+						$app->file->mkdirs(escapeshellcmd($homedir), '0755');
+						$app->system->chown(escapeshellcmd($homedir),'root');
+						$app->system->chgrp(escapeshellcmd($homedir),'root');
 						$app->system->web_folder_protection($web['document_root'], true);
 					} else {
 						if(!is_dir($homedir)){
 							$app->system->web_folder_protection($web['document_root'], false);
 							if(!is_dir($data['new']['dir'].'/home')){
-								$app->file->mkdirs(escapeshellcmd($data['new']['dir'].'/home'), '0750');
-								$app->system->chown(escapeshellcmd($data['new']['dir'].'/home'),escapeshellcmd($data['new']['puser']));
-								$app->system->chgrp(escapeshellcmd($data['new']['dir'].'/home'),escapeshellcmd($data['new']['pgroup']));
+								$app->file->mkdirs(escapeshellcmd($data['new']['dir'].'/home'), '0755');
+								$app->system->chown(escapeshellcmd($data['new']['dir'].'/home'),'root');
+								$app->system->chgrp(escapeshellcmd($data['new']['dir'].'/home'),'root');
 							}
 							$app->file->mkdirs(escapeshellcmd($homedir), '0750');
 							$app->system->chown(escapeshellcmd($homedir),escapeshellcmd($data['new']['puser']));
@@ -275,7 +283,7 @@
 					//* Create .bash_history file
 					if(!is_file($data['new']['dir']).'/.bash_history') {
 						$app->system->touch(escapeshellcmd($homedir).'/.bash_history');
-						$app->system->chmod(escapeshellcmd($homedir).'/.bash_history', 0755);
+						$app->system->chmod(escapeshellcmd($homedir).'/.bash_history', 0750);
 						$app->system->chown(escapeshellcmd($homedir).'/.bash_history', escapeshellcmd($data['new']['username']));
 						$app->system->chgrp(escapeshellcmd($homedir).'/.bash_history', escapeshellcmd($data['new']['pgroup']));
 					}

--
Gitblit v1.9.1