From b6ff2e1bd5c4dd8ecd8fad575ca201a0cd5cc8dd Mon Sep 17 00:00:00 2001
From: tbrehm <t.brehm@ispconfig.org>
Date: Thu, 14 Jan 2010 09:42:56 -0500
Subject: [PATCH] Fixed: FS#907 - possible security issue deleting maildir .. plugin module [ mail_plugin.inc.php ]

---
 server/plugins-available/shelluser_jailkit_plugin.inc.php |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/server/plugins-available/shelluser_jailkit_plugin.inc.php b/server/plugins-available/shelluser_jailkit_plugin.inc.php
index b915e9f..60524ba 100755
--- a/server/plugins-available/shelluser_jailkit_plugin.inc.php
+++ b/server/plugins-available/shelluser_jailkit_plugin.inc.php
@@ -85,15 +85,15 @@
 				$this->app = $app;
 				$this->jailkit_config = $app->getconf->get_server_config($conf["server_id"], 'jailkit');
 				
-				$command .= 'usermod -U '.escapeshellcmd($data['new']['username']);
-				exec($command);
-				
 				$this->_update_website_security_level();
-				
-				$this->_add_jailkit_user();
 			
 				$this->_setup_jailkit_chroot();
 				
+				$this->_add_jailkit_user();
+				
+				$command .= 'usermod -U '.escapeshellcmd($data['new']['username']);
+				exec($command);
+				
 				$this->_update_website_security_level();
 			}
 		

--
Gitblit v1.9.1