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.

Sign up for GitHub

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

Jump to bottom

Fix output indexes to be represented as a u32 (*requires reindex*) #69

Open
wants to merge 2 commits into
base: new-index
Choose a base branch
from
Open

Fix output indexes to be represented as a u32 (*requires reindex*) #69

wants to merge 2 commits into from

Commits on Feb 20, 2024

  1. Fix output indexes to be represented as a u32 

    Bitcoin's consensus rules allows for output indexes larger than u16,
which would result in an overflow prior to this fix.

This recently happened with a transaction on testnet:
https://blockstream.info/testnet/tx/ca3b75556430e1adf9e9790bce9c73a3d9afdb42305588e64c65b258c06c05c9

Based on @junderw's mempool/electrs#75. Thanks!

This change requires a full database reindex.
    @shesek
    shesek committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    097f600 View commit details
    Browse the repository at this point in the history

  2. Increase DB_VERSION to force a database reindex

    @shesek
    shesek committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    6ff4fdd View commit details
    Browse the repository at this point in the history