Merge pull request #794 from wagnert/1.0

Change order of switching user/group to group/user because of necessa…
appserver-io · May 8, 2015 · 4a19232 · 4a19232
2 parents e1d05b6 + abc27e6
commit 4a19232
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/src/AppserverIo/Appserver/Core/Server.php b/src/AppserverIo/Appserver/Core/Server.php
@@ -597,18 +597,21 @@ protected function switchProcessUser()
             }
         }
 
-        // As we should only change user and group AFTER we made all chown and chgrp changes we will do it here
-        // after collecting if we are able to
-        if ($userChangeable) {
+        // As we should only change user and group AFTER we made all chown and chgrp
+        // changes we will do it here after collecting if we are able to.
 
-            // change the user ID
-            posix_setuid($userId);
-        }
+        // ATTENTION: We first need to change the group, because we need to be root
+        //            to do that. After that we can change the user also!!!!!!!!!!!
         if ($groupChangeable) {
 
             // change the group ID
             posix_setgid($groupId);
         }
+        if ($userChangeable) {
+
+            // change the user ID
+            posix_setuid($userId);
+        }
 
         // log a message with the time needed for restart
         $this->getSystemLogger()->info(