From 7ab11dfd7ea900871933704eed103fa652d7a93d Mon Sep 17 00:00:00 2001
From: Serge Rider <serge@jkiss.org>
Date: Thu, 28 May 2020 15:35:08 +0300
Subject: [PATCH] Check for empty user/role name

---
 .../io/cloudbeaver/service/admin/impl/WebServiceAdmin.java  | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/server/bundles/io.cloudbeaver.service.admin/src/io/cloudbeaver/service/admin/impl/WebServiceAdmin.java b/server/bundles/io.cloudbeaver.service.admin/src/io/cloudbeaver/service/admin/impl/WebServiceAdmin.java
index 41a776db74..f9393dd1c9 100644
--- a/server/bundles/io.cloudbeaver.service.admin/src/io/cloudbeaver/service/admin/impl/WebServiceAdmin.java
+++ b/server/bundles/io.cloudbeaver.service.admin/src/io/cloudbeaver/service/admin/impl/WebServiceAdmin.java
@@ -90,6 +90,9 @@ public class WebServiceAdmin implements DBWServiceAdmin {
     @NotNull
     @Override
     public AdminUserInfo createUser(@NotNull WebSession webSession, String userName) throws DBWebException {
+        if (userName.isEmpty()) {
+            throw new DBWebException("Empty user name");
+        }
         try {
             WebUser newUser = new WebUser(userName);
             CBPlatform.getInstance().getApplication().getSecurityController().createUser(newUser);
@@ -112,6 +115,9 @@ public class WebServiceAdmin implements DBWServiceAdmin {
     @NotNull
     @Override
     public AdminRoleInfo createRole(@NotNull WebSession webSession, String roleId) throws DBWebException {
+        if (roleId.isEmpty()) {
+            throw new DBWebException("Empty role ID");
+        }
         try {
             WebRole newRole = new WebRole(roleId);
             CBPlatform.getInstance().getApplication().getSecurityController().createRole(newRole);