From: William Roush <william.roush@roushtech.net>
Date: Mon, 20 Apr 2015 05:28:19 +0000 (-0400)
Subject: Fixes issue with LDAP inserting users with blank names.
X-Git-Tag: v0.9.99~1397^2~4^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=8363c9dd0f2161d4ca34f4b1e3eb5e02cf738a31;p=gitea.git

Fixes issue with LDAP inserting users with blank names.
---

diff --git a/models/repo.go b/models/repo.go
index cc4b53b0f2..f144be5a3f 100644
--- a/models/repo.go
+++ b/models/repo.go
@@ -40,6 +40,7 @@ var (
 	ErrRepoFileNotLoaded = errors.New("Repository file not loaded")
 	ErrMirrorNotExist    = errors.New("Mirror does not exist")
 	ErrInvalidReference  = errors.New("Invalid reference specified")
+	ErrNameEmpty         = errors.New("Name is empty")
 )
 
 var (
@@ -259,7 +260,11 @@ var (
 
 // IsUsableName checks if name is reserved or pattern of name is not allowed.
 func IsUsableName(name string) error {
-	name = strings.ToLower(name)
+	name = strings.TrimSpace(strings.ToLower(name))
+	if utf8.RuneCountInString(name) == 0 {
+		return ErrNameEmpty
+	}
+
 	for i := range reservedNames {
 		if name == reservedNames[i] {
 			return ErrNameReserved{name}