LDAP: fix generation of alternate internal name on conflicts. Use also smaller number...

diff --git a/apps/user_ldap/lib/access.php b/apps/user_ldap/lib/access.php
index 8c372766c006dce65e2554a5c03a46494ce1dc04..a6029445bd1c96ef4f7f2ec8bc9cfb95602bd8a2 100644 (file)
--- a/apps/user_ldap/lib/access.php
+++ b/apps/user_ldap/lib/access.php
@@ -441,8 +441,8 @@ abstract class Access {
                //while loop is just a precaution. If a name is not generated within
                //20 attempts, something else is very wrong. Avoids infinite loop.
                while($attempts < 20){
-                       $altName = $name . '_' . uniqid();
-                       if(\OCP\User::userExists($altName)) {
+                       $altName = $name . '_' . rand(1000,9999);
+                       if(!\OCP\User::userExists($altName)) {
                                return $altName;
                        }
                        $attempts++;