Removed lock acquiring for the method BlockCommitSigs.

harmony-one · Aug 12, 2024 · 200d508 · 200d508
1 parent 051e244
commit 200d508
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/consensus/consensus_v2.go b/consensus/consensus_v2.go
@@ -277,11 +277,10 @@ func (consensus *Consensus) BlockCommitSigs(blockNum uint64) ([]byte, error) {
 		return nil, nil
 	}
 	lastCommits, err := consensus.Blockchain().ReadCommitSig(blockNum)
-	consensus.mutex.Lock()
-	defer consensus.mutex.Unlock()
-	if err != nil ||
-		len(lastCommits) < bls.BLSSignatureSizeInBytes {
-		msgs := consensus.FBFTLog().GetMessagesByTypeSeq(
+	if err != nil || len(lastCommits) < bls.BLSSignatureSizeInBytes {
+		consensus.mutex.Lock()
+		defer consensus.mutex.Unlock()
+		msgs := consensus.fBFTLog.GetMessagesByTypeSeq(
 			msg_pb.MessageType_COMMITTED, blockNum,
 		)
 		if len(msgs) != 1 {