diff --git a/modules/passon/api_inroom.go b/modules/passon/api_inroom.go
index c5cc7e197..704d4ba30 100644
--- a/modules/passon/api_inroom.go
+++ b/modules/passon/api_inroom.go
@@ -113,6 +113,10 @@ func (this *apiComp) InRoom(session comm.IUserSession, req *pb.PassonInRoomReq)
 	if errdata = this.module.ModuleHero.PassonHero(session, map[string]bool{hero.Id: true}); errdata != nil {
 		return
 	}
+
+	passon.Student[req.Index].State = 1
+	passon.Student[req.Index].Heroid = hero.Id
+
 	if err = this.module.modelPasson.updateUserPasson(session.GetUserId(), passon); err != nil {
 		errdata = &pb.ErrorData{
 			Code:    pb.ErrorCode_DBError,
diff --git a/modules/passon/api_outroom.go b/modules/passon/api_outroom.go
index 803fae63d..86ec35450 100644
--- a/modules/passon/api_outroom.go
+++ b/modules/passon/api_outroom.go
@@ -36,12 +36,12 @@ func (this *apiComp) OutRoom(session comm.IUserSession, req *pb.PassonOutRoomReq
 		}
 		return
 	}
-	passon.Student[req.Index].State = 2
-	passon.Student[req.Index].Cdtime = configure.Now().Unix()
+
 	if errdata = this.module.ModuleHero.PassonHero(session, map[string]bool{req.Heroid: false}); errdata != nil {
 		return
 	}
-
+	passon.Student[req.Index].State = 2
+	passon.Student[req.Index].Cdtime = configure.Now().Unix()
 	if err = this.module.modelPasson.updateUserPasson(session.GetUserId(), passon); err != nil {
 		errdata = &pb.ErrorData{
 			Code:    pb.ErrorCode_DBError,