Skip to content

Do not create CAddress for invalid accepts#842

Merged
sipa merged 1 commit intobitcoin:masterfrom
sipa:checkaf
Feb 16, 2012
Merged

Do not create CAddress for invalid accepts#842
sipa merged 1 commit intobitcoin:masterfrom
sipa:checkaf

Conversation

@sipa
Copy link
Copy Markdown
Member

@sipa sipa commented Feb 15, 2012

No description provided.

sipa added a commit that referenced this pull request Feb 16, 2012
Do not create CAddress for invalid accepts
@sipa sipa merged commit 2030fc6 into bitcoin:master Feb 16, 2012
coblee referenced this pull request in litecoin-project/litecoin Jul 17, 2012
Do not create CAddress for invalid accepts
destenson pushed a commit to destenson/bitcoin--bitcoin that referenced this pull request Jun 26, 2016
Add full UTF-8 support to RPC
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Apr 6, 2019
0772a68 Reduce log spam (Fuzzbawls)
d367d5d Fix irrelevant receipt status message (Fuzzbawls)
417261d Don't error out when a range has already been computed (Fuzzbawls)
8082f27 fixup makefile.am (Fuzzbawls)
c224501 Fixup bad logprintf (Fuzzbawls)
e95e8ea Cleanup arbitrary sleep times in CreateCoinStake (Fuzzbawls)
8376665 Prevent lock spamming when no stakable inputs are present (Fuzzbawls)
4e5dc84 Better logging for CheckCoinSpend (Fuzzbawls)
b3bb83a Fixup the interface_rest functional test (Fuzzbawls)
4da1dca Disable precompute for functional tests (Fuzzbawls)
53dd8f5 Remove security level in `DoZpivSpend()` call (Fuzzbawls)
7491d6c Fix wrapped serial functional test (Fuzzbawls)
c503cd9 Fixup coinspend unit test header includes (Fuzzbawls)
580bec5 Fix assertion failure in mining/staking (Fuzzbawls)
4841189 Remove duplicate functions from rebase (Fuzzbawls)
092a027 re added LNZP removed files (furszy)
32e863c getaccumulatorwitness method fix (furszy)
6eaf457 precomputation-upstream conflicts solved (furszy)
f3f7a32 reorg + refactoring + conflicts (furszy)
803f46e Add global boolean, that stop precompute cache lock (blondfrogs)
8e0f565 Convert precompute cache into LRU cache (blondfrogs)
eca3fe8 Clear database cache with rpc call (blondfrogs)
135f19b Fix rebase conflicts from precomputing (warrows)
6e3df1f Update precompute cache (blondfrogs)
540120b Add Databasing of zpiv precomputes, they are not encrypted yet (blondfrogs)
6702fa5 Allow zPIV precomputing to be disabled (Fuzzbawls)
e8e6d48 Qt: Add precomputed percentage indicator column to zPIV control UI (Fuzzbawls)
07d590f Precompute zPIV spends in a dedicated thread (Fuzzbawls)
6e9d7af Fix RPC spendzerocoin (Fuzzbawls)
be02574 Make GetSerialHash return const again (Fuzzbawls)
39e80f5 more work on pre-computing (Fuzzbawls)
1f0b5b5 re-add zpivchain.h (fixes merge conflict error) (Fuzzbawls)
84b4f5f Add clearspendcache RPC command (Fuzzbawls)
f668d67 Remove security level from UI (Fuzzbawls)
e736149 don't hold cd_spendcache for the entire loop (Fuzzbawls)
6e2cf8d benchmark fixup (Fuzzbawls)
7c7fd45 don't duplicate function call (Fuzzbawls)
67b2e45 fully initialize member variables when setting the witness data (Fuzzbawls)
326f4e4 add benchmarking to zPIV spend inner functions (Fuzzbawls)
452a2b1 remove spam log output in stakeTargetHit() (Fuzzbawls)
cfa7852 set the accumulator start height when initializing the witness (Fuzzbawls)
92e3819 Include dependent headers in stakeinput.h (Fuzzbawls)
571b9fd ensure override is set for GetSerialHash() (Fuzzbawls)
a612119 prevent infinate loop in miner (Fuzzbawls)
924ede4 Remove security level from zerocoin spending. (presstab)
7adb7f2 More progress on caching spend data. (presstab)
fdb6c07 more progress (presstab)
928e6df In progress multi-input witness generation. (presstab)

Tree-SHA512: 3beffd57e005f4d6b82d8f1bf6ea09a09b9aa519dc1727ea6341c817970be2a17a84cfa838587492d8dd1fdcc382d184c9cdd28ff794be640f3bc12c241df3a9
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Jan 10, 2020
51cbb00 [Cleanup][Tests] Remove precompute option in default framework node conf (random-zebra)
7960c6e [Cleanup] remove fPrecompute option in SelectStakeCoins (random-zebra)
0187f0d [Cleanup][Tests] Remove crazy useful unittest created by "Tom" (random-zebra)
29a934c [Cleanup][Wallet] Remove zpiv spend cache from zpiv tracker (random-zebra)
bd3eebb [Cleanup][DB] Remove DB functions for zerocoin precomputing (random-zebra)
837f4e2 [Cleanup] Remove zPIV precomputing global variables. (random-zebra)
520284e [Cleanup][Qt] remove COLUMN_PRECOMPUTE from zpivcontroldialog (random-zebra)
2585532 [Cleanup] Init: remove precompute-related helps in strUsage (random-zebra)
fbcf37b [Cleanup] Init: remove "precompute" debug category (not used anywhere) (random-zebra)

Pull request description:

  Builds on top of bitcoin#1234 .
  Reviewers should double check with bitcoin#842 to ensure that the cleanup is complete

ACKs for top commit:
  Fuzzbawls:
    ACK 51cbb00
  furszy:
    Tested ACK 51cbb00.

Tree-SHA512: d496a314e3c9fa61efbc0e48d24806776ab43a8218941e04505aeafd278ad26c95fddaf0705cb0cda0d9b0e06a14bd3f20c18766b508931b8d3e2ac3789191c1
@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.

1 participant