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.

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

Fix incorrect usage of begin() when genesis block is requested in "protx diff" #2699

Merged
merged 2 commits into from
Feb 12, 2019

Conversation

codablock
Copy link

.begin() on mapBlockIndex does NOT return the genesis block, but just the
block with lowest hash.

The fix is to use chainActive[0] to get the genesis block.

…otx diff"

.begin() on mapBlockIndex does NOT return the genesis block, but just the
block with lowest hash.

The fix is to use chainActive[0] to get the genesis block.
src/evo/simplifiedmns.cpp Outdated Show resolved Hide resolved
Co-Authored-By: codablock <ablock84@gmail.com>
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK

@UdjinM6 UdjinM6 merged commit c0cb274 into dashpay:develop Feb 12, 2019
@codablock codablock deleted the pr_dip4_fixdiff branch March 6, 2019 12:57
codablock added a commit to codablock/dash that referenced this pull request Mar 7, 2019
…otx diff" (dashpay#2699)

* Fix incorrect usage of begin() when genesis block is requested in "protx diff"

.begin() on mapBlockIndex does NOT return the genesis block, but just the
block with lowest hash.

The fix is to use chainActive[0] to get the genesis block.

* Update src/evo/simplifiedmns.cpp

Co-Authored-By: codablock <ablock84@gmail.com>
codablock added a commit to codablock/dash that referenced this pull request Mar 7, 2019
…otx diff" (dashpay#2699)

* Fix incorrect usage of begin() when genesis block is requested in "protx diff"

.begin() on mapBlockIndex does NOT return the genesis block, but just the
block with lowest hash.

The fix is to use chainActive[0] to get the genesis block.

* Update src/evo/simplifiedmns.cpp

Co-Authored-By: codablock <ablock84@gmail.com>
MIPPL pushed a commit to biblepay/biblepay that referenced this pull request May 26, 2019
* commit '4570079e53ad859c571aeac3f132cf2969888e14':
  Add missing entry to changelog (dashpay#2769)
  Move IS block filtering into ConnectBlock (dashpay#2766)
  [0.13.x] Bump version to 0.13.2 and add release notes (dashpay#2749)
  Bump minChainWork and AssumeValid to block #1033120 (dashpay#2750)
  Fix error message for invalid voting addresses (dashpay#2747)
  Make -masternodeblsprivkey mandatory when -masternode is given (dashpay#2745)
  Implement 2-stage commit for CEvoDB to avoid inconsistencies after crashes (dashpay#2744)
  Add collateraladdress into masternode/protx list rpc output (dashpay#2740)
  Only include selected TX types into CMerkleBlock (dashpay#2737)
  Stop g_connman first before deleting it (dashpay#2734)
  Fix incorrect usage of begin() when genesis block is requested in "protx diff" (dashpay#2699)
  Do not process blocks in CDeterministicMNManager before dip3 activation (dashpay#2698)
  Backport bitcoin#14701: build: Add CLIENT_VERSION_BUILD to CFBundleGetInfoString (dashpay#2687)
  Fix some typos in doc/guide-startmany.md (dashpay#2711)
  Minimal fix for litemode vs bad-protx-key-not-same issue (dashpay#2694)
  Release notes v0.13.1.0 (dashpay#2689)

# Conflicts:
#	configure.ac
#	doc/guide-startmany.md
#	doc/release-notes.md
#	src/chainparams.cpp
#	src/clientversion.h
#	src/validation.cpp
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.

None yet

2 participants