Translation file for Chinese zh_CN

Fix invalid zero valued fields

…t-early-block-check

Initialization: revert moving out of order block storage detection

Fix typo

Expedited blocks and Request Manager

Update coincontroldialog.cpp

Update overviewpage.ui

Update sendcoinsdialog.ui

Update guiutil.cpp

Update optionsmodel.cpp

Update overviewpage.cpp

Update sendcoinsentry.cpp

Update paymentrequestdata.h

Update paymentservertests.cpp

Update mining.cpp

Update mining.cpp

Update compress_tests.cpp

Update miner_tests.cpp

Update rpc_tests.cpp

Change the description

Update README.md

Change the name from "bitcoin" to "bgold"

Change bgold to bgoldd

Be more specific about the fork date

Update readme with exact block height

Link to our own website for security issues, until we have a dedicated URL

Better than linking to Bitcoin core, we submit bugs to them, our users submit bugs to us. Zcash has a dedicated `doc/security.md` which Hush inherited and that I maintain for Hush.

Link to our website.

Added our site url, now that our site is finished.

Implement difficulty adjustment each block
Changed ports to 8338 for mainnet and 18338 for testnet

Changed name of daemon and cli to bgoldd and bgold-cli

Moved directory to BitcoinGold / .bitcoingold

Fallback to previous difficulty change logic if BTG block is created.

fixed block for the slip

Precalculated the change of the diff adjust algorithm and corrected the values. Minor other changes.

Set testnet exact fork time. Minor changes.

Removed unnecessary assertion line in the regtest config

Fixed test execution

Update README.md

Skip p2p-compactblocks.py in TravisCI test.

Change default BITCOIND to bgoldd.

Fix bad testnet consensus params.

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py

Fix build errors.

- Link boost_system for Equihash library.
- Remove unnecessary LIBEQUIHASH_LIBS from libbitcoinconsensus.la.

Tested:
./autogen.sh && make

Support "legacy" param in RPCs

Change mainnet address version.

- Prefix 38 (G) for P2PKH.
- Prefix 90 (d) for P2SH.

Change P2SH prefix to "A" (23).

Also fixed unit tests.

Add `-convertaddress` in bgold-cli.

Usage: `bgold-cli -testnet=false -convertaddress=<address>`

Fix broken tests for new address format.

Update README.md

update to release 1.0.15

updatae to new relase needed because compiler crashed on ubuntu
14.04 & 16.04 with older versions installed.

solves bitcoin#43
see also bitcoin#53

update source for libsodium to 1.0.15

compiling fails with older version than 1.0.13 and libsodium needs #
to be compiled from the scratch for ubuntu 14.04 and 16.04

solves bitcoin#43
see also bitcoin#53

Update README.md

Fix various places referring to the old BTC port number

Update a few places with our new 8338 port

updated for OSX builders

Updated build tutorial for OSX contributors  (missing dependencies) .

Update README.md

Add foundersReward

Prevent transaction in premineWindow and create premineAddressWhitelist.

remove -lboost_system

Update build-unix.md

add libsodium-dev missing ...

Disable LogPrint in target libconsensus_la.

Disable bitcoin libs to make TravisCI happy.

Tested:
./autogen.sh && ./configure && make

Change powLimit for Equihash.

The default powLimit is too difficult for Equihash. We relex it to the same as Zcash.

Other changes:
- Fix edge case of powLimit.
- Revert `pow_tests.cpp` to bitcoin version.

Mark client version as not a realse version.

Change testnet fork height to 1210320.

Misc fixes.

- Change the premine whitelist placeholder to BTG address format.
- Reduce testnet premine window to 200 blocks.

Removed main seeds

Added libsodium

Fix bad testnet consensus params.

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Changed label

Removed main seeds

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

After rebase

Changed minimum difficulty.

Fixes not working magic

Removed previous seeds

Removed main seeds

Update README.md

WIP

WIP

Changed PCH message start

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Removed main seeds

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Changed minimum difficulty.

Fixes not working magic

Update pow.cpp

Update pow.cpp

Update README.md

Removed wrong indent

Removed empty line

Reverted pow.cpp

Removed empty line

Removed commented DNS seeds

Add SIGHASH_FORKID, a 2-way replay protection

Require FORKID mechanism to be enabled by default, except for some
special cases like legacy tests and blocks under BTG HF height

Add fork id to legacy tests and add replay attacks

All unit tests are passed

Add fork id to regtests

Add -bootstrap for syncing block history

Initial commit

Rename README.md to CORE-README.md

Remove LICENSE file

README

Will be continupusly creating content

PoW retargeting change for BTG hard fork permine proid.

- Fork from block #487427.
- Premine 16000 blocks.
- Minimal difficulty for the premining peroid.

PoW: Add dependency libsdoium for equihash.

Tested:
make (macOS 10.12.5, Xcode 8.3.3)

PoW: Import equihash.

Tested:
make

Add Equihash parameters N & K to chainparams.

Hardfork: Introduce new protocol version & versionbits.

Protocol version: 70016 (BTG_HARD_FORK_VERSION)
Version bits: 0x60000000

Hardfork: New fields in block header

- Add two new fields: nHeight, nSolution (for Equihash)
- Calculate block hash in new block format after the hard fork.
- Serialize block in legacy format when talking to Bitcoin node.

Tested:
unittest

Link sodium in qt wallet.

Tested:
./autogen.sh && make

PoW: Add Equihash validation functions.

- Initialize sodium on startup
- Add Equihash validation logic
- Fix missing htole32 problem in mingw build.

Backward compatible block serialization.

- Serialize in different formats based on the net protocol version.
- Remove the version bit 0x40000000.
- Support legacy block format for RPCs: getblock, getblockheader, getblocktemplate, submitblock.
- Temporarily disable SEND_CMPT.

Tested:
unit tests

Fix broken regression tests.

Intended broken testes:
- p2p-compactblocks.py
- rest.py

Tested:
reg tests.

Fix typo.

Block header compatible with Zcash.

- Add fields to block header for Zcash compatibility
- Extend the nonce from `uint32_t` to `uint256`
- Add 'nonceUint32' for backward compatibility:
  - RPC: 'getblockheader', 'getblock'
  - Rest: '/rest/headers', '/rest/block'

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py
(all passed except intended failures: p2p-compactblocks.py, rest.py)

Fix unaligned block header problem.

Also add a test to ensure Zcash compability.

Tested:
./src/test/test_bitcoin --run_test=btg_tests

Add the sanity test for libsodium.

Tested:
./src/test/test_bitcoin --run_test=sanity_tests

Fix linux build issue.

misc: Fix linux build and reduce verbose logging

Add unittests for Equihash PoW.

Tested:
./src/test/test_bitcoin --run_test=equihash_tests

Validate Equihash solution.

RPC for mining and basic tests.

- RPC
  - generate
  - generatetoaddress
  - getblocktemplate
- Add regtest `btg-hardfork.py`

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py

Fix rest api and skip p2p-compactblocks.py

Updated Configure.ac

Changed branding.

Add Travis-CI build status image in README.md

Changing client name

Changed from "Satoshi" to "Bitcoin Gold"

Slight edits

Small edit

Small edits

Misc

name edits

Misc

--

Misc

--

Misc, name

Name changing

Misc

Added the right project name.

Change name to Bitcoin Gold

---

Fix name

More name edits

Name edit.

name edit

BTC -> BTG

Update amount.h

Change unti from BTC to BTG

Switch name

Not sure if I should change the strings to such as "Bitcoins" to "Bitcoin Golds"

Update bitcoinunits.h

Update coincontroldialog.cpp

Update overviewpage.ui

Update sendcoinsdialog.ui

Update guiutil.cpp

Update optionsmodel.cpp

Update overviewpage.cpp

Update sendcoinsentry.cpp

Update paymentrequestdata.h

Update paymentservertests.cpp

Update mining.cpp

Update mining.cpp

Update compress_tests.cpp

Update miner_tests.cpp

Update rpc_tests.cpp

Change the description

Update README.md

Change the name from "bitcoin" to "bgold"

Change bgold to bgoldd

Be more specific about the fork date

Update readme with exact block height

Link to our own website for security issues, until we have a dedicated URL

Better than linking to Bitcoin core, we submit bugs to them, our users submit bugs to us. Zcash has a dedicated `doc/security.md` which Hush inherited and that I maintain for Hush.

Link to our website.

Added our site url, now that our site is finished.

Implement difficulty adjustment each block
Changed ports to 8338 for mainnet and 18338 for testnet

Changed name of daemon and cli to bgoldd and bgold-cli

Moved directory to BitcoinGold / .bitcoingold

Fallback to previous difficulty change logic if BTG block is created.

fixed block for the slip

Precalculated the change of the diff adjust algorithm and corrected the values. Minor other changes.

Set testnet exact fork time. Minor changes.

Removed unnecessary assertion line in the regtest config

Fixed test execution

Update README.md

Skip p2p-compactblocks.py in TravisCI test.

Change default BITCOIND to bgoldd.

Fix bad testnet consensus params.

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py

Fix build errors.

- Link boost_system for Equihash library.
- Remove unnecessary LIBEQUIHASH_LIBS from libbitcoinconsensus.la.

Tested:
./autogen.sh && make

Support "legacy" param in RPCs

Change mainnet address version.

- Prefix 38 (G) for P2PKH.
- Prefix 90 (d) for P2SH.

Change P2SH prefix to "A" (23).

Also fixed unit tests.

Add `-convertaddress` in bgold-cli.

Usage: `bgold-cli -testnet=false -convertaddress=<address>`

Fix broken tests for new address format.

Update README.md

update to release 1.0.15

updatae to new relase needed because compiler crashed on ubuntu
14.04 & 16.04 with older versions installed.

solves bitcoin#43
see also bitcoin#53

update source for libsodium to 1.0.15

compiling fails with older version than 1.0.13 and libsodium needs #
to be compiled from the scratch for ubuntu 14.04 and 16.04

solves bitcoin#43
see also bitcoin#53

Update README.md

Fix various places referring to the old BTC port number

Update a few places with our new 8338 port

updated for OSX builders

Updated build tutorial for OSX contributors  (missing dependencies) .

Update README.md

Add foundersReward

Prevent transaction in premineWindow and create premineAddressWhitelist.

remove -lboost_system

Update build-unix.md

add libsodium-dev missing ...

Disable LogPrint in target libconsensus_la.

Disable bitcoin libs to make TravisCI happy.

Tested:
./autogen.sh && ./configure && make

Change powLimit for Equihash.

The default powLimit is too difficult for Equihash. We relex it to the same as Zcash.

Other changes:
- Fix edge case of powLimit.
- Revert `pow_tests.cpp` to bitcoin version.

Mark client version as not a realse version.

Change testnet fork height to 1210320.

Misc fixes.

- Change the premine whitelist placeholder to BTG address format.
- Reduce testnet premine window to 200 blocks.

Removed main seeds

Added libsodium

Fix bad testnet consensus params.

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Changed label

Removed main seeds

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

After rebase

Changed minimum difficulty.

Fixes not working magic

Removed previous seeds

Removed main seeds

Update README.md

WIP

WIP

Changed PCH message start

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Removed main seeds

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Changed minimum difficulty.

Fixes not working magic

Update pow.cpp

Update pow.cpp

Update README.md

Removed wrong indent

Removed empty line

Reverted pow.cpp

Removed empty line

Removed commented DNS seeds

Add SIGHASH_FORKID, a 2-way replay protection

Require FORKID mechanism to be enabled by default, except for some
special cases like legacy tests and blocks under BTG HF height

Add fork id to legacy tests and add replay attacks

All unit tests are passed

Add fork id to regtests

Add -bootstrap for syncing block history

Fix bootstrap related issues.

- Cannot finish Initial Block Download stage
- Refuse to connect to non-default ports in 50 tries.

Temporarily disable mainnet as it's not ready yet.

Fix the basic info of testnet fork block.

Use the same network magic for regtest mode.

Adjusted difficulty change based on BTCGPU/BTCGPU#78

Update README.md

Allow to connect to Bitcoin network with -bootstrap flag.

Fix wrong GetDifficulty calculation.

Minor fixes.

- Allow bitcoin magic whe bootstraping
- Temporarily disable premine conbase check

Fixed difficulty adjustment

Change magic of signed message

Flag to disable IBD after fork.

fix typo

A few edits to paths and names

force new tx with FORKID to use bip143 transaction digest algorithm

Fix p2p-segwit & p2pfullblcoktest

add explicit amount in all regtests, all regtests are passed

Corrected the PoW adjustment parameters.

Changed the pow averaging window to 30 from 20C
Changed network magic.

Add getblocksubsidy to rpc.

Make Zcash compatible.

Update command help.

Rebranded core wallet

Allow to connect to Bitocin mainnet in `-bootstrap` mode.

Fix CScript string decoding issue

Test CLTV-multi-sig wallet for Bitcoin Gold.

Tested:
./test/functional/btg-timelock.py

Add CLTV-multisig whitelist for premine.

Tested:
./src/test/test_bitcoin --run_test=btg_tests

Move cltv_multisig_data.h out of generated data directory.

Fix nits in regtest btg-timelock.py.

Enforce CLTV-multi-sig for premine

Enable mainnet connection.

Change default windows install path.

Tested:
make deploy

Add more dnsseed.

Fix linearize-data.py for BTG block format.

Update README.md

fix typo

Rename configuration file

Change 'Bitcoin server' to 'BitcoinGold' server

Update Readme.md

Fix wrong word

Fixed gitian build

Clean gitian PGP keys from Core

Add gitian PGP key: wjcloud

Adding public key for gitian build

Fix naming of build artifacts to bitcoin-gold-...

Build artifacts (tarball produced by
`make dist` and the packages produced by gitian builds
are now named bitcoin-gold-XXXXXXXXXXXXXX
where the XXXXXXXXX part includes the version number
and followed by more specifics such as platform / architecture
in case of gitian build products.

Fix inconsistencies and grammar in various files & remove old md guide

Remove lxcbr0 lines from gitian-build.sh

Because it errors when building with lxc bitcoin#11391

Rollback some changes

Updated website url.

Add pgp key for h4x3rotab

Rename README.md to CORE-README.md

Remove LICENSE file

README

Will be continupusly creating content

PoW retargeting change for BTG hard fork permine proid.

- Fork from block #487427.
- Premine 16000 blocks.
- Minimal difficulty for the premining peroid.

PoW: Add dependency libsdoium for equihash.

Tested:
make (macOS 10.12.5, Xcode 8.3.3)

PoW: Import equihash.

Tested:
make

Add Equihash parameters N & K to chainparams.

Hardfork: Introduce new protocol version & versionbits.

Protocol version: 70016 (BTG_HARD_FORK_VERSION)
Version bits: 0x60000000

Hardfork: New fields in block header

- Add two new fields: nHeight, nSolution (for Equihash)
- Calculate block hash in new block format after the hard fork.
- Serialize block in legacy format when talking to Bitcoin node.

Tested:
unittest

Link sodium in qt wallet.

Tested:
./autogen.sh && make

PoW: Add Equihash validation functions.

- Initialize sodium on startup
- Add Equihash validation logic
- Fix missing htole32 problem in mingw build.

Backward compatible block serialization.

- Serialize in different formats based on the net protocol version.
- Remove the version bit 0x40000000.
- Support legacy block format for RPCs: getblock, getblockheader, getblocktemplate, submitblock.
- Temporarily disable SEND_CMPT.

Tested:
unit tests

Fix broken regression tests.

Intended broken testes:
- p2p-compactblocks.py
- rest.py

Tested:
reg tests.

Fix typo.

Block header compatible with Zcash.

- Add fields to block header for Zcash compatibility
- Extend the nonce from `uint32_t` to `uint256`
- Add 'nonceUint32' for backward compatibility:
  - RPC: 'getblockheader', 'getblock'
  - Rest: '/rest/headers', '/rest/block'

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py
(all passed except intended failures: p2p-compactblocks.py, rest.py)

Fix unaligned block header problem.

Also add a test to ensure Zcash compability.

Tested:
./src/test/test_bitcoin --run_test=btg_tests

Add the sanity test for libsodium.

Tested:
./src/test/test_bitcoin --run_test=sanity_tests

Fix linux build issue.

misc: Fix linux build and reduce verbose logging

Add unittests for Equihash PoW.

Tested:
./src/test/test_bitcoin --run_test=equihash_tests

Validate Equihash solution.

RPC for mining and basic tests.

- RPC
  - generate
  - generatetoaddress
  - getblocktemplate
- Add regtest `btg-hardfork.py`

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py

Fix rest api and skip p2p-compactblocks.py

Updated Configure.ac

Changed branding.

Add Travis-CI build status image in README.md

Changing client name

Changed from "Satoshi" to "Bitcoin Gold"

Slight edits

Small edit

Small edits

Misc

name edits

Misc

--

Misc

--

Misc, name

Name changing

Misc

Added the right project name.

Change name to Bitcoin Gold

---

Fix name

More name edits

Name edit.

name edit

BTC -> BTG

Update amount.h

Change unti from BTC to BTG

Switch name

Not sure if I should change the strings to such as "Bitcoins" to "Bitcoin Golds"

Update bitcoinunits.h

Update coincontroldialog.cpp

Update overviewpage.ui

Update sendcoinsdialog.ui

Update guiutil.cpp

Update optionsmodel.cpp

Update overviewpage.cpp

Update sendcoinsentry.cpp

Update paymentrequestdata.h

Update paymentservertests.cpp

Update mining.cpp

Update mining.cpp

Update compress_tests.cpp

Update miner_tests.cpp

Update rpc_tests.cpp

Change the description

Update README.md

Change the name from "bitcoin" to "bgold"

Change bgold to bgoldd

Be more specific about the fork date

Update readme with exact block height

Link to our own website for security issues, until we have a dedicated URL

Better than linking to Bitcoin core, we submit bugs to them, our users submit bugs to us. Zcash has a dedicated `doc/security.md` which Hush inherited and that I maintain for Hush.

Link to our website.

Added our site url, now that our site is finished.

Implement difficulty adjustment each block
Changed ports to 8338 for mainnet and 18338 for testnet

Changed name of daemon and cli to bgoldd and bgold-cli

Moved directory to BitcoinGold / .bitcoingold

Fallback to previous difficulty change logic if BTG block is created.

fixed block for the slip

Precalculated the change of the diff adjust algorithm and corrected the values. Minor other changes.

Set testnet exact fork time. Minor changes.

Removed unnecessary assertion line in the regtest config

Fixed test execution

Update README.md

Skip p2p-compactblocks.py in TravisCI test.

Change default BITCOIND to bgoldd.

Fix bad testnet consensus params.

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py

Fix build errors.

- Link boost_system for Equihash library.
- Remove unnecessary LIBEQUIHASH_LIBS from libbitcoinconsensus.la.

Tested:
./autogen.sh && make

Support "legacy" param in RPCs

Change mainnet address version.

- Prefix 38 (G) for P2PKH.
- Prefix 90 (d) for P2SH.

Change P2SH prefix to "A" (23).

Also fixed unit tests.

Add `-convertaddress` in bgold-cli.

Usage: `bgold-cli -testnet=false -convertaddress=<address>`

Fix broken tests for new address format.

Update README.md

update to release 1.0.15

updatae to new relase needed because compiler crashed on ubuntu
14.04 & 16.04 with older versions installed.

solves bitcoin#43
see also bitcoin#53

update source for libsodium to 1.0.15

compiling fails with older version than 1.0.13 and libsodium needs #
to be compiled from the scratch for ubuntu 14.04 and 16.04

solves bitcoin#43
see also bitcoin#53

Update README.md

Fix various places referring to the old BTC port number

Update a few places with our new 8338 port

updated for OSX builders

Updated build tutorial for OSX contributors  (missing dependencies) .

Update README.md

Add foundersReward

Prevent transaction in premineWindow and create premineAddressWhitelist.

remove -lboost_system

Update build-unix.md

add libsodium-dev missing ...

Disable LogPrint in target libconsensus_la.

Disable bitcoin libs to make TravisCI happy.

Tested:
./autogen.sh && ./configure && make

Change powLimit for Equihash.

The default powLimit is too difficult for Equihash. We relex it to the same as Zcash.

Other changes:
- Fix edge case of powLimit.
- Revert `pow_tests.cpp` to bitcoin version.

Mark client version as not a realse version.

Change testnet fork height to 1210320.

Misc fixes.

- Change the premine whitelist placeholder to BTG address format.
- Reduce testnet premine window to 200 blocks.

Removed main seeds

Added libsodium

Fix bad testnet consensus params.

Tested:
./src/test/test_bitcoin
./test/functional/test_runner.py

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Changed label

Removed main seeds

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

After rebase

Changed minimum difficulty.

Fixes not working magic

Removed previous seeds

Removed main seeds

Update README.md

WIP

WIP

Changed PCH message start

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Removed main seeds

WIP

WIP

Changed PCH message start

Changed bitcoin node port

Legacy logic for Magic

Minor changes on the Magic

Changed incorrect Magic for testnet
Reversed flags

Changed minimum difficulty.

Fixes not working magic

Update pow.cpp

Update pow.cpp

Update README.md

Removed wrong indent

Removed empty line

Reverted pow.cpp

Removed empty line

Removed commented DNS seeds

Add SIGHASH_FORKID, a 2-way replay protection

Require FORKID mechanism to be enabled by default, except for some
special cases like legacy tests and blocks under BTG HF height

Add fork id to legacy tests and add replay attacks

All unit tests are passed

Add fork id to regtests

Add -bootstrap for syncing block history

Fix bootstrap related issues.

- Cannot finish Initial Block Download stage
- Refuse to connect to non-default ports in 50 tries.

Temporarily disable mainnet as it's not ready yet.

Fix the basic info of testnet fork block.

Use the same network magic for regtest mode.

Adjusted difficulty change based on BTCGPU/BTCGPU#78

Update README.md

Allow to connect to Bitcoin network with -bootstrap flag.

Fix wrong GetDifficulty calculation.

Minor fixes.

- Allow bitcoin magic whe bootstraping
- Temporarily disable premine conbase check

Fixed difficulty adjustment

Change magic of signed message

Flag to disable IBD after fork.

fix typo

A few edits to paths and names

force new tx with FORKID to use bip143 transaction digest algorithm

Fix p2p-segwit & p2pfullblcoktest

add explicit amount in all regtests, all regtests are passed

Corrected the PoW adjustment parameters.

Changed the pow averaging window to 30 from 20C
Changed network magic.

Add getblocksubsidy to rpc.

Make Zcash compatible.

Update command help.

Rebranded core wallet

Allow to connect to Bitocin mainnet in `-bootstrap` mode.

Fix CScript string decoding issue

Test CLTV-multi-sig wallet for Bitcoin Gold.

Tested:
./test/functional/btg-timelock.py

Add CLTV-multisig whitelist for premine.

Tested:
./src/test/test_bitcoin --run_test=btg_tests

Move cltv_multisig_data.h out of generated data directory.

Fix nits in regtest btg-timelock.py.

Enforce CLTV-multi-sig for premine

Enable mainnet connection.

Change default windows install path.

Tested:
make deploy

Add more dnsseed.

Fix linearize-data.py for BTG block format.

Update README.md

fix typo

Rename configuration file

Change 'Bitcoin server' to 'BitcoinGold' server

Update Readme.md

Fix wrong word

Fixed gitian build

Clean gitian PGP keys from Core

Add gitian PGP key: wjcloud

Adding public key for gitian build

Fix naming of build artifacts to bitcoin-gold-...

Build artifacts (tarball produced by
`make dist` and the packages produced by gitian builds
are now named bitcoin-gold-XXXXXXXXXXXXXX
where the XXXXXXXXX part includes the version number
and followed by more specifics such as platform / architecture
in case of gitian build products.

Fix inconsistencies and grammar in various files & remove old md guide

Remove lxcbr0 lines from gitian-build.sh

Because it errors when building with lxc bitcoin#11391

Rollback some changes

Updated website url.

Add pgp key for h4x3rotab

rm md

up

…ixes-block-change

Added more fixes for python qa tests, mostly fixes related to blocks/…

updatae to new relase needed because compiler crashed on ubuntu
14.04 & 16.04 with older versions installed.

solves bitcoin#43
see also bitcoin#53

compiling fails with older version than 1.0.13 and libsodium needs #
to be compiled from the scratch for ubuntu 14.04 and 16.04

solves bitcoin#43
see also bitcoin#53

Update GUI strings

v0.18.1 update

* Depencies, change to macOS and macOS version bump

* Update MacOSX to macOS

* Fix typo and unneeded var

* Added Make Deploy for Portable Installs

* Build System Fix(Gitain) 

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* Build System Fix(Gitain) (#31)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* Updates + crash fixes (#34)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* [Model] Invalid iterator position crash fix.

* background loading crash fixed.

* random: fix crash on some 64bit platforms

rbx needs to be stashed in a 64bit register on 64bit platforms. With this crash
in particular, it was holding a stack canary which was not properly restored
after the cpuid.

Split out the x86+PIC case so that x86_64 doesn't have to worry about it.

* random: Add fallback if getrandom syscall not available

If the code was compiled with newer (>=3.17) kernel headers but executed
on a system without the system call, every use of random would crash the
program. Add a fallback for that case.

* Don't assert if we were beaten to the block

A timing window exists where a wallet could be creating a new block from within the miner thread when a new block is received to the wallet.  This window will create a situation where TestBlockValidity() fails because the chain tip has changed between the time it created the new block and the time it tested the validity of the block.

This situation would result in the wallet being asserted; however this is a little overkill.  rather than asserting if the tip has changed, it is better to throw the block away.

This problem was revealed during a testnet test of an altcoin, and very prevalent when multiple wallet existed with the exact same number of staking coins received in the same transaction; or when multiple wallets were staking the same coins via import private key.  The problem happens significantly less in more normal circumstances, but was still observed in a testing environment with fast blocks.

It is likely that this scenario has been encountered but never determined to be root cause, as a crashed wallet could be restarted, re-indexed and never investigated further.

* [Wallet] Add some LOCK to avoid crash

Github-Pull: #625
Rebased-From: ed23d0d62054156e0c2cb632fecbd65588c50e8c

* Sync develop with master (#35)

* Build System Fix(Gitain)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* Fix for supply --dirty

dirty fix for moneysupply

* Revert "Fix for supply --dirty"

This reverts commit 8060c6c1f840f477cb7803000e91be8a632aa03f.

* Another attempt at syncing master and develop (#36)

* Build System Fix(Gitain)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* Fix for supply --dirty

dirty fix for moneysupply

* Revert "Fix for supply --dirty"

This reverts commit 8060c6c1f840f477cb7803000e91be8a632aa03f.

* Mining Fixes (#37)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* [Model] Invalid iterator position crash fix.

* background loading crash fixed.

* random: fix crash on some 64bit platforms

rbx needs to be stashed in a 64bit register on 64bit platforms. With this crash
in particular, it was holding a stack canary which was not properly restored
after the cpuid.

Split out the x86+PIC case so that x86_64 doesn't have to worry about it.

* random: Add fallback if getrandom syscall not available

If the code was compiled with newer (>=3.17) kernel headers but executed
on a system without the system call, every use of random would crash the
program. Add a fallback for that case.

* Don't assert if we were beaten to the block

A timing window exists where a wallet could be creating a new block from within the miner thread when a new block is received to the wallet.  This window will create a situation where TestBlockValidity() fails because the chain tip has changed between the time it created the new block and the time it tested the validity of the block.

This situation would result in the wallet being asserted; however this is a little overkill.  rather than asserting if the tip has changed, it is better to throw the block away.

This problem was revealed during a testnet test of an altcoin, and very prevalent when multiple wallet existed with the exact same number of staking coins received in the same transaction; or when multiple wallets were staking the same coins via import private key.  The problem happens significantly less in more normal circumstances, but was still observed in a testing environment with fast blocks.

It is likely that this scenario has been encountered but never determined to be root cause, as a crashed wallet could be restarted, re-indexed and never investigated further.

* [Wallet] Add some LOCK to avoid crash

Github-Pull: #625
Rebased-From: ed23d0d62054156e0c2cb632fecbd65588c50e8c

* testnet mining fixes

* Update developer-notes.md

* Update Copyright

* Fix compile error (#38)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* [Model] Invalid iterator position crash fix.

* background loading crash fixed.

* random: fix crash on some 64bit platforms

rbx needs to be stashed in a 64bit register on 64bit platforms. With this crash
in particular, it was holding a stack canary which was not properly restored
after the cpuid.

Split out the x86+PIC case so that x86_64 doesn't have to worry about it.

* random: Add fallback if getrandom syscall not available

If the code was compiled with newer (>=3.17) kernel headers but executed
on a system without the system call, every use of random would crash the
program. Add a fallback for that case.

* Don't assert if we were beaten to the block

A timing window exists where a wallet could be creating a new block from within the miner thread when a new block is received to the wallet.  This window will create a situation where TestBlockValidity() fails because the chain tip has changed between the time it created the new block and the time it tested the validity of the block.

This situation would result in the wallet being asserted; however this is a little overkill.  rather than asserting if the tip has changed, it is better to throw the block away.

This problem was revealed during a testnet test of an altcoin, and very prevalent when multiple wallet existed with the exact same number of staking coins received in the same transaction; or when multiple wallets were staking the same coins via import private key.  The problem happens significantly less in more normal circumstances, but was still observed in a testing environment with fast blocks.

It is likely that this scenario has been encountered but never determined to be root cause, as a crashed wallet could be restarted, re-indexed and never investigated further.

* [Wallet] Add some LOCK to avoid crash

Github-Pull: #625
Rebased-From: ed23d0d62054156e0c2cb632fecbd65588c50e8c

* testnet mining fixes

* fix placement of utxo signer

* dont sign twice

* prevent logging twice

* random files updates

* Add Chacha20 crypto files

* Merge #643: [Crypto] Use stronger rand for key generation

b7dda924cfc2405386968b0eaa32aa0e546f322c [Log] Replace a string by the function name in a log (warrows)
977f089d0d2883ca3ca1648b14423b738c9248ef [Refactor] Use arrays instead of unic vars in Chacha20 (warrows)
d8abe323bd4f5cff3ca23fc00f0093ab7b143ead [Random] Add a missing include (warrows)
27663b8f16dfaac75bae1ab0e4a0c465b838c826 Do not permit copying FastRandomContexts (Pieter Wuille)
64e03e6de1a65c8d934d8df65af48e2b92bf887a Bugfix: randbytes should seed when needed (non reachable issue) (Pieter Wuille)
e8f12aa3621e98588b82533edb56ed4865645862 Check if sys/random.h is required for getentropy on OSX. (James Hilliard)
de85c7ae55681f8a9d322f4c48c6e9ba15ee0cf1 Add attribute [[noreturn]] (C++11) to functions that will not return (practicalswift)
df46c7ff5a2e6a71ba636fa39d77f58e018816a6 Fix resource leak (Dag Robole)
d426d856d23fe6c5d6ef3d57fe2c420b9a8e4e5e Clarify entropy source (Pieter Wuille)
30a320b3d12d3529dfca1daae0e477d76acd9ae7 Use cpuid intrinsics instead of asm code (Pieter Wuille)
0c21204e6dc714de6b917d9de12b1029d0bfb67b random: fix crash on some 64bit platforms (Cory Fields)
b8bbb9c68d996b1bea579956b3c91872d6d94b8f Use rdrand as entropy source on supported platforms (Pieter Wuille)
8e19443e41f748ad54d0f8aa25907bdad0815c19 [Tests] Fix compilation (warrows)
f53edec73901a91a6c4f595b248e4058e8be5468 [Rand/test] scripted-diff: Use new naming style for insecure_rand* functions (warrows)
272f3a5ac83ba0ab72a799ce524b6605b3555751 [Random / tests]  scripted-diff: Use randbits/bool instead of randrange (warrows)
0173ee3332990ac1eea134cca5c21cc451b81d92 Replace rand() & ((1 << N) - 1) with randbits(N) (Pieter Wuille)
250de7426e8317f86fc25fe60e960c2e9e903b69 Replace more rand() % NUM by randranges (Pieter Wuille)
d6904136030d94e844074fb75b9cafda46253ce0 [Random / tests] scripted-diff: use insecure_rand256/randrange more (warrows)
4a811ff9542439d43e3df3460114d9a24f849a36 Merge test_random.h into test_bitcoin.h (Pieter Wuille)
f275e638bf0a752d9f231f2e4a781432deafbfe8 Add various insecure_rand wrappers for tests (Pieter Wuille)
602af4fb3ef45a2a821f1b76d0dbd418ec852683 Add FastRandomContext::rand256() and ::randbytes() (Pieter Wuille)
90549785824f4c13ee8cf36e77f737b5d89d0f02 Add perf counter data to GetStrongRandBytes state in scheduler (Matt Corallo)
17dd13e746b567f44015fbefaa634b278010fccf Add internal method to add new random data to our internal RNG state (Matt Corallo)
c7a1602879aff2ae72bdefe959d4026ee8c0aefc Use sanity check timestamps as entropy (Pieter Wuille)
f671fe99994cb220dede903459307c61475aeef8 Test that GetPerformanceCounter() increments (Pieter Wuille)
dcb536fb2cef719abef9bb4104242410987bfd3d Use hardware timestamps in RNG seeding (Pieter Wuille)
7c3f290b50d32a3290a2c49542d72a3a7466da58 [Random] Fix compilation (warrows)
22b78957cc90252c958bc66439c098001d47b8fc random: only use getentropy on openbsd (Cory Fields)
e5750e5c9831edf01026643deed249e0efb28f0b Add a FastRandomContext::randrange and use it (Pieter Wuille)
2a0f6cd1ca2224e2ba30aea4ab312a1919dedc0c Switch FastRandomContext to ChaCha20 (Pieter Wuille)
401ca7db468939184681b73f4b346aebcfc378c6 Introduce FastRandomContext::randbool() (Pieter Wuille)
3d056d6ed478c5bbd42d7dde4f1ab0a75ffd42b1 Add ChaCha20 (Pieter Wuille)
3c97f3f2cb1b3dc2f5d642877d9a2de0a85c06bf Kill insecure_random and associated global state (Wladimir J. van der Laan)
68ba16c78593bf221844dae95fc5edacbf69ba62 Maintain state across GetStrongRandBytes calls (Pieter Wuille)
c3c399e199c2d25d10e782dca2ef17f55b6939d0 random: Add fallback if getrandom syscall not available (Wladimir J. van der Laan)
7a8111f915fc7ed23aaec86cd95904bf5d5972fd sanity: Move OS random to sanity check function (Wladimir J. van der Laan)
387c2e9574b56ae941b7723c561ed39488891782 squashme: comment that NUM_OS_RANDOM_BYTES should not be changed lightly (Wladimir J. van der Laan)
9e8c266b158c26b361d68a3cccdbcfd7bee0b376 util: Specific GetOSRandom for Linux/FreeBSD/OpenBSD (Wladimir J. van der Laan)
f989b865f215c797e77483a04cc7fe9026904517 Don't use assert for catching randomness failures (Pieter Wuille)
a15419e13204d96416b0cce9aad2ae4e35ea5ed5 Always require OS randomness when generating secret keys (Pieter Wuille)

Pull request description:

  Since #576 hasn't changed in over a month, here is a reworked version of it.
  So in this PR:
  -We add the memory_cleanse function from upstream, to remove a number of OpenSSL calls.
  -We use OS randomness in addition to OpenSSL randomness (see #576 for why it's needed).

ACKs for top commit:
  random-zebra:
    ACK https://github.com/PIVX-Project/PIVX/pull/643/commits/b7dda924cfc2405386968b0eaa32aa0e546f322c
  furszy:
    ACK [`b7dda92`](https://github.com/PIVX-Project/PIVX/commit/b7dda924cfc2405386968b0eaa32aa0e546f322c)

Tree-SHA512: d92cbc14d844263ced753248e646f5cd4f03ec37546f50ff1b558fc3076b6d777c7efcb899c0400bc510e21311b5bd93d9aca26d811033fedb370f8457204035

* Revert "Merge #643: [Crypto] Use stronger rand for key generation"

This reverts commit ae3e2562220577420e37a6367feeaa3cfedc67e9.

* Revert "Add Chacha20 crypto files"

This reverts commit ddd102fca86499062dac0ea3a40a8d8b26d039f5.

* Revert "random files updates"

This reverts commit f29e0844099487811d4f3c7af7f6d089adc2f6cf.

* Revert "random: fix crash on some 64bit platforms"

This reverts commit 5a5241452e1515fbd554ee99cc14e552fcb98051.

* Revert "random: Add fallback if getrandom syscall not available"

This reverts commit b46c1cb2b466f4bce60db364bf26f2cf791cdf38.

* fix redeclaration

* fix signing code

* do not return in void func

* remove unused funcs

* remove on error

* revert "Refactor miner"

* Fix name collision + mac crashes (#39)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* [Model] Invalid iterator position crash fix.

* background loading crash fixed.

* random: fix crash on some 64bit platforms

rbx needs to be stashed in a 64bit register on 64bit platforms. With this crash
in particular, it was holding a stack canary which was not properly restored
after the cpuid.

Split out the x86+PIC case so that x86_64 doesn't have to worry about it.

* random: Add fallback if getrandom syscall not available

If the code was compiled with newer (>=3.17) kernel headers but executed
on a system without the system call, every use of random would crash the
program. Add a fallback for that case.

* Don't assert if we were beaten to the block

A timing window exists where a wallet could be creating a new block from within the miner thread when a new block is received to the wallet.  This window will create a situation where TestBlockValidity() fails because the chain tip has changed between the time it created the new block and the time it tested the validity of the block.

This situation would result in the wallet being asserted; however this is a little overkill.  rather than asserting if the tip has changed, it is better to throw the block away.

This problem was revealed during a testnet test of an altcoin, and very prevalent when multiple wallet existed with the exact same number of staking coins received in the same transaction; or when multiple wallets were staking the same coins via import private key.  The problem happens significantly less in more normal circumstances, but was still observed in a testing environment with fast blocks.

It is likely that this scenario has been encountered but never determined to be root cause, as a crashed wallet could be restarted, re-indexed and never investigated further.

* [Wallet] Add some LOCK to avoid crash

Github-Pull: #625
Rebased-From: ed23d0d62054156e0c2cb632fecbd65588c50e8c

* testnet mining fixes

* fix placement of utxo signer

* dont sign twice

* prevent logging twice

* random files updates

* Add Chacha20 crypto files

* Merge #643: [Crypto] Use stronger rand for key generation

b7dda924cfc2405386968b0eaa32aa0e546f322c [Log] Replace a string by the function name in a log (warrows)
977f089d0d2883ca3ca1648b14423b738c9248ef [Refactor] Use arrays instead of unic vars in Chacha20 (warrows)
d8abe323bd4f5cff3ca23fc00f0093ab7b143ead [Random] Add a missing include (warrows)
27663b8f16dfaac75bae1ab0e4a0c465b838c826 Do not permit copying FastRandomContexts (Pieter Wuille)
64e03e6de1a65c8d934d8df65af48e2b92bf887a Bugfix: randbytes should seed when needed (non reachable issue) (Pieter Wuille)
e8f12aa3621e98588b82533edb56ed4865645862 Check if sys/random.h is required for getentropy on OSX. (James Hilliard)
de85c7ae55681f8a9d322f4c48c6e9ba15ee0cf1 Add attribute [[noreturn]] (C++11) to functions that will not return (practicalswift)
df46c7ff5a2e6a71ba636fa39d77f58e018816a6 Fix resource leak (Dag Robole)
d426d856d23fe6c5d6ef3d57fe2c420b9a8e4e5e Clarify entropy source (Pieter Wuille)
30a320b3d12d3529dfca1daae0e477d76acd9ae7 Use cpuid intrinsics instead of asm code (Pieter Wuille)
0c21204e6dc714de6b917d9de12b1029d0bfb67b random: fix crash on some 64bit platforms (Cory Fields)
b8bbb9c68d996b1bea579956b3c91872d6d94b8f Use rdrand as entropy source on supported platforms (Pieter Wuille)
8e19443e41f748ad54d0f8aa25907bdad0815c19 [Tests] Fix compilation (warrows)
f53edec73901a91a6c4f595b248e4058e8be5468 [Rand/test] scripted-diff: Use new naming style for insecure_rand* functions (warrows)
272f3a5ac83ba0ab72a799ce524b6605b3555751 [Random / tests]  scripted-diff: Use randbits/bool instead of randrange (warrows)
0173ee3332990ac1eea134cca5c21cc451b81d92 Replace rand() & ((1 << N) - 1) with randbits(N) (Pieter Wuille)
250de7426e8317f86fc25fe60e960c2e9e903b69 Replace more rand() % NUM by randranges (Pieter Wuille)
d6904136030d94e844074fb75b9cafda46253ce0 [Random / tests] scripted-diff: use insecure_rand256/randrange more (warrows)
4a811ff9542439d43e3df3460114d9a24f849a36 Merge test_random.h into test_bitcoin.h (Pieter Wuille)
f275e638bf0a752d9f231f2e4a781432deafbfe8 Add various insecure_rand wrappers for tests (Pieter Wuille)
602af4fb3ef45a2a821f1b76d0dbd418ec852683 Add FastRandomContext::rand256() and ::randbytes() (Pieter Wuille)
90549785824f4c13ee8cf36e77f737b5d89d0f02 Add perf counter data to GetStrongRandBytes state in scheduler (Matt Corallo)
17dd13e746b567f44015fbefaa634b278010fccf Add internal method to add new random data to our internal RNG state (Matt Corallo)
c7a1602879aff2ae72bdefe959d4026ee8c0aefc Use sanity check timestamps as entropy (Pieter Wuille)
f671fe99994cb220dede903459307c61475aeef8 Test that GetPerformanceCounter() increments (Pieter Wuille)
dcb536fb2cef719abef9bb4104242410987bfd3d Use hardware timestamps in RNG seeding (Pieter Wuille)
7c3f290b50d32a3290a2c49542d72a3a7466da58 [Random] Fix compilation (warrows)
22b78957cc90252c958bc66439c098001d47b8fc random: only use getentropy on openbsd (Cory Fields)
e5750e5c9831edf01026643deed249e0efb28f0b Add a FastRandomContext::randrange and use it (Pieter Wuille)
2a0f6cd1ca2224e2ba30aea4ab312a1919dedc0c Switch FastRandomContext to ChaCha20 (Pieter Wuille)
401ca7db468939184681b73f4b346aebcfc378c6 Introduce FastRandomContext::randbool() (Pieter Wuille)
3d056d6ed478c5bbd42d7dde4f1ab0a75ffd42b1 Add ChaCha20 (Pieter Wuille)
3c97f3f2cb1b3dc2f5d642877d9a2de0a85c06bf Kill insecure_random and associated global state (Wladimir J. van der Laan)
68ba16c78593bf221844dae95fc5edacbf69ba62 Maintain state across GetStrongRandBytes calls (Pieter Wuille)
c3c399e199c2d25d10e782dca2ef17f55b6939d0 random: Add fallback if getrandom syscall not available (Wladimir J. van der Laan)
7a8111f915fc7ed23aaec86cd95904bf5d5972fd sanity: Move OS random to sanity check function (Wladimir J. van der Laan)
387c2e9574b56ae941b7723c561ed39488891782 squashme: comment that NUM_OS_RANDOM_BYTES should not be changed lightly (Wladimir J. van der Laan)
9e8c266b158c26b361d68a3cccdbcfd7bee0b376 util: Specific GetOSRandom for Linux/FreeBSD/OpenBSD (Wladimir J. van der Laan)
f989b865f215c797e77483a04cc7fe9026904517 Don't use assert for catching randomness failures (Pieter Wuille)
a15419e13204d96416b0cce9aad2ae4e35ea5ed5 Always require OS randomness when generating secret keys (Pieter Wuille)

Pull request description:

  Since #576 hasn't changed in over a month, here is a reworked version of it.
  So in this PR:
  -We add the memory_cleanse function from upstream, to remove a number of OpenSSL calls.
  -We use OS randomness in addition to OpenSSL randomness (see #576 for why it's needed).

ACKs for top commit:
  random-zebra:
    ACK https://github.com/PIVX-Project/PIVX/pull/643/commits/b7dda924cfc2405386968b0eaa32aa0e546f322c
  furszy:
    ACK [`b7dda92`](https://github.com/PIVX-Project/PIVX/commit/b7dda924cfc2405386968b0eaa32aa0e546f322c)

Tree-SHA512: d92cbc14d844263ced753248e646f5cd4f03ec37546f50ff1b558fc3076b6d777c7efcb899c0400bc510e21311b5bd93d9aca26d811033fedb370f8457204035

* Revert "Merge #643: [Crypto] Use stronger rand for key generation"

This reverts commit ae3e2562220577420e37a6367feeaa3cfedc67e9.

* Revert "Add Chacha20 crypto files"

This reverts commit ddd102fca86499062dac0ea3a40a8d8b26d039f5.

* Revert "random files updates"

This reverts commit f29e0844099487811d4f3c7af7f6d089adc2f6cf.

* Revert "random: fix crash on some 64bit platforms"

This reverts commit 5a5241452e1515fbd554ee99cc14e552fcb98051.

* Revert "random: Add fallback if getrandom syscall not available"

This reverts commit b46c1cb2b466f4bce60db364bf26f2cf791cdf38.

* fix redeclaration

* fix signing code

* do not return in void func

* remove unused funcs

* remove on error

* revert "Refactor miner"

* fix repeat of masternode in fundamental node payment log

* fix name collision

* [Wallet] Look at last CoinsView block for corruption fix process

* Fix crashes

* Merge #941: [Refactor] Move ThreadStakeMinter out of net.cpp

42fe4040acff08450168416ccf35413abdc1747a [Refactor] Move ThreadStakeMinter out of net.cpp (Fuzzbawls)

Pull request description:

  This moves the implementation function and thread creation to a more
  appropriate file (`miner.cpp`).

  Resolves #938

ACKs for top commit:
  CaveSpectre11:
    ACK https://github.com/PIVX-Project/PIVX/commit/42fe4040acff08450168416ccf35413abdc1747a
  random-zebra:
    ACK https://github.com/PIVX-Project/PIVX/pull/941/commits/42fe4040acff08450168416ccf35413abdc1747a
  furszy:
    utACK [42fe404](https://github.com/PIVX-Project/PIVX/pull/941/commits/42fe4040acff08450168416ccf35413abdc1747a)

Tree-SHA512: d3417f03cab63aa41b1ff0fb4d391d42f4448b83efe70391d4014407008af34eb265a23b5807ebecb90cf9637fb74e5b330b02c92e7038e5400a5f16608800ee

* Merge #865: [Main] Don't return an invalid state when shutting down the wallet

8126729 Don't return an invalid state when shutting down the wallet (Fuzzbawls)

Tree-SHA512: b7042df42af620faca5d415382ad0a6ae973e9dbad7e527957c2ac3c7d0cb5470fcca555b3fe262f5f06e334504ded657db879c2471ecc290247e0f7e2bbf5f3

* Merge #958: [Staking] Modify miner and staking thread for efficiency

b7a9a9cdb20cdee869906fb24e2bc984e8a93802 [Staking] Modify miner and staking thread for efficency (Cave Spectre)

Pull request description:

  ### **Release notes**
  - [Mining] Unnecessary mining threads now exit after PoS has begun
  - [Staking] The staking thread dormancy is more efficient during PoW
  - [Performance] Some unnecessary processing in the mining thread removed
  - [RPC] setgenerate errors if attempted to turn on after end of PoW
  - [Build] Wallet only routines conditionalized in miner.h
  - [Refactoring] Log messages with bitcoin named routines changed for easier rename of routines in the future.

  In order to better explain the changes in this PR, a review of the existing code would be helpful.

  ### **Backstory**
  When fixing a PIVX forked coin's transition from PoW to PoS, it was observed that mining threads go into tight infinite loops after the switch to proof of stake.  This can be seen with a  simple `pivx-qt -testnet -gen`, and watching the debug log.

  This observation triggered the below code review (re-written for the current release; some of the
  findings in the code originally reviewed had already been addressed in PIVX).

  ### **Code Review**

  _BitcoinMiner()_
  This routine is a worker routine for both staking and mining threads; fProofOfStake is set to true when running in the staking thread, false when running in the mining thread.  fGeneratedBitcoins is set when mining is started; and cleared on shutdown or when mining is stopped.

  _In the initial fProofOfStake section:_
  ```
              if ((GetTime() - nMintableLastCheck > 5 * 60)) // 5 minute check time
              {
                  nMintableLastCheck = GetTime();
                  fMintableCoins = pwallet->MintableCoins();
              }

              if (chainActive.Tip()->nHeight < Params().LAST_POW_BLOCK()) {
                  MilliSleep(5000);
                  continue;
              }
  ```
  Every 5 minutes this code will scan through the wallet and look to see if mintable coins exist.  After it's done that, it checks if the network is still in the proof of work phase.  It doesn't make sense to scan the wallet until after the proof of work phase is ending.

  If it is in the proof of work phase; the staking thread sleeps for 5 seconds  (`MilliSleep(5000)`).  This only makes sense if the block time is 5 seconds.  Technically the thread could determine the number of blocks remaining in the proof of work phase, and use that to calculate a hibernation of sorts.  Using the target spacing to wait for the block to pass is a 91% reduction in the number of passes through the loop  (12 vs. 1 when the target spacing is one minute), and doesn't run into issues where adjustments to the actual spacing throw the timing of a hibernation off.

  ```
                  if (!fMintableCoins) {
                      if (GetTime() - nMintableLastCheck > 1 * 60) // 1 minute check time
                      {
                          nMintableLastCheck = GetTime();
                          fMintableCoins = pwallet->MintableCoins();
                      }
                  }
                  MilliSleep(5000);
  ```

  We come to this section of code in the waiting while loop; if we don't have mintable coins yet, or our wallet is locked, or we're not synched; or a host of other potential things that would prevent staking.  However, we check again for mintable coins; and then we wait 5 seconds after that check before we come out.

  Since the first time we come into this loop; we have already checked mintable coins (within the last 5 minutes); we should sleep first, and then do the check closer to the time we're actually going to go around the loop again; so we're working with the most recent data.

  ```
                  if (!fGenerateBitcoins && !fProofOfStake)
                      continue;
  ```

  This check is buried in a `if (fProofOfStake)` conditional, where fProofOfStake is a parameter that is passed in by the caller to BitcoinMiner().  `!fProofOfStake` will never be true; so this condition will never be true.  Removing it we are left with !fGenerateBitcoins; which is unrelated to this section of code anyway.  If we're down to this point, we're not in PoW mode anymore, so there shouldn't be a mining thread (more on that later).  But the most compelling part of this condition,whether it passes or not, is that it "continues" the while loop it's in; and since it's at the end of the actual while loop, it's going to iterate into another round of the while loop as soon as it finishes with this conditional anyway.  Long story short; the code doesn't do anything.

  _Overall logic_
  The mining thread will run until mining is turned off (`setgenerate false`, or the mining flags taken off the invocation).  However, there is no need to continue to mine with proof of work after the proof of stake phase begins.  In fact, there really is no consideration of that at all; PoW will continue to try to generate blocks well after PoS has begun.  Yes, it's within the users control to stop mining; but it's within the power of the code to take care of that for the user, and stop the mining thread(s) after the transition to proof of stake.

  ### **This PR:**
  The logical issues above have been corrected.  The dormancy for the proof of stake thread is held for just one block rather than any extreme hibernation.  Combined with moving that code to be the first thing in the while loop; it also removes the processing done to search through the wallets looking for stakeable coins.

  Since a significant amount of code assumes there will not be a transition from PoS to PoW; code was added to BitcoinMiner() to exit the mining threads if it's transitioned into the PoS phase.  The threads will continue for a little bit, in case there is any rewind on the chain; but after 6 PoS blocks are accepted, the miner threads will exit.

  To prevent the PoW thread from trying to generate blocks after the PoS phase has begun, logic was added in CreateNewBlockWithKey() to return quickly.  Rather than create a situation where the mining thread ends up in a tight loop in the initial phases of PoS, the mining thread will be held for 1/2 of the target spacing before returning.

  Lastly; Some tweaks were made to the logging; removing hardcoded references to "BitcoinMiner()", in case refactoring in the future changes the name of the routine.  Similarly, since there is a log message when the mining thread starts "PIVXMiner started"; rather than reporting thread exits or errors as "ThreadBitcoinMiner", they have been changed to PIVXMiner to match.

  Much of this can be observed with an errant `pivx-qt -gen`.  With this PR, it simply tries to start the miner, and then exits.

  _With PR code - `pivx-qt -gen -testnet`_
  ```
  2019-07-21 20:04:22 PIVXMiner started
  2019-07-21 20:04:22 BitcoinMiner: Exiting Proof of Work Mining Thread at height: 1160732
  2019-07-21 20:04:22 PIVXMiner exiting
  ```

  _Current release:_
  ```
  2019-07-21 19:37:06 CreateNewBlock(): total size 1000
  2019-07-21 19:37:06 ERROR: CheckProofOfWork() : hash doesn't match nBits
  2019-07-21 19:37:06 ERROR: CheckBlockHeader() : proof of work failed
  2019-07-21 19:37:06 ERROR: CheckBlock() : CheckBlockHeader failed
  2019-07-21 19:37:06 CreateNewBlock() : TestBlockValidity failed
  ```
  is repeated in a tight loop.

ACKs for top commit:
  furszy:
    ACK [b7a9a9c](https://github.com/PIVX-Project/PIVX/commit/b7a9a9cdb20cdee869906fb24e2bc984e8a93802)
  random-zebra:
    ACK https://github.com/PIVX-Project/PIVX/pull/958/commits/b7a9a9cdb20cdee869906fb24e2bc984e8a93802 and merging...

Tree-SHA512: 064619ce1fde75f1baf35ddb202b4573f2a8c0effc5d2939a678612e3f1d56b08d5c5d7c7bb30ce919e5a648f3f406cca35093b9a3a04c799f746fff5c6a35b6

* remove duplicate stake thread code

* add missing include

* remove zvitae pos validator

* Refine signblock code

* revert zerocoin added check

* Miner reference updates

* revert miner changes

* fix fundamentalnode sync checker

* [Refactor] Move rpc files to its own folder

* update includes

* fix rpcserver include to rpc/server.h

* [Refactor] Move wallet files to wallet dir

* [Trivial] Fix db.h path

* Merge #982: [Miner] Don't create new keys when generating PoS blocks

74e7fc7d432c7f6d586d54ff276d6babad0fe505 [Miner] Don't create new keys when generating PoS blocks (random-zebra)

Pull request description:

  This fixes the calls to `CreateNewBlockWithKey` with PoS blocks, which are not needed.
  We simply call `CreateNewBlock` with an empy script for coinbase.

  It should prevent the problem described in https://github.com/PIVX-Project/PIVX/pull/956#issuecomment-520166966

ACKs for top commit:
  CaveSpectre11:
    utACK https://github.com/PIVX-Project/PIVX/pull/982/commits/74e7fc7d432c7f6d586d54ff276d6babad0fe505
  akshaynexus:
    utACK https://github.com/PIVX-Project/PIVX/commit/74e7fc7d432c7f6d586d54ff276d6babad0fe505
  furszy:
    utACK [74e7fc7](https://github.com/PIVX-Project/PIVX/pull/982/commits/74e7fc7d432c7f6d586d54ff276d6babad0fe505)
  Warrows:
    Pretty straightforward indeed, utACK 74e7fc7

Tree-SHA512: bcb9752a689a058b6c33317136d16afd694b5da66c69cdd32a8b89b71059c5b2c5dc20c88045c52429fae99d45f508ea223f1b7d1afa5db3c77eb59befd48d51

* Merge #549: [Crypto] Switch to libsecp256k1 signature verification and update the lib

f10439c [Crypto] Add ctx initialisation for bip38 (warrows)
21234db [Crypto] Bring back function CKey.SetPrivKey for zPIV (warrows)
65e009a [Tests] Add new auto generated script tests (warrows)
6839f3b [Crypto] Switch from openssl to secp256k1 for consensus (warrows)
8a901f9 Squashed 'src/secp256k1/' content from commit 452d8e4d2 (warrows)
d98a584 [Refactor] Delete secp256k1 folder for subtreefication (warrows)

Tree-SHA512: f0f6777be57777ba86f83af1b891a6c0f384e6b059afc9249599269c71e5d3bf46a6498325488878af71b6685c6dac6cb672d0147c2ebf43b36f6d786fc38a10

* fix keystore refs

* update libscepk265k1

*  [RPC/REST] Migrate to libevent based httpd server

* fix mempooltojson and misc erroros

* Merge #769: [Main] Unify shutdown proceedure in init rather than per-app

518542f [Main] Unify shutdown proceedure in init rather than per-app (Fuzzbawls)

Tree-SHA512: 6e734ce002e236c3c2c59c2e8e93d87f5fe577b3f5546804b117a71255266cc4012d847d6a1d9b019a7b763124b45987c2dc3f95294670489cac269beceb2e3a

* Merge #542: [Network] Remove vfReachable and modify IsReachable to only use vfLimited.

38ed737 Remove vfReachable and modify IsReachable to only use vfLimited. (Patrick Strateman)

Tree-SHA512: 8aa61f9abd8356522f3ba71b35497aac07e10333a39ff4b98bb635d1246e6b1be6b0ee0b236944fbfaa558af0e6c0cc219e540eacb2232bc05db393e4a72b088

* [Refactor] Refactor bignum header file into several files

* [zerocoin] Free memory from ToString()

Simple fix for a minor memory leak

* Revert "[Refactor] Refactor bignum header file into several files"

This reverts commit c9aa98fe451e27d2bb4564747ffb0768cce56d53.

* fix delays in mining code in testnet (#40)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* [Model] Invalid iterator position crash fix.

* background loading crash fixed.

* random: fix crash on some 64bit platforms

rbx needs to be stashed in a 64bit register on 64bit platforms. With this crash
in particular, it was holding a stack canary which was not properly restored
after the cpuid.

Split out the x86+PIC case so that x86_64 doesn't have to worry about it.

* random: Add fallback if getrandom syscall not available

If the code was compiled with newer (>=3.17) kernel headers but executed
on a system without the system call, every use of random would crash the
program. Add a fallback for that case.

* Don't assert if we were beaten to the block

A timing window exists where a wallet could be creating a new block from within the miner thread when a new block is received to the wallet.  This window will create a situation where TestBlockValidity() fails because the chain tip has changed between the time it created the new block and the time it tested the validity of the block.

This situation would result in the wallet being asserted; however this is a little overkill.  rather than asserting if the tip has changed, it is better to throw the block away.

This problem was revealed during a testnet test of an altcoin, and very prevalent when multiple wallet existed with the exact same number of staking coins received in the same transaction; or when multiple wallets were staking the same coins via import private key.  The problem happens significantly less in more normal circumstances, but was still observed in a testing environment with fast blocks.

It is likely that this scenario has been encountered but never determined to be root cause, as a crashed wallet could be restarted, re-indexed and never investigated further.

* [Wallet] Add some LOCK to avoid crash

Github-Pull: #625
Rebased-From: ed23d0d62054156e0c2cb632fecbd65588c50e8c

* testnet mining fixes

* fix placement of utxo signer

* dont sign twice

* prevent logging twice

* random files updates

* Add Chacha20 crypto files

* Merge #643: [Crypto] Use stronger rand for key generation

b7dda924cfc2405386968b0eaa32aa0e546f322c [Log] Replace a string by the function name in a log (warrows)
977f089d0d2883ca3ca1648b14423b738c9248ef [Refactor] Use arrays instead of unic vars in Chacha20 (warrows)
d8abe323bd4f5cff3ca23fc00f0093ab7b143ead [Random] Add a missing include (warrows)
27663b8f16dfaac75bae1ab0e4a0c465b838c826 Do not permit copying FastRandomContexts (Pieter Wuille)
64e03e6de1a65c8d934d8df65af48e2b92bf887a Bugfix: randbytes should seed when needed (non reachable issue) (Pieter Wuille)
e8f12aa3621e98588b82533edb56ed4865645862 Check if sys/random.h is required for getentropy on OSX. (James Hilliard)
de85c7ae55681f8a9d322f4c48c6e9ba15ee0cf1 Add attribute [[noreturn]] (C++11) to functions that will not return (practicalswift)
df46c7ff5a2e6a71ba636fa39d77f58e018816a6 Fix resource leak (Dag Robole)
d426d856d23fe6c5d6ef3d57fe2c420b9a8e4e5e Clarify entropy source (Pieter Wuille)
30a320b3d12d3529dfca1daae0e477d76acd9ae7 Use cpuid intrinsics instead of asm code (Pieter Wuille)
0c21204e6dc714de6b917d9de12b1029d0bfb67b random: fix crash on some 64bit platforms (Cory Fields)
b8bbb9c68d996b1bea579956b3c91872d6d94b8f Use rdrand as entropy source on supported platforms (Pieter Wuille)
8e19443e41f748ad54d0f8aa25907bdad0815c19 [Tests] Fix compilation (warrows)
f53edec73901a91a6c4f595b248e4058e8be5468 [Rand/test] scripted-diff: Use new naming style for insecure_rand* functions (warrows)
272f3a5ac83ba0ab72a799ce524b6605b3555751 [Random / tests]  scripted-diff: Use randbits/bool instead of randrange (warrows)
0173ee3332990ac1eea134cca5c21cc451b81d92 Replace rand() & ((1 << N) - 1) with randbits(N) (Pieter Wuille)
250de7426e8317f86fc25fe60e960c2e9e903b69 Replace more rand() % NUM by randranges (Pieter Wuille)
d6904136030d94e844074fb75b9cafda46253ce0 [Random / tests] scripted-diff: use insecure_rand256/randrange more (warrows)
4a811ff9542439d43e3df3460114d9a24f849a36 Merge test_random.h into test_bitcoin.h (Pieter Wuille)
f275e638bf0a752d9f231f2e4a781432deafbfe8 Add various insecure_rand wrappers for tests (Pieter Wuille)
602af4fb3ef45a2a821f1b76d0dbd418ec852683 Add FastRandomContext::rand256() and ::randbytes() (Pieter Wuille)
90549785824f4c13ee8cf36e77f737b5d89d0f02 Add perf counter data to GetStrongRandBytes state in scheduler (Matt Corallo)
17dd13e746b567f44015fbefaa634b278010fccf Add internal method to add new random data to our internal RNG state (Matt Corallo)
c7a1602879aff2ae72bdefe959d4026ee8c0aefc Use sanity check timestamps as entropy (Pieter Wuille)
f671fe99994cb220dede903459307c61475aeef8 Test that GetPerformanceCounter() increments (Pieter Wuille)
dcb536fb2cef719abef9bb4104242410987bfd3d Use hardware timestamps in RNG seeding (Pieter Wuille)
7c3f290b50d32a3290a2c49542d72a3a7466da58 [Random] Fix compilation (warrows)
22b78957cc90252c958bc66439c098001d47b8fc random: only use getentropy on openbsd (Cory Fields)
e5750e5c9831edf01026643deed249e0efb28f0b Add a FastRandomContext::randrange and use it (Pieter Wuille)
2a0f6cd1ca2224e2ba30aea4ab312a1919dedc0c Switch FastRandomContext to ChaCha20 (Pieter Wuille)
401ca7db468939184681b73f4b346aebcfc378c6 Introduce FastRandomContext::randbool() (Pieter Wuille)
3d056d6ed478c5bbd42d7dde4f1ab0a75ffd42b1 Add ChaCha20 (Pieter Wuille)
3c97f3f2cb1b3dc2f5d642877d9a2de0a85c06bf Kill insecure_random and associated global state (Wladimir J. van der Laan)
68ba16c78593bf221844dae95fc5edacbf69ba62 Maintain state across GetStrongRandBytes calls (Pieter Wuille)
c3c399e199c2d25d10e782dca2ef17f55b6939d0 random: Add fallback if getrandom syscall not available (Wladimir J. van der Laan)
7a8111f915fc7ed23aaec86cd95904bf5d5972fd sanity: Move OS random to sanity check function (Wladimir J. van der Laan)
387c2e9574b56ae941b7723c561ed39488891782 squashme: comment that NUM_OS_RANDOM_BYTES should not be changed lightly (Wladimir J. van der Laan)
9e8c266b158c26b361d68a3cccdbcfd7bee0b376 util: Specific GetOSRandom for Linux/FreeBSD/OpenBSD (Wladimir J. van der Laan)
f989b865f215c797e77483a04cc7fe9026904517 Don't use assert for catching randomness failures (Pieter Wuille)
a15419e13204d96416b0cce9aad2ae4e35ea5ed5 Always require OS randomness when generating secret keys (Pieter Wuille)

Pull request description:

  Since #576 hasn't changed in over a month, here is a reworked version of it.
  So in this PR:
  -We add the memory_cleanse function from upstream, to remove a number of OpenSSL calls.
  -We use OS randomness in addition to OpenSSL randomness (see #576 for why it's needed).

ACKs for top commit:
  random-zebra:
    ACK https://github.com/PIVX-Project/PIVX/pull/643/commits/b7dda924cfc2405386968b0eaa32aa0e546f322c
  furszy:
    ACK [`b7dda92`](https://github.com/PIVX-Project/PIVX/commit/b7dda924cfc2405386968b0eaa32aa0e546f322c)

Tree-SHA512: d92cbc14d844263ced753248e646f5cd4f03ec37546f50ff1b558fc3076b6d777c7efcb899c0400bc510e21311b5bd93d9aca26d811033fedb370f8457204035

* Revert "Merge #643: [Crypto] Use stronger rand for key generation"

This reverts commit ae3e2562220577420e37a6367feeaa3cfedc67e9.

* Revert "Add Chacha20 crypto files"

This reverts commit ddd102fca86499062dac0ea3a40a8d8b26d039f5.

* Revert "random files updates"

This reverts commit f29e0844099487811d4f3c7af7f6d089adc2f6cf.

* Revert "random: fix crash on some 64bit platforms"

This reverts commit 5a5241452e1515fbd554ee99cc14e552fcb98051.

* Revert "random: Add fallback if getrandom syscall not available"

This reverts commit b46c1cb2b466f4bce60db364bf26f2cf791cdf38.

* fix redeclaration

* fix signing code

* do not return in void func

* remove unused funcs

* remove on error

* revert "Refactor miner"

* fix repeat of masternode in fundamental node payment log

* fix name collision

* [Wallet] Look at last CoinsView block for corruption fix process

* Fix crashes

* Merge #941: [Refactor] Move ThreadStakeMinter out of net.cpp

42fe4040acff08450168416ccf35413abdc1747a [Refactor] Move ThreadStakeMinter out of net.cpp (Fuzzbawls)

Pull request description:

  This moves the implementation function and thread creation to a more
  appropriate file (`miner.cpp`).

  Resolves #938

ACKs for top commit:
  CaveSpectre11:
    ACK https://github.com/PIVX-Project/PIVX/commit/42fe4040acff08450168416ccf35413abdc1747a
  random-zebra:
    ACK https://github.com/PIVX-Project/PIVX/pull/941/commits/42fe4040acff08450168416ccf35413abdc1747a
  furszy:
    utACK [42fe404](https://github.com/PIVX-Project/PIVX/pull/941/commits/42fe4040acff08450168416ccf35413abdc1747a)

Tree-SHA512: d3417f03cab63aa41b1ff0fb4d391d42f4448b83efe70391d4014407008af34eb265a23b5807ebecb90cf9637fb74e5b330b02c92e7038e5400a5f16608800ee

* Merge #865: [Main] Don't return an invalid state when shutting down the wallet

8126729 Don't return an invalid state when shutting down the wallet (Fuzzbawls)

Tree-SHA512: b7042df42af620faca5d415382ad0a6ae973e9dbad7e527957c2ac3c7d0cb5470fcca555b3fe262f5f06e334504ded657db879c2471ecc290247e0f7e2bbf5f3

* Merge #958: [Staking] Modify miner and staking thread for efficiency

b7a9a9cdb20cdee869906fb24e2bc984e8a93802 [Staking] Modify miner and staking thread for efficency (Cave Spectre)

Pull request description:

  ### **Release notes**
  - [Mining] Unnecessary mining threads now exit after PoS has begun
  - [Staking] The staking thread dormancy is more efficient during PoW
  - [Performance] Some unnecessary processing in the mining thread removed
  - [RPC] setgenerate errors if attempted to turn on after end of PoW
  - [Build] Wallet only routines conditionalized in miner.h
  - [Refactoring] Log messages with bitcoin named routines changed for easier rename of routines in the future.

  In order to better explain the changes in this PR, a review of the existing code would be helpful.

  ### **Backstory**
  When fixing a PIVX forked coin's transition from PoW to PoS, it was observed that mining threads go into tight infinite loops after the switch to proof of stake.  This can be seen with a  simple `pivx-qt -testnet -gen`, and watching the debug log.

  This observation triggered the below code review (re-written for the current release; some of the
  findings in the code originally reviewed had already been addressed in PIVX).

  ### **Code Review**

  _BitcoinMiner()_
  This routine is a worker routine for both staking and mining threads; fProofOfStake is set to true when running in the staking thread, false when running in the mining thread.  fGeneratedBitcoins is set when mining is started; and cleared on shutdown or when mining is stopped.

  _In the initial fProofOfStake section:_
  ```
              if ((GetTime() - nMintableLastCheck > 5 * 60)) // 5 minute check time
              {
                  nMintableLastCheck = GetTime();
                  fMintableCoins = pwallet->MintableCoins();
              }

              if (chainActive.Tip()->nHeight < Params().LAST_POW_BLOCK()) {
                  MilliSleep(5000);
                  continue;
              }
  ```
  Every 5 minutes this code will scan through the wallet and look to see if mintable coins exist.  After it's done that, it checks if the network is still in the proof of work phase.  It doesn't make sense to scan the wallet until after the proof of work phase is ending.

  If it is in the proof of work phase; the staking thread sleeps for 5 seconds  (`MilliSleep(5000)`).  This only makes sense if the block time is 5 seconds.  Technically the thread could determine the number of blocks remaining in the proof of work phase, and use that to calculate a hibernation of sorts.  Using the target spacing to wait for the block to pass is a 91% reduction in the number of passes through the loop  (12 vs. 1 when the target spacing is one minute), and doesn't run into issues where adjustments to the actual spacing throw the timing of a hibernation off.

  ```
                  if (!fMintableCoins) {
                      if (GetTime() - nMintableLastCheck > 1 * 60) // 1 minute check time
                      {
                          nMintableLastCheck = GetTime();
                          fMintableCoins = pwallet->MintableCoins();
                      }
                  }
                  MilliSleep(5000);
  ```

  We come to this section of code in the waiting while loop; if we don't have mintable coins yet, or our wallet is locked, or we're not synched; or a host of other potential things that would prevent staking.  However, we check again for mintable coins; and then we wait 5 seconds after that check before we come out.

  Since the first time we come into this loop; we have already checked mintable coins (within the last 5 minutes); we should sleep first, and then do the check closer to the time we're actually going to go around the loop again; so we're working with the most recent data.

  ```
                  if (!fGenerateBitcoins && !fProofOfStake)
                      continue;
  ```

  This check is buried in a `if (fProofOfStake)` conditional, where fProofOfStake is a parameter that is passed in by the caller to BitcoinMiner().  `!fProofOfStake` will never be true; so this condition will never be true.  Removing it we are left with !fGenerateBitcoins; which is unrelated to this section of code anyway.  If we're down to this point, we're not in PoW mode anymore, so there shouldn't be a mining thread (more on that later).  But the most compelling part of this condition,whether it passes or not, is that it "continues" the while loop it's in; and since it's at the end of the actual while loop, it's going to iterate into another round of the while loop as soon as it finishes with this conditional anyway.  Long story short; the code doesn't do anything.

  _Overall logic_
  The mining thread will run until mining is turned off (`setgenerate false`, or the mining flags taken off the invocation).  However, there is no need to continue to mine with proof of work after the proof of stake phase begins.  In fact, there really is no consideration of that at all; PoW will continue to try to generate blocks well after PoS has begun.  Yes, it's within the users control to stop mining; but it's within the power of the code to take care of that for the user, and stop the mining thread(s) after the transition to proof of stake.

  ### **This PR:**
  The logical issues above have been corrected.  The dormancy for the proof of stake thread is held for just one block rather than any extreme hibernation.  Combined with moving that code to be the first thing in the while loop; it also removes the processing done to search through the wallets looking for stakeable coins.

  Since a significant amount of code assumes there will not be a transition from PoS to PoW; code was added to BitcoinMiner() to exit the mining threads if it's transitioned into the PoS phase.  The threads will continue for a little bit, in case there is any rewind on the chain; but after 6 PoS blocks are accepted, the miner threads will exit.

  To prevent the PoW thread from trying to generate blocks after the PoS phase has begun, logic was added in CreateNewBlockWithKey() to return quickly.  Rather than create a situation where the mining thread ends up in a tight loop in the initial phases of PoS, the mining thread will be held for 1/2 of the target spacing before returning.

  Lastly; Some tweaks were made to the logging; removing hardcoded references to "BitcoinMiner()", in case refactoring in the future changes the name of the routine.  Similarly, since there is a log message when the mining thread starts "PIVXMiner started"; rather than reporting thread exits or errors as "ThreadBitcoinMiner", they have been changed to PIVXMiner to match.

  Much of this can be observed with an errant `pivx-qt -gen`.  With this PR, it simply tries to start the miner, and then exits.

  _With PR code - `pivx-qt -gen -testnet`_
  ```
  2019-07-21 20:04:22 PIVXMiner started
  2019-07-21 20:04:22 BitcoinMiner: Exiting Proof of Work Mining Thread at height: 1160732
  2019-07-21 20:04:22 PIVXMiner exiting
  ```

  _Current release:_
  ```
  2019-07-21 19:37:06 CreateNewBlock(): total size 1000
  2019-07-21 19:37:06 ERROR: CheckProofOfWork() : hash doesn't match nBits
  2019-07-21 19:37:06 ERROR: CheckBlockHeader() : proof of work failed
  2019-07-21 19:37:06 ERROR: CheckBlock() : CheckBlockHeader failed
  2019-07-21 19:37:06 CreateNewBlock() : TestBlockValidity failed
  ```
  is repeated in a tight loop.

ACKs for top commit:
  furszy:
    ACK [b7a9a9c](https://github.com/PIVX-Project/PIVX/commit/b7a9a9cdb20cdee869906fb24e2bc984e8a93802)
  random-zebra:
    ACK https://github.com/PIVX-Project/PIVX/pull/958/commits/b7a9a9cdb20cdee869906fb24e2bc984e8a93802 and merging...

Tree-SHA512: 064619ce1fde75f1baf35ddb202b4573f2a8c0effc5d2939a678612e3f1d56b08d5c5d7c7bb30ce919e5a648f3f406cca35093b9a3a04c799f746fff5c6a35b6

* remove duplicate stake thread code

* add missing include

* remove zvitae pos validator

* Refine signblock code

* revert zerocoin added check

* Miner reference updates

* revert miner changes

* fix fundamentalnode sync checker

* [Refactor] Move rpc files to its own folder

* update includes

* fix rpcserver include to rpc/server.h

* [Refactor] Move wallet files to wallet dir

* [Trivial] Fix db.h path

* Merge #982: [Miner] Don't create new keys when generating PoS blocks

74e7fc7d432c7f6d586d54ff276d6babad0fe505 [Miner] Don't create new keys when generating PoS blocks (random-zebra)

Pull request description:

  This fixes the calls to `CreateNewBlockWithKey` with PoS blocks, which are not needed.
  We simply call `CreateNewBlock` with an empy script for coinbase.

  It should prevent the problem described in https://github.com/PIVX-Project/PIVX/pull/956#issuecomment-520166966

ACKs for top commit:
  CaveSpectre11:
    utACK https://github.com/PIVX-Project/PIVX/pull/982/commits/74e7fc7d432c7f6d586d54ff276d6babad0fe505
  akshaynexus:
    utACK https://github.com/PIVX-Project/PIVX/commit/74e7fc7d432c7f6d586d54ff276d6babad0fe505
  furszy:
    utACK [74e7fc7](https://github.com/PIVX-Project/PIVX/pull/982/commits/74e7fc7d432c7f6d586d54ff276d6babad0fe505)
  Warrows:
    Pretty straightforward indeed, utACK 74e7fc7

Tree-SHA512: bcb9752a689a058b6c33317136d16afd694b5da66c69cdd32a8b89b71059c5b2c5dc20c88045c52429fae99d45f508ea223f1b7d1afa5db3c77eb59befd48d51

* Merge #549: [Crypto] Switch to libsecp256k1 signature verification and update the lib

f10439c [Crypto] Add ctx initialisation for bip38 (warrows)
21234db [Crypto] Bring back function CKey.SetPrivKey for zPIV (warrows)
65e009a [Tests] Add new auto generated script tests (warrows)
6839f3b [Crypto] Switch from openssl to secp256k1 for consensus (warrows)
8a901f9 Squashed 'src/secp256k1/' content from commit 452d8e4d2 (warrows)
d98a584 [Refactor] Delete secp256k1 folder for subtreefication (warrows)

Tree-SHA512: f0f6777be57777ba86f83af1b891a6c0f384e6b059afc9249599269c71e5d3bf46a6498325488878af71b6685c6dac6cb672d0147c2ebf43b36f6d786fc38a10

* fix keystore refs

* update libscepk265k1

*  [RPC/REST] Migrate to libevent based httpd server

* fix mempooltojson and misc erroros

* Merge #769: [Main] Unify shutdown proceedure in init rather than per-app

518542f [Main] Unify shutdown proceedure in init rather than per-app (Fuzzbawls)

Tree-SHA512: 6e734ce002e236c3c2c59c2e8e93d87f5fe577b3f5546804b117a71255266cc4012d847d6a1d9b019a7b763124b45987c2dc3f95294670489cac269beceb2e3a

* Merge #542: [Network] Remove vfReachable and modify IsReachable to only use vfLimited.

38ed737 Remove vfReachable and modify IsReachable to only use vfLimited. (Patrick Strateman)

Tree-SHA512: 8aa61f9abd8356522f3ba71b35497aac07e10333a39ff4b98bb635d1246e6b1be6b0ee0b236944fbfaa558af0e6c0cc219e540eacb2232bc05db393e4a72b088

* [Refactor] Refactor bignum header file into several files

* [zerocoin] Free memory from ToString()

Simple fix for a minor memory leak

* Revert "[Refactor] Refactor bignum header file into several files"

This reverts commit c9aa98fe451e27d2bb4564747ffb0768cce56d53.

* fix delay in mining blocks

* Vitaefy Debian Docs

* Fix error when trying to stop mining process (#41)

* Give error if pow phase is finished

* Force Sync to allow for staking without mnsync

* Fix error

* Update rpcmisc.cpp

* Update rpcmisc.cpp

* more ignore

* Update fundamentalnode-sync.cpp

* Add gitian build python script

* Update Univalue

* add missing gencpp

* Add missing includes

* Fix Gitian build

Update Univalue


add missing gencpp


Add missing includes

* Revert "Update fundamentalnode-sync.cpp"

This reverts commit 323b13fc599b03982b0ca7c3f32342537a6b992f

* Fix gitian build and minor updates

* remove forcesync

* [Model] Invalid iterator position crash fix.

* background loading crash fixed.

* random: fix crash on some 64bit platforms

rbx needs to be stashed in a 64bit register on 64bit platforms. With this crash
in particular, it was holding a stack canary which was not properly restored
after the cpuid.

Split out the x86+PIC case so that x86_64 doesn't have to worry about it.

* random: Add fallback if getrandom syscall not available

If the code was compiled with newer (>=3.17) kernel headers but executed
on a system without the system call, every use of random would crash the
program. Add a fallback for that case.

* Don't assert if we were beaten to the block

A timing window exists where a wallet could be creating a new block from within the miner thread when a new block is received to the wallet.  This window will create a situation where TestBlockValidity() fails because the chain tip has changed between the time it created the new block and the time it tested the validity of the block.

This situation would result in the wallet being asserted; however this is a little overkill.  rather than asserting if the tip has changed, it is better to throw the block away.

This problem was revealed during a testnet test of an altcoin, and very prevalent when multiple wallet existed with the exact same number of staking coins received in the same transaction; or when multiple wallets were staking the same coins via import private key.  The problem happens significantly less in more normal circumstances, but was still observed in a testing environment with fast blocks.

It is likely that this scenario has been encountered but never determined to be root cause, as a crashed wallet could be restarted, re-indexed and never investigated further.

* [Wallet] Add some LOCK to avoid crash

Github-Pull: #625
Rebased-From: ed23d0d62054156e0c2cb632fecbd65588c50e8c

* testnet mining fixes

* fix placement of utxo signer

* dont sign twice

* prevent logging twice

* random files updates

* Add Chacha20 crypto files

* Merge #643: [Crypto] Use stronger rand for key generation

b7dda924cfc2405386968b0eaa32aa0e546f322c [Log] Replace a string by the function name in a log (warrows)
977f089d0d2883ca3ca1648b14423b738c9248ef [Refactor] Use arrays instead of unic vars in Chacha20 (warrows)
d8abe323bd4f5cff3ca23fc00f0093ab7b143ead [Random] Add a missing include (warrows)
27663b8f16dfaac75bae1ab0e4a0c465b838c826 Do not permit copying FastRandomContexts (Pieter Wuille)
64e03e6de1a65c8d934d8df65af48e2b92bf887a Bugfix: randbytes should seed when needed (non reachable issue) (Pieter Wuille)
e8f12aa3621e98588b82533edb56ed4865645862 Check if sys/random.h is required for getentropy on OSX. (James Hilliard)
de85c7ae55681f8a9d322f4c48c6e9ba15ee0cf1 Add attribute [[noreturn]] (C++11) to functions that will not return (practicalswift)
df46c7ff5a2e6a71ba636fa39d77f58e018816a6 Fix resource leak (Dag Robole)
d426d856d23fe6c5d6ef3d57fe2c420b9a8e4e5e Clarify entropy source (Pieter Wuille)
30a320b3d12d3529dfca1daae0e477d76acd9ae7 Use cpuid intrinsics instead of asm code (Pieter Wuille)
0c21204e6dc714de6b917d9de12b1029d0bfb67b random: fix crash on some 64bit platforms (Cory Fields)
b8bbb9c68d996b1bea579956b3c91872d6d94b8f Use rdrand as entropy source on supported platforms (Pieter Wuille)
8e19443e41f748ad54d0f8aa25907bdad0815c19 [Tests] Fix compilation (warrows)
f53edec73901a91a6c4f595b248e4058e8be5468 [Rand/test] scripted-diff: Use new naming style for insecure_rand* functions (warrows)
272f3a5ac83ba0ab72a799ce524b6605b3555751 [Random / tests]  scripted-diff: Use randbits/bool instead of randrange (warrows)
0173ee3332990ac1eea134cca5c21cc451b81d92 Replace rand() & ((1 << N) - 1) with randbits(N) (Pieter Wuille)
250de7426e8317f86fc25fe60e960c2e9e903b69 Replace more rand() % NUM by randranges (Pieter Wuille)
d6904136030d94e844074fb75b9cafda46253ce0 [Random / tests] scripted-diff: use insecure_rand256/randrange more (warrows)
4a811ff9542439d43e3df3460114d9a24f849a36 Merge test_random.h into test_bitcoin.h (Pieter Wuille)
f275e638bf0a752d9f231f2e4a781432deafbfe8 Add various insecure_rand wrappers for tests (Pieter Wuille)
602af4fb3ef45a2a821f1b76d0dbd418ec852683 Add FastRandomContext::rand256() and ::randbytes() (Pieter Wuille)
90549785824f4c13ee8cf36e77f737b5d89d0f02 Add perf counter data to GetStrongRandBytes state in scheduler (Matt Corallo)
17dd13e746b567f44015fbefaa634b278010fccf Add internal method to add new random data to our internal RNG state (Matt Corallo)
c7a1602879aff2ae72bdefe959d4026ee8c0aefc Use sanity check timestamps as entropy (Pieter Wuille)
f671fe99994cb220dede903459307c61475aeef8 Test that GetPerformanceCounter() increments (Pieter Wuille)
dcb536fb2cef719abef9bb4104242410987bfd3d Use hardware timestamps in RNG seeding (Pieter Wuille)
7c3f290b50d32a3290a2c49542d72a3a7466da58 [Random] Fix compilation (warrows)
22b78957cc90252c958bc66439c098001d47b8fc random: only use getentropy on openbsd (Cory Fields)
e5750e5c9831edf01026643deed249e0efb28f0b Add a FastRandomContext::randrange and use it (Pieter Wuille)
2a0f6cd1ca2224e2ba30aea4ab312a1919dedc0c Switch FastRandomContext to ChaCha20 (Pieter Wuille)
401ca7db468939184681b73f4b346aebcfc378c6 Introduce FastRandomContext::randbool() (Pieter Wuille)
3d056d6ed478c5bbd42d7dde4f1ab0a75ffd42b1 Add ChaCha20 (Pieter Wuille)
3c97f3f2cb1b3dc2f5d642877d9a2de0a85c06bf Kill insecure_random and associated global state (Wladimir J. van der Laan)
68ba16c78593bf221844dae95fc5edacbf69ba62 Maintain state across GetStrongRandBytes calls (Pieter Wuille)
c3c399e199c2d25d10e782dca2ef17f55b6939d0 random: Add fallback if getrandom syscall not available (Wladimir J. van der Laan)
7a8111f915fc7ed23aaec86cd95904bf5d5972fd sanity: Move OS random to sanity check function (Wladimir J. van der Laan)
387c2e9574b56ae941b7723c561ed39488891782 squashme: comment that NUM_OS_RANDOM_BYTES should not be changed lightly (Wladimir J. van der Laan)
9e8c266b158c26b361d68a3cccdbcfd7bee0b376 util: Specific GetOSRandom for Linux/FreeBSD/OpenBSD (Wladimir J. van der Laan)
f989b865f2…

This reverts commit 565e630.

b5ef9be675 Merge #1: Merge changes from upstream
9e7f512430 Merge remote-tracking branch 'origin/master' into bitcoin-fork
1f85030246 Add support for ARM64 darwin (#43)
3bb959c982 Remove unnecessary reinterpret_cast (#42)
2e97ab26b1 Fix (unused) ReadUint64LE for BE machines (#41)
47b40d2209 Bump dependencies. (#40)
ba74185625 Move CI to Visual Studio 2019.
efa301a7e5 Allow different C/C++ standards when this is used as a subproject.
cc6d71465e CMake: Use configure_package_config_file()

git-subtree-dir: src/crc32c
git-subtree-split: b5ef9be6755a2e61e2988bb238f13d1c0ee1fa0a

EPIC: Grand library refactoring and revising

update build script