From dd4468664dc811618511f8fa83cacbf05b4d69b8 Mon Sep 17 00:00:00 2001
From: Sebastian Sdorra <sebastian.sdorra@cloudogu.com>
Date: Tue, 4 Jan 2011 17:17:48 +0100
Subject: [PATCH] improve user validation

---
 .../src/main/java/sonia/scm/user/xml/XmlUserManager.java     | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/scm-webapp/src/main/java/sonia/scm/user/xml/XmlUserManager.java b/scm-webapp/src/main/java/sonia/scm/user/xml/XmlUserManager.java
index 540dcf1b67..0863e12bab 100644
--- a/scm-webapp/src/main/java/sonia/scm/user/xml/XmlUserManager.java
+++ b/scm-webapp/src/main/java/sonia/scm/user/xml/XmlUserManager.java
@@ -52,6 +52,7 @@ import sonia.scm.user.AbstractUserManager;
 import sonia.scm.user.User;
 import sonia.scm.user.UserAllreadyExistException;
 import sonia.scm.user.UserException;
+import sonia.scm.util.AssertUtil;
 import sonia.scm.util.IOUtil;
 import sonia.scm.util.SecurityUtil;
 import sonia.scm.util.Util;
@@ -163,6 +164,8 @@ public class XmlUserManager extends AbstractUserManager
       throw new ScmSecurityException("admin account is required");
     }
 
+    AssertUtil.assertIsValid(user);
+
     if (userDB.contains(user.getName()))
     {
       throw new UserAllreadyExistException();
@@ -266,6 +269,8 @@ public class XmlUserManager extends AbstractUserManager
       throw new ScmSecurityException("admin account is required");
     }
 
+    AssertUtil.assertIsValid(user);
+
     String name = user.getName();
 
     if (userDB.contains(name))