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

more fuzzer fixes (continuation of #1399) #1401

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

apoelstra
Copy link
Member

Some combination of rebasing and the fact that MarcoFalke is clearly grinding his commit hashes to all start with fa led to these being dropped in #1399.

MarcoFalke added 2 commits February 11, 2025 13:47
* feeDelta tracked the delta (to be applied on top of the actual fee)
* m_modified_fee tracks the actual fee with the delta included
* Instead of passing in the new total delta to the Updater, pass in by
  how much the total delta should be modified.

This is needed for the next commit, but makes sense on its own because
the same is done by UpdateDescendantState and UpdateAncestorState.

Cherry-pick of fa52cf8
bitcoin/bitcoin#23418 (1/2)
Copy link
Member

@delta1 delta1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 56eec70

@delta1 delta1 merged commit d020546 into ElementsProject:master Feb 11, 2025
10 of 13 checks passed
@apoelstra apoelstra deleted the 2025-02--fuzz-fixes-2 branch February 11, 2025 16:00
delta1 added a commit to delta1/elements that referenced this pull request Feb 13, 2025
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.

2 participants