diff --git a/game-server/app/servers/battle/handler/pvpHandler.ts b/game-server/app/servers/battle/handler/pvpHandler.ts
index 7367a679a..5d342c0d6 100644
--- a/game-server/app/servers/battle/handler/pvpHandler.ts
+++ b/game-server/app/servers/battle/handler/pvpHandler.ts
@@ -188,7 +188,7 @@ export class PvpHandler {
         await checkTask(serverId, roleId, sid, TASK_TYPE.PVP);
 
         // 对手记录更新
-        await PvpHistoryOppModel.setStatus(roleId, oppRoleId, 1);
+        await PvpHistoryOppModel.setStatus(curOpp.oppDef.toString(), 1);
 
         let update: pvpUpdateInter = {
             oppBeforePlayers: pvpDefense.oppPlayers
diff --git a/shared/db/PvpHistoryOpp.ts b/shared/db/PvpHistoryOpp.ts
index 6e092947f..6d14cdbee 100644
--- a/shared/db/PvpHistoryOpp.ts
+++ b/shared/db/PvpHistoryOpp.ts
@@ -69,8 +69,8 @@ export default class PvpHistoryOpp extends BaseModel {
         return result;
     }
 
-    public static async setStatus(roleId: string, oppRoleId: string, status: number) {
-        const result: PvpHistoryOppType = await PvpHistoryOppModel.findOneAndUpdate({ roleId, oppRoleId }, {status}, {new: true}).lean();
+    public static async setStatus(_id: string, status: number) {
+        const result: PvpHistoryOppType = await PvpHistoryOppModel.findByIdAndUpdate(_id, { $set: {status}}, {new: true}).lean();
         return result;
     }