diff --git a/game-server/app/servers/battle/handler/pvpHandler.ts b/game-server/app/servers/battle/handler/pvpHandler.ts
index 8a5234eb3..55163d6b4 100644
--- a/game-server/app/servers/battle/handler/pvpHandler.ts
+++ b/game-server/app/servers/battle/handler/pvpHandler.ts
@@ -225,14 +225,14 @@ export class PvpHandler {
         if(!curOpp) return resResult(STATUS.PVP_ROLE_NOT_FOUND);
 
         const role = await RoleModel.findByRoleId(roleId);
-
-        const myRecInfo = await generMyRecInfo(heroScores, winStreakNum, role, isSuccess, pos, myHeroes);
         if(isSuccess) {
             winStreakNum ++;
         } else {
             winStreakNum = 0;
         }
 
+        const myRecInfo = await generMyRecInfo(heroScores, winStreakNum, role, isSuccess, pos, myHeroes);
+      
         let { attackInfo, showHeroScores, addSumScore } = myRecInfo
 
         score += addSumScore;
@@ -283,7 +283,7 @@ export class PvpHandler {
     async saveDefense(msg: { heroes: Array<{actorId: number, dataId: number, order: number}>, isDefaultHero:boolean }, session: BackendSession) {
         let {heroes, isDefaultHero} = msg;
         let roleId = session.get('roleId');
-        if (heroes.length > 5) {
+        if (heroes.length > 5 || heroes.length <= 0) {
             return resResult(STATUS.WRONG_PARMS);
         }
         heroes = uniq(heroes, function(item) {
diff --git a/game-server/app/services/pvpService.ts b/game-server/app/services/pvpService.ts
index cc4240c0f..f1a1abd8a 100644
--- a/game-server/app/services/pvpService.ts
+++ b/game-server/app/services/pvpService.ts
@@ -551,3 +551,4 @@ export async function generPVPOppRecInfo(isSuccess: boolean, curOpp: OppPlayers,
         score: 0
     }
 }
+