Skip to content

Conversation

sgimenez
Copy link
Contributor

I'm resurrecting this pull-request by patvarilly, which is very important I think.
(OSX users seems even more affected, see Issue #258)

The original pull/388 is not available anymore for unknown reasons (404).
The first commit already had two ACKs.
I added a similar change for DNS seeding.

@jgarzik
Copy link
Contributor

jgarzik commented Jul 14, 2011

ACK

@TheBlueMatt
Copy link
Contributor

ACK, needed for 0.4

sipa added a commit that referenced this pull request Jul 14, 2011
@sipa sipa merged commit 24271c5 into bitcoin:master Jul 14, 2011
ptschip pushed a commit to ptschip/bitcoin that referenced this pull request Apr 2, 2017
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Feb 23, 2019
It is possible for a spend of 7 to be larger than the max transaction size. Catch these cases and send a correct error report up the chain to the GUI.

Github-Merge: bitcoin#412
Rebased-From: 0983755
rajarshimaitra pushed a commit to rajarshimaitra/bitcoin that referenced this pull request Aug 5, 2021
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants