Avoid reject message feedback loops

Merge pull request #4885

aa3c697 Store fewer orphan tx by default, add -maxorphantx option (Gavin Andresen)
c74332c Stricter handling of orphan transactions (Gavin Andresen)

Merge pull request #4892

e982b57 Use explicit fflush() instead of setvbuf() (Adam Weiss)

Store fewer orphan tx by default, add -maxorphantx option

There is no reason to store thousands of orphan transactions;
normally an orphan's parents will either be broadcast or
mined reasonably quickly.

This pull drops the maximum number of orphans from 10,000 down
to 100, and adds a command-line option (-maxorphantx) that is
just like -maxorphanblocks to override the default.

Stricter handling of orphan transactions

Prevent denial-of-service attacks by banning
peers that send us invalid orphan transactions
and only storing orphan transactions given to
us by a peer while the peer is connected.

Use explicit fflush() instead of setvbuf()

Flushing after every line when printing to console is desirable when
running with systemd but setvbuf() has slightly different semantics
on Windows that causes warnings.  Just do an explicit fflush() after
each line print to console instead.

Fix crashing bug caused by orphan(s) with duplicate prevout.hash

Merge pull request #4878

540ac45 Avoid returning many "inv" orphans (Jeff Garzik)
d4168c8 Limit CNode::mapAskFor (Wladimir J. van der Laan)

Merge pull request #4623

e84843c Broken addresses on command line no longer trigger testnet. (Ross Nicoll)

Merge pull request #4755

c1e433b Rename scriptutils.o to wallet_ismine.o (jtimon)
8b59a3d Move CAffectedKeysVisitor to wallet.cpp (remove ExtractAffectedKeys) (jtimon)
0d2fa14 Move scriptutils.o to wallet (jtimon)

Rename scriptutils.o to wallet_ismine.o

Move CAffectedKeysVisitor to wallet.cpp (remove ExtractAffectedKeys)

Move scriptutils.o to wallet

Merge pull request #4874

21f139b qt: fix tablet crash. closes #4854. (Cory Fields)

Merge pull request #4867

a95b119 qt: Remove thousands separators after decimal point (Wladimir J. van der Laan)

Merge pull request #4822

629d75f Combine CCoinsViewCache's HaveCoins and const GetCoins into AccessCoins. (Pieter Wuille)

Merge pull request #4882

faadbe1 remove unneeded cast in rpcmisc.cpp (Philip Kaufmann)

remove unneeded cast in rpcmisc.cpp

Merge pull request #4868

2c2cc5d Remove some unnecessary c_strs() in logging and the GUI (Philip Kaufmann)
f7d0a86 netbase: Use .data() instead of .c_str() on binary string (Wladimir J. van der Laan)

Remove some unnecessary c_strs() in logging and the GUI

Includes `core: remove unneeded c_str() / Qt: replace c_str() with Qt code`
by P. Kaufmann.

Avoid returning many "inv" orphans

Limit CNode::mapAskFor

Tighten resource constraints on CNode.

Merge pull request #4873

89d91f6 Avoid repeated lookups in mapOrphanTransactions and mapOrphanTransactionsByPrev (Wladimir J. van der Laan)

qt: fix tablet crash. closes #4854.

This backports the relevant parts of:
https://codereview.qt-project.org/#/c/82689/

Merge pull request #4754

e088d65 Separate script/sign (jtimon)
9294a4b Separate CScriptCompressor (jtimon)
c4408a6 Separate script/standard (jtimon)
da03e6e Separate script/interpreter (jtimon)
cbd22a5 Move CScript class and dependencies to script/script (jtimon)
86dbeea Rename script.h/.cpp to scriptutils.h/.cpp (plus remove duplicated includes) (jtimon)

Rebased-by: Pieter Wuille

Separate script/sign

Separate CScriptCompressor

Separate script/standard

Separate script/interpreter

Move CScript class and dependencies to script/script

Rename script.h/.cpp to scriptutils.h/.cpp (plus remove duplicated includes)

Avoid repeated lookups in mapOrphanTransactions and mapOrphanTransactionsByPrev

Merge pull request #4865

8e44f2e Clarify 'compressed nor uncompressed' error message (Peter Todd)

netbase: Use .data() instead of .c_str() on binary string

`.c_str()` is only guaranteed to return the data up to the first NUL
character.

Merge pull request #4712

80daee0 [Qt] Call checkBalanceChanged() periodically instead for every updated transaction (Cozz Lovan)

Merge pull request #4697

da2ede2 [Wallet] Improve ReorderTransactions(..) (Cozz Lovan)

qt: Remove thousands separators after decimal point

Revert thousands separators after decimal point, as introduced in #4167.

Merge pull request #4862

45a4baf Add testnet DNS seed of Andreas Schildbach. (Andreas Schildbach)

Clarify 'compressed nor uncompressed' error message

Add testnet DNS seed of Andreas Schildbach.

It runs sipas crawler, but rather than using its custom nameserver implementation it serves a generated zonefile via bind9. The zone always contains 25 IPv4 and 25 IPv6 peers. FWIW, the zone is secured using DNSSEC.

Merge pull request #4851

41ef558 univalue: make spaceStr thread-safe (Wladimir J. van der Laan)

Merge pull request #4839

346193b Cleanup messy error messages (R E Broadley)

Cleanup messy error messages

Merge pull request #4849

a264e44 remove dup include of foreach.hpp in script.cpp (Philip Kaufmann)

Merge pull request #4836 from morcos/fix-make_change

Fix make_change to not create half-satoshis

Fix make_change to not create half-satoshis

univalue: make spaceStr thread-safe

Simply add spaces to the existing string instead of using a
temporary.

Fixes #4756.

Improve readability of CAddrInfo::IsTerrible

- Replaced 86400 with 24*60*60
- Remove references to specific timespans in comments

Github-Pull: #4724

Merge pull request #4850

1ffb99b [Qt] copyright, style and indentation cleanup of Qt tests (Philip Kaufmann)

[Qt] copyright, style and indentation cleanup of Qt tests

Merge pull request #4843

81061ff travis: add osx build (Cory Fields)
e4cf9f8 depends: fix typo in source downloading. Regression from c897b1e732. (Cory Fields)

remove dup include of foreach.hpp in script.cpp

Merge pull request #4838

1e4f87f Use memcmp for uint256 equality/inequality (Pieter Wuille)
8a41e1e Use boost::unordered_map for mapBlockIndex (Pieter Wuille)
145d5be Introduce BlockMap type for mapBlockIndex (Pieter Wuille)
a0dbe43 checkpoints.cpp depends on main, it can use mapBlockIndex directly (Pieter Wuille)

travis: add osx build

depends: fix typo in source downloading. Regression from c897b1e732.

Some sources are renamed after download, since the filenames don't play nice
with (for example) gitian. This fixes the rename.

Needed for OSX build as it renames a file.

Merge pull request #4783

fbe0fca [Qt] minor watch-only changes (Philip Kaufmann)

Merge pull request #4762

399cdbc contrib/linearize: Add feature to set file's timestamp based on block header time. (Jeff Garzik)
8f5a423 contrib/linearize: split block files based on year-month, not just year (Jeff Garzik)
75400a2 contrib/linearize: Guarantee that output is generated in-order (Jeff Garzik)

contrib/linearize: Add feature to set file's timestamp based on block header time.

Merge pull request #4833

bbda402 net: Remove MilliSleep from StopNode (Wladimir J. van der Laan)

Merge pull request #4790

9189f5f remove useless millisleep (phantomcircuit)

Merge pull request #4842

fb51e28 Added XML syntax highlighting (himynameismartin)

Added XML syntax highlighting

Use memcmp for uint256 equality/inequality

Use boost::unordered_map for mapBlockIndex

Introduce BlockMap type for mapBlockIndex

checkpoints.cpp depends on main, it can use mapBlockIndex directly

Merge pull request #4808

3f6540a Rename IMPLEMENT_SERIALIZE to ADD_SERIALIZE_METHODS (Pieter Wuille)
47eb765 Serializer simplifications after IMPLEMENT_SERIALIZE overhaul (Pieter Wuille)

Merge pull request #4820

910526d Use OR of respective block flags for masks (Pieter Wuille)

Merge pull request #4671

550d4fa Remove DNS Seeds run by entities which were never well-established. (Matt Corallo)

Combine CCoinsViewCache's HaveCoins and const GetCoins into AccessCoins.

The efficient version of CCoinsViewCache::GetCoins only works for known-to-exist
cache entries, requiring a separate HaveCoins call beforehand. This is
inefficient as both perform a hashtable lookup.

Replace the non-mutable GetCoins with AccessCoins, which returns a potentially-NULL
pointer. This also decreases the overloading of GetCoins.

Also replace some copying (inefficient) GetCoins calls with equivalent AccessCoins,
decreasing the copying.

net: Remove MilliSleep from StopNode

I don't understand why it would be there in the first place. This looks
like voodoo, not programming.

Remove DNS Seeds run by entities which were never well-established.

remove useless millisleep

reduces time to service requests improving performance

Use OR of respective block flags for masks

Merge pull request #4812

53efb09 Discover some missing includes (jtimon)
8d5e510 Remove unused function StackString() and class CCoins; (jtimon)

Merge pull request #4810

c0f5d4a Fixing Compiler Error C2466 (ENikS)

Rename IMPLEMENT_SERIALIZE to ADD_SERIALIZE_METHODS

Discover some missing includes

Remove unused function StackString() and class CCoins;

Fixing Compiler Error C2466

Serializer simplifications after IMPLEMENT_SERIALIZE overhaul

Merge pull request #4737

31e9a83 Use CSizeComputer to avoid counting sizes in SerializationOp (Pieter Wuille)
84881f8 rework overhauled serialization methods to non-static (Kamil Domanski)
5d96b4a remove fields of ser_streamplaceholder (Kamil Domanski)
3d796f8 overhaul serialization code (Kamil Domanski)

Merge pull request #4779

093303a add missing header end comments (Philip Kaufmann)

Merge pull request #4767

b144a74 depends: bump miniupnpc to 1.9.20140701. (Cory Fields)
f628127 depends: bump openssl to 1.0.1i (Cory Fields)
9f7f504 build: add -DMINIUPNP_STATICLIB for new version (Cory Fields)

Merge pull request #4780

4d04492 add missing copyright headers (Philip Kaufmann)

Merge pull request #4776

6f5d33b Update translations after update script improvements (Wladimir J. van der Laan)
da59f28 Add deeper XML checking to update-translation script (Wladimir J. van der Laan)

Update translations after update script improvements

Add deeper XML checking to update-translation script

- Catch problems such as mismatched formatting characters. Remove
  messages that can give problems at runtime.

- Also remove unfinished/untranslated messages, they just take up space
  in the ts and waste parsing time.

Fixes #4774.

Merge pull request #4768

2e28031 Perform CVerifyDB on pcoinsdbview instead of pcoinsTip (Wladimir J. van der Laan)

Use CSizeComputer to avoid counting sizes in SerializationOp

rework overhauled serialization methods to non-static

Thanks to Pieter Wuille for most of the work on this commit.
I did not fixup the overhaul commit, because a rebase conflicted
with "remove fields of ser_streamplaceholder".
I prefer not to risk making a mistake while resolving it.

remove fields of ser_streamplaceholder

The nType and nVersion fields of stream objects are never accessed
from outside the class (or perhaps from the inside too, I haven't checked).
Thus no need to have them in a placeholder, whose only purpose is to
fill the "Stream" template parameter in serialization implementation.

overhaul serialization code

The implementation of each class' serialization/deserialization is no longer
passed within a macro. The implementation now lies within a template of form:

template <typename T, typename Stream, typename Operation>
inline static size_t SerializationOp(T thisPtr, Stream& s, Operation ser_action, int nType, int nVersion) {
    size_t nSerSize = 0;
    /* CODE */
    return nSerSize;
}

In cases when codepath should depend on whether or not we are just deserializing
(old fGetSize, fWrite, fRead flags) an additional clause can be used:
bool fRead = boost::is_same<Operation, CSerActionUnserialize>();

The IMPLEMENT_SERIALIZE macro will now be a freestanding clause added within
class' body (similiar to Qt's Q_OBJECT) to implement GetSerializeSize,
Serialize and Unserialize. These are now wrappers around
the "SerializationOp" template.

Broken addresses on command line no longer trigger testnet.

When passing a bitcoin: URI on the command line, invalid addresses do not incorrectly send the
user to the test network.

changed field types in some structures to equivalent unambiguous types

Conflicts:
src/core.cpp

Rebased-By: Wladimir J. van der Laan
Github-Pull: #4180

Merge pull request #4789

bac5586 Replace weird characters by normal spaces (Pieter Wuille)

Merge pull request #4781

187115c cleanup include of assert.h (Philip Kaufmann)

Replace weird characters by normal spaces

Merge pull request #4778

Merge pull request #4788