Fix concurrency issue for RBF

[chenyukang]

What problem does this PR solve?

Problem Summary:

When multiple conflicted tx are added into txpool at the same time, because multiple threads may access same old data, resolve_tx may not resolve the conflicted tx to Dead, then check_rbf will be skipped, spec test RbfConcurrency is added to reproduce this scenario.

What is changed and how it works?

In this PR, check_rbf is moved to submit_entry, which will be guarded with a write lock, this change will make sure there is no double-spending tx in Pending, and with an extra benefit that only tx passed verify_rtx may invoke check_rbf.

What's Changed:

Related changes

• PR to update owner/repo:
• Need to cherry-pick to the release branch

Check List

Tests

• Unit test
• Integration test
• Manual test (add detailed scripts or steps below)
• No code ci-runs-only: [ quick_checks,linters ]

Side effects

• Performance regression
• Breaking backward compatibility

Release note

Title Only: Include only the PR title in the release note.