Return improved error message when trying to spend Coinbase coins (#1373).

Extra parameter added to AvailableCoins to include or exclude Coinbase coins.
SelectCoins, used for sending taddr->taddr, will exclude Coinbase coins.

Added qa rpc test and a runtime parameter -regtestprotectcoinbase to enforce
the coinbase->zaddr consensus rule in regtest mode.

Auto merge of #1492 - arcalinea:document-wallet-reindex, r=bitcartel

Document wallet reindexing for z_importkey description in payment-api.md

Documents how to enable funds for imported keys  #1395

Rm beta 1 release note about encrypted wallets

Document wallet reindexing for z_importkey description in payment-api.md

Update release process to sign release tags

v1.0.0-beta2

Auto merge of #1458 - str4d:1386-remove-findanddelete-and-codeseparator, r=ageis

Remove FindAndDelete and disable OP_CODESEPARATOR

Closes #1386

Auto merge of #1480 - ageis:update-release-process, r=str4d

Adds note about updating guide during testnet deployment

Auto merge of #1477 - zcash:812.change-address-prefixes.0, r=str4d

Update address prefixes

Closes #812 and #1450

Adds note about updating guide during testnet deployment

Repair rpc-tests/signrawtransactions.py.

Signed-off-by: Daira Hopwood <[email protected]>

Repair bitcoin-util-test.

Signed-off-by: Daira Hopwood <[email protected]>

Update address in Founders' Reward gtest

Fix zkey test

Auto merge of #1476 - ebfull:update-parameters, r=str4d

Update to beta2 public parameters

Also gets rid of the `testnet3`/`regtest` subdirectories/symlinks and other such things.

Auto merge of #1474 - ageis:update-release-process, r=str4d

Inserts some notes related to testnet deployment

Should satisfy #959 for now. I don't want to get too detailed here, since the infrastructure automation is in a still non-public repository, but instead supply a basic overview. We can write more detailed documentation and attach it to that repo.

Auto merge of #1429 - str4d:867-update-dependencies, r=str4d

Update all dependencies

Also re-enables disabled compiler warnings.

Closes #867, #944, #1241, #1255, #1348.

Fix secp256k1 test compilation

Re-encode Founders' Reward keys

Closes #1450

Re-encode hard-coded addresses in tests

CBitcoinAddress should use nVersionBytes == 2.

Signed-off-by: Daira Hopwood <[email protected]>

Reencode keys in JSON test data. refs #812

Signed-off-by: Daira Hopwood <[email protected]>

WIP: update address prefixes. refs #812

Signed-off-by: Daira Hopwood <[email protected]>

Auto merge of #1427 - bitcartel:master_1345_getbalance, r=bitcartel

Fixes #1345 so that RPC getbalance star calculates balance of UTXOs correctly.

Mark previously-valid test data as invalid

Update to `beta2` public parameters, remove `regtest`/`testnet3` parameters
subdirectories.

Auto merge of #1374 - str4d:1106-alert-safe-mode, r=str4d

Enable high-priority alerts to put the RPC into safe mode

This reverts the changes in 986b5e257e2bb9d7aaed5111ca335732f8808b2d and adds a priority check.

Continuation of #1337
Closes #1106

Merge pull request #1473 from arcalinea/update-documentation

Update documentation

Final edits

Fixes #1345 so that UTXO debit and credits are computed correctly for a transaction.

Inserts some notes related to testnet deployment

Update wording

Auto merge of #1468 - str4d:1138-change-versioning-scheme, r=str4d

Change versioning scheme

Closes #1138

Merge pull request #4 from daira/1138-change-versioning-scheme

Repair FormatSubVersion tests. refs #1138

Repair FormatSubVersion tests. refs #1138

Signed-off-by: Daira Hopwood <[email protected]>

Auto merge of #1467 - ageis:wget-cert-verify, r=str4d

Verify TLS certificates w/ wget in fetch-params.sh

Per NCC-2016-017, wget was being run with `--no-check-certificate`, which means that the connection can be man-in-the-middled, even if we are verifying the integrity of the parameters later via hash sums.
The rationale cited in the Bash comments does not stand up to scrutiny. There's really no persuasive reason not to verify certificates.

Fixes #1346.

Formatting fix

Auto merge of #1398 - bitcartel:master_fr48, r=bitcartel

Add multiple founders' reward addresses

./zcash-gtest --gtest_filter="founders_reward_test.*"

Closes #1405.

Update mainnet addresses used for testing to have the correct number

Update release process with version schema

Use CLIENT_VERSION_BUILD to represent -beta and -rc in client version

Downgrade bdb to 5.3.28

bdb 6.X was released under the AGPL, which is incompatible with MIT-licensed
software (the result must be licensed under AGPL). bdb 5.X uses the same license
as bdb 4.8, and thus retains the same compatibility as in upstream Bitcoin.

Thanks to Luke-Jr for raising this issue.

Add field fMinerTestModeForFoundersRewardScript to chainparams

Add mainnet 2-of-3 multisig addresses for testing.

Update testnet founders reward addresses

Update founders reward test to output path of temporary wallet.dat file which contains keys which can be used for testing founders reward addresses.

Update comment per review

Refactor and fix per review

Refactor to add test to verify number of rewards each mainnet address will receive

Founders reward: Refactor test and formatting per review.

Founders reward: changed index computation, added new test and some refactoring.

Add founders reward to ChainParams.
Fix bug where subsidy slow shift was ignored.

Auto merge of #1433 - str4d:gtest-expected-failures, r=str4d

Add a make command for checking expected failures

With this merged, we can add gtests prefixed with `DISABLED_`, and then configure Buildbot to run those tests but not require they pass in order to merge.

Verify TLS certificates w/ wget in fetch-params.sh

Per NCC-2016-017, wget was run with --no-check-certificate,
which means that the connection can be man-in-the-middled,
even if we are verifying the integrity of the params later via hash sums.
The rationale cited in the Bash comments does not stand up to scrutiny.
There's really no persuasive reason not to verify certificates.

Fixes #1346.

Remove OP_CODESEPARATOR from tests

Disable OP_CODESEPARATOR

Add section abt confs and reorgs to security-warnings.md

Auto merge of #1444 - str4d:1394-reindex-clear-witness-caches, r=str4d

Clear note witness caches on reindexing

This PR also fixes a test that was passing arguments in the wrong order.

Closes #1394

Ensure correctness if asserts are compiled out

Fix comment and formatting per review

Add tests for alerts enabling RPC safe mode

Link to z.cash on security-warnings.md

Update to use new API in CCryptoKeyStore and store a viewing key in walletdb.

Add support for spending keys to the encrypted wallet.

Simplify ClearNoteWitnessCache()

Use asserts to check allocation errors in CECKey::Recover

Remove no-autoalginit and no-autoerrinit OpenSSL flags

These flags are potentially risky, because they require that the app explicitly
initialize stuff that it wouldn't otherwise need to initialize, and we don't
have time for the necessary review.

Auto merge of #1370 - ageis:improve-fetch-params, r=daira

Improvements to fetch-params.sh

Two instances of the fetch-params.sh script running at once will result in corruption of the proving key and an error when the hashes are computed. This implements a lock to stop such a scenario from occurring.

We also terminate and don't create the symlinks if the downloaded parameters fail the checksum command.

Fixes #1306. This is the same as #1310 but now targeting master.

Auto merge of #1372 - str4d:1199-spending-key-wallet-encryption, r=str4d

Encrypt spending keys in the wallet

This is a continuation of #1210.

Auto merge of #1451 - zcash:daira-update-CoC-email, r=str4d

Update my email address in the Code of Conduct.

Auto merge of #1445 - bitcartel:master_fix_filtered_notes, r=str4d

Fix casting error in GetFilteredNotes

Use int for minDepth like upstream instead of size_t which can lead to casting problems if a wallet tx has a depth of -1.
Also don't use FindMyNotes as mapNoteData has already been set on wallet tx.

@str4d As dicussed. This should be merged before other PRs related to wallet.

Assert that new OpenSSL allocators succeed

Add libsnark to pre-release dependency checks

Disable nearly everything in OpenSSL

Flags obtained from:
https://gist.github.com/steakknife/8247726#gistcomment-1739713

Some flags not enabled because they aren't supported in OpenSSL 1.1.0b.

Flags not enabled because they break compilation:
- no-deprecated
- no-dh
- no-ec

Re-enable disabled compiler warnings

Reverts:
- be1bae14e8a769966b833026ed47a43f5250a4b9
- ec3bd70b6d6ef8be901a5523a401e333c52e7bbe

Closes #944

Replace auto_ptr with unique_ptr

Fix auto_ptr deprecation warning in Boost

Release process: check dependencies for updates

Update my email address in the Code of Conduct.

Upgrade ccache to 3.3.1

Upgrade miniupnpc to 2.0

Upgrade OpenSSL to 1.1.0b

Closes #1348

build: remove libressl check

Now that BIP66 passed, OpenSSL is no longer directly part of the
consensus. What matters is that DER signatures are correctly parsed, and
secp256k1 crypto is implemented correctly (as well as the other
functions we use from OpenSSL, such as random number generation)

This means that effectively, using LibreSSL is not a larger risk than
using another version of OpenSSL.

Remove the specific check for LibreSSL.

Includes the still-relevant part of #6729: make sure CHECK_HEADER is
called using the right CXXFLAGS, not CFLAGS (as AC_LANG is c++).

Zcash Note:
This includes a code removal from d9add719519e2019b1f353d8d6832dcfc88ee850
that was omitted from 5a3913361d0c045db69932ac5f152c4cf76e13cb during
backporting of bitcoin/bitcoin#6501 to 0.11.* in bitcoin/bitcoin#6703

[depends] OpenSSL 1.0.1k - update config_opts

Upgrade libgmp to 6.1.1

Upgrade Boost to 1.62.0

Part of #944
Closes #1241

Changes to upgrade bdb to 6.2.23

Closes #1255

Use correct lock for spending keys

Cache note decryptors in encrypted keystore

Update test to filter and find notes.

Auto merge of #1384 - str4d:1378-atomic-witness-storage, r=ebfull

Write note witness cache atomically to disk to avoid corruption

Closes #1378

Implement CSecureDataStream for streaming CKeyingMaterial

Test that invalid keys fail to unlock the keystore

Write note witness cache atomically to disk to avoid corruption

Closes #1378

Fix GetFilteredNotes to use int for minDepth like upstream and avoid casting problems. Don't use FindMyNotes as mapNoteData has already been set on wallet tx.

Clear note witness caches on reindex

Add wallet method to clear the note witness cache

Fix test

Remove FindAndDelete. refs #1386

Signed-off-by: Daira Hopwood <[email protected]>

Updates dns.testnet.z.cash -> dnsseed.testnet.z.cash

People seem to agree that dnsseed.* is a more intuitive address.
The original address was committed to beta1 as a result of a misunderstanding.
The DNS seeder service will be updated to coincide with at beta2 release.