diff --git a/proxy/vmess/inbound/inbound.go b/proxy/vmess/inbound/inbound.go
index 409900b1..fe86ef2e 100644
--- a/proxy/vmess/inbound/inbound.go
+++ b/proxy/vmess/inbound/inbound.go
@@ -47,11 +47,11 @@ func newUserByEmail(config *DefaultConfig) *userByEmail {
 
 func (v *userByEmail) addNoLock(u *protocol.MemoryUser) bool {
 	email := strings.ToLower(u.Email)
-	user, found := v.cache[email]
+	_, found := v.cache[email]
 	if found {
 		return false
 	}
-	v.cache[email] = user
+	v.cache[email] = u
 	return true
 }