Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Apr 3, 2021

No description provided.

@maflcko maflcko changed the title 2104 assumeutxo crash03 Fix assumeutxo crash due to truncated file Apr 3, 2021
@maflcko
Copy link
Member Author

maflcko commented Apr 3, 2021

Introduced in #19806

@maflcko maflcko force-pushed the 2104-assumeutxoCrash03 branch from 53c2981 to fa73ce6 Compare April 3, 2021 15:53
@maflcko maflcko marked this pull request as ready for review April 3, 2021 15:53
@jamesob
Copy link
Contributor

jamesob commented Apr 7, 2021

ACK fa73ce6

Thanks!

Copy link
Contributor

@ryanofsky ryanofsky left a comment

Choose a reason for hiding this comment

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

Code review ACK fa73ce6. Easy fix. It seems like this could have been caught in review, though.

@maflcko maflcko merged commit a12962c into bitcoin:master Apr 14, 2021
@maflcko maflcko deleted the 2104-assumeutxoCrash03 branch April 14, 2021 13:19
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 14, 2021
fa73ce6 Fix assumeutxo crash due to truncated file (MarcoFalke)

Pull request description:

ACKs for top commit:
  jamesob:
    ACK fa73ce6
  ryanofsky:
    Code review ACK fa73ce6. Easy fix. It seems like this could have been caught in review, though.

Tree-SHA512: 3a98687c386e3995114ddf0ad7194fadd9520989290681ef703b578e3ca21aee51eadfb83aa38a489bac13d12709ea137b9b184b08e5bfa2919cca177aab90be
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants