fix: correct mutex usage in consensus logic to prevent potential deadlock (#4946)

Frozen · web-flow · commit 85254d61257a · 2025-09-23T20:21:00.000+08:00
diff --git a/consensus/consensus_v2.go b/consensus/consensus_v2.go
@@ -318,7 +318,7 @@ func (consensus *Consensus) BlockCommitSigs(blockNum uint64) ([]byte, error) {
 	defer consensus.mutex.Unlock()
 	if err != nil ||
 		len(lastCommits) < bls.BLSSignatureSizeInBytes {
-		msgs := consensus.FBFTLog().GetMessagesByTypeSeq(
+		msgs := consensus.fBFTLog.GetMessagesByTypeSeq(
 			msg_pb.MessageType_COMMITTED, blockNum,
 		)
 		if len(msgs) != 1 {

Original file line number	Diff line number	Diff line change
`@@ -318,7 +318,7 @@ func (consensus *Consensus) BlockCommitSigs(blockNum uint64) ([]byte, error) {`
`318`	`318`	`defer consensus.mutex.Unlock()`
`319`	`319`	`if err != nil \|\|`
`320`	`320`	`len(lastCommits) < bls.BLSSignatureSizeInBytes {`
`321`		`- msgs := consensus.FBFTLog().GetMessagesByTypeSeq(`
	`321`	`+ msgs := consensus.fBFTLog.GetMessagesByTypeSeq(`
`322`	`322`	`msg_pb.MessageType_COMMITTED, blockNum,`
`323`	`323`	`)`
`324`	`324`	`if len(msgs) != 1 {`