Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dash backports #2956

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

Duddino
Copy link
Member

@Duddino Duddino commented Jan 14, 2025

Each commits backports a different PR, as mentioned in the commit name

codablock and others added 6 commits January 14, 2025 15:10
Inter means between two groups, intra means inside of a group.

Signed-off-by: pasta <[email protected]>
…shpay#3678)

* Always check for previous quorum set in llmq IS

* Refactor SelectQuorumForSigning and related code

Should have no changes in behaviour

* Do not use SIGN_HEIGHT_OFFSET when checking pending IS locks, use actual chain tip

This commit actually changes the behaviour
* Fix potential deadlock in `CSporkManager::UpdateSpork()`

* Protect `inputRequestIds` with cs lock

* Protect `curDBTransaction` in `CEvoDB::CommitRootTransaction()`

* Check for `AssertLockNotHeld` in `EnforceBestChainLock()` instead of just having a comment in code

* Protect spork maps on (de)serialization
@Duddino Duddino force-pushed the dash-backports-duddino branch from cc6ee9b to 595a672 Compare January 14, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants