fix(pbft): fix concurrent access issue of srPbftMessage (#5703)

* fix(pbft): fix the concurrent access issue of srPbftMessage

Author: jwrct

consensus/src/main/java/org/tron/consensus/pbft/PbftMessageHandle.java

@@ -262,7 +262,7 @@ public boolean isSyncing() {
   }
 
   //Cleanup related status
-  private void remove(String no) {
+  private synchronized void remove(String no) {
     String pre = String.valueOf(no) + "_";
     preVotes.remove(no);
     pareVoteMap.keySet().removeIf(vp -> StringUtils.startsWith(vp, pre));
@@ -285,7 +285,7 @@ private void remove(String no) {
     //just try once
     if (srPbftMessage != null && StringUtils.equals(no, srPbftMessage.getNo())) {
       try {
-        Thread.sleep(100);
+        wait(100);
       } catch (InterruptedException e) {
         Thread.currentThread().interrupt();
       } catch (Exception e) {

framework/src/test/java/org/tron/core/pbft/PbftTest.java

@@ -29,6 +29,7 @@ public void testPbftSrMessage() {
         ByteString.copyFrom(ByteArray.fromHexString("41df309fef25b311e7895562bd9e11aab2a58816d2")));
     PbftMessage pbftSrMessage = PbftMessage
         .prePrepareSRLMsg(blockCapsule, srList, 1, miner);
+    PbftMessage.fullNodePrePrepareSRLMsg(blockCapsule, srList, 1);
     System.out.println(pbftSrMessage);
   }