Skip to content

Saturate CalculateExtraTxWeight and cap GUI datacarriercost to 1024#268

Open
kwsantiago wants to merge 1 commit intobitcoinknots:29.x-knotsfrom
privkeyio:fix-extra-weight-overflow
Open

Saturate CalculateExtraTxWeight and cap GUI datacarriercost to 1024#268
kwsantiago wants to merge 1 commit intobitcoinknots:29.x-knotsfrom
privkeyio:fix-extra-weight-overflow

Conversation

@kwsantiago
Copy link
Copy Markdown

@kwsantiago kwsantiago commented Feb 24, 2026

Summary

  • Saturate mod_weight in CalculateExtraTxWeight to prevent int32_t` overflow wrapping negative
  • Cap GUI datacarriercost maximum from MAX_BLOCK_SERIALIZED_SIZE to 1024

Comment thread src/policy/policy.cpp Outdated
@kwsantiago kwsantiago force-pushed the fix-extra-weight-overflow branch from 19f5713 to df5e4a5 Compare February 25, 2026 22:19
@kwsantiago kwsantiago requested a review from luke-jr February 25, 2026 22:20
@luke-jr
Copy link
Copy Markdown
Collaborator

luke-jr commented Mar 23, 2026

Please split this into two commits Actually, don't

@luke-jr luke-jr added this to the 29.3 (2nd) milestone Apr 3, 2026
luke-jr pushed a commit that referenced this pull request Apr 13, 2026
Github-Pull: #268
Rebased-From: df5e4a5 (partial)
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