From 0ae9a2c9fefb16be52362826c8981e4efe2fa878 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=B6rn=20Friedrich=20Dreyer?= <jfd@butonic.de>
Date: Thu, 29 Sep 2016 10:16:13 +0200
Subject: [PATCH] Prevent user with empty uid

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
---
 apps/user_ldap/lib/User/User.php | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/apps/user_ldap/lib/User/User.php b/apps/user_ldap/lib/User/User.php
index 3d247663b3f..51fd577104c 100644
--- a/apps/user_ldap/lib/User/User.php
+++ b/apps/user_ldap/lib/User/User.php
@@ -6,6 +6,7 @@
  * @author Joas Schilling <coding@schilljs.com>
  * @author Morris Jobke <hey@morrisjobke.de>
  * @author Thomas Müller <thomas.mueller@tmit.eu>
+ * @author Jörn Friedrich Dreyer <jfd@butonic.de>
  *
  * @license AGPL-3.0
  *
@@ -111,6 +112,12 @@ class User {
 		IConfig $config, FilesystemHelper $fs, Image $image,
 		LogWrapper $log, IAvatarManager $avatarManager, IUserManager $userManager) {
 
+		if ($username === null) {
+			throw new \InvalidArgumentException("uid for '$dn' must not be null!");
+		} else if ($username === '') {
+			throw new \InvalidArgumentException("uid for '$dn' must not be an empty string!");
+		}
+
 		$this->access        = $access;
 		$this->connection    = $access->getConnection();
 		$this->config        = $config;