diff --git a/modules/guildgve/api_challengefinish.go b/modules/guildgve/api_challengefinish.go
index 50754a6c5..21f0f0243 100644
--- a/modules/guildgve/api_challengefinish.go
+++ b/modules/guildgve/api_challengefinish.go
@@ -68,7 +68,7 @@ func (this *apiComp) ChallengeFinish(session comm.IUserSession, req *pb.GuildGve
 		}
 		return
 	}
-	if user = this.module.ModuleUser.GetUser(session.GetUserId()); user != nil {
+	if user = this.module.ModuleUser.GetUser(session.GetUserId()); user == nil {
 		errdata = &pb.ErrorData{
 			Code:    pb.ErrorCode_DBError,
 			Title:   pb.ErrorCode_DBError.ToString(),
diff --git a/modules/guildgve/modelUnionRoulette.go b/modules/guildgve/modelUnionRoulette.go
index 9a81cb7bc..9c87be5f3 100644
--- a/modules/guildgve/modelUnionRoulette.go
+++ b/modules/guildgve/modelUnionRoulette.go
@@ -80,7 +80,7 @@ func (this *ModelUnionroulette) roulettechangePush(unionid string, uid string, a
 		users = append(users, v.Uid)
 	}
 
-	lock, _ := this.module.modelGuildGve.userlock(unionid)
+	lock, _ := this.userlock(unionid)
 	err = lock.Lock()
 	if err != nil {
 		this.module.Error("公会战分布式锁 err!", log.Field{Key: "Unionid", Value: unionid}, log.Field{Key: "err", Value: err.Error()})
diff --git a/modules/tools/module.go b/modules/tools/module.go
index 140e40c67..b3d320504 100644
--- a/modules/tools/module.go
+++ b/modules/tools/module.go
@@ -74,7 +74,7 @@ func (this *Tools) GetPlayerlvConf(uid string) (conf *cfg.GamePlayerlvData, errd
 		user *pb.DBUser
 		err  error
 	)
-	if user = this.ModuleUser.GetUser(uid); user != nil {
+	if user = this.ModuleUser.GetUser(uid); user == nil {
 		errdata = &pb.ErrorData{
 			Code:    pb.ErrorCode_DBError,
 			Title:   pb.ErrorCode_DBError.String(),