Skip to content

Commit

Permalink
Bug 1863736 - Avoid skipping sequence numbers in IPCFuzzController. r…
Browse files Browse the repository at this point in the history
…=truber

Differential Revision: https://phabricator.services.mozilla.com/D193067
  • Loading branch information
choller committed Nov 13, 2023
1 parent 45bb1e9 commit aaf0054
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tools/fuzzing/ipc/IPCFuzzController.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -579,11 +579,6 @@ bool IPCFuzzController::MakeTargetDecision(
*seqno = seqNos.first - 1;
*fseqno = seqNos.second + 1;

if (update) {
portSeqNos.insert_or_assign(*name,
std::pair<int32_t, uint64_t>(*seqno, *fseqno));
}

// If a type is already specified, we must be in preserveHeaderMode.
bool isPreserveHeader = *type;

Expand Down Expand Up @@ -638,6 +633,11 @@ bool IPCFuzzController::MakeTargetDecision(
IPC::StringFromIPCMessageType(*type), *type, actorIndex, actors.size(),
*actorId, isPreserveHeader);

if (update) {
portSeqNos.insert_or_assign(*name,
std::pair<int32_t, uint64_t>(*seqno, *fseqno));
}

return true;
}

Expand Down

0 comments on commit aaf0054

Please sign in to comment.