Compile with DEBUG_LOCKORDER to detect inconsistent lock orderings that can cause deadlocks

Remove unused ScanMessageStart function

src/makefile.unix: remove -DFOURWAYSSE2

Spotted by Venkatesh Srinivas <me@endeavour.zapto.org>

Don't std::advance past beginning of transactions array.  Fixes #465

Merge pull request #458 from TheBlueMatt/copyright

Unify copyright notices.

Merge pull request #459 from jgarzik/char-msgstart

Use 'unsigned char' rather than 'char' for pchMessageStart.

Merge pull request #460 from jgarzik/make-tuple

Qualify make_tuple with boost:: namespace.

Use 'unsigned char' rather than 'char' for pchMessageStart.

Regarding https://bitcointalk.org/index.php?topic=28022.0

main.cpp has: "char pchMessageStart[4] = { 0xf9, 0xbe, 0xb4, 0xd9 };"
Per discussion on the thread linked, leaving the signedness of
pchMessageStart is unsafe for values > 0x80. This patch specifies
'unsigned char' in main.cpp and net.h.

Signed-off-by: Jeff Garzik <jgarzik@pobox.com>

Qualify make_tuple with boost:: namespace.

db.cpp has a number of uses of make_tuple and has 'using namespace
std' and 'using namespace boost'. Without qualifying make_tuple,
std::make_tuple is preferred, which is incorrect. This patch qualifies
make_tuple.

Signed-off-by: Jeff Garzik <jgarzik@exmulti.com>

Test for SO_NOSIGPIPE rather than assuming all BSDs support it.

Signed-off-by: Jeff Garzik <jgarzik@exmulti.com>

Do-nothing MapPort() ifndef USE_UPNP.  fixes #450

Unify copyright notices.

To a variation on:
// Copyright (c) 2009-2010 Satoshi Nakamoto
// Copyright (c) 2011 The Bitcoin developers

Cleanup test suite output to be more useful.

Add file for transaction tests.

Add a file for script tests

Add prototype for EvalScript() to script.h

This is needed for the script unit tests.

Fix testing setup

There were some problems with the existing testing setup:

 - Makefile rules for test-file compilation used CFLAGS instead of
   CXXFLAGS in makefile.unix

Add missing include to serialize.h

Add missing includes to net.h

Add missing include to script.h

Add missing includes to key.h

Merge pull request #443 from TheBlueMatt/dupin

Check for duplicate txins in CheckTransaction.

Merge branch 'tmp2'

Merge pull request #448 from sipa/cbitcoinaddress

avoid strAddress + validity checks

Merge pull request #446 from WakiMiko/upnp-desc

Identify as "Bitcoin + version number" when mapping UPnP port

Merge pull request #451 from TheBlueMatt/tornolisten

Don't listen if on TOR (resolves #441).

Don't listen if on TOR (resolves #441).

Make it clear that setting proxy requires restart to fully apply.

avoid strAddress + validity checks

Avoid references to addresses using strings, and use CBitcoinAddress
as much as possible. Also added some validity checks on addresses
entered using RPC.

Merge pull request #444 from Jeroenz0r/patch-2

Translation from "Open Bitcoin" to "Verstuur Bitcoins"

Identify as "Bitcoin + version number" when mapping UPnP port

Makes Bitcoin identify itself as "Bitcoin + version number" instead of
the nondescript "libminiupnpc" when forwarding a port via UPnP.

Merge pull request #437 from codler/patch-2

Comment "deprecated"

Translation from "Open Bitcoin" to "Verstuur Bitcoins"

Check for duplicate txins in CheckTransaction.

Comment "deprecated"

Merge pull request #436 from TheBlueMatt/master

Fix EncryptKeys crash introduced by a9ba471, identified by TD.

Fix EncryptKeys crash introduced by a9ba4710, identified by TD.

CAddrDB::LoadAddresses: properly initialize CAddress

Fixes issue #424

Merge pull request #435 from sipa/fixextract

Bugfix: don't overuse limited ExtractAddress

Bugfix: don't overuse limited ExtractAddress

ExtractAddress was called with the keystore as argument in RPC and
UI, limiting results to own keys. This caused empty "address"
fields.

Merge pull request #434 from laanwj/addkey_retval

make SetHash160 return a value (as specified in the function signature)

make SetHash160 return a value (as specified in the function signature)

Fix UNIX-specific thread handle leak.

Merge pull request #403 from sipa/cbitcoinaddress

keys indexed by address + introduced CBitcoinaddress

Merge pull request #428 from sipa/mintxfeefix

Fix for small change outputs

Fix for small change outputs

With the separation of CENT and MIN_TX_FEE, it is now reasonable
to create change outputs between 0.01 and 0.0005, as these are
spendable according to the policy, even though they require a fee
to be paid.

Also, when enough fee was already present, everything can go into
a change output, without further increasing the fee.

Merge pull request #423 from TheBlueMatt/mapalreadyaskedforfix

Actually use mapAlreadyAskedFor.

Actually use mapAlreadyAskedFor.

Previously, mapAlreadyAskedFor was read from, but never added to.
The original intent was to use mapAlreadyAskedFor to keep track
of the time an item was requested and "Each retry is 2 minutes
after the last".
This implements that intent.

Merge pull request #1 from AbrahamJewowich/cbitcoinaddress

Cbitcoinaddress

Fix bug with accessing vchData[0] when vchData is empty.
Fix typo in CBase58Data::CompareTo

Merge pull request #419 from mibe/translation-de

Updated German translation

Merge pull request #395 from mibe/issue-395

Error in Russian Translation

split off CBase58Data from CBitcoinAddress

Split off features unrelated to addresses from CBitcoinAddress to
CBase58Data, so they can be reused.

Use CBitcoinAddress instead of string/uint160

Instead of conversion functions between pubkey/uint160/address in
base58.h, have a fully fledged class CBitcoinAddress (CAddress was
already taken) to represent addresses.

get rid of mapPubKeys

Make CKeyStore's interface work on uint160's instead of pubkeys, so
no separate global mapPubKeys is necessary anymore.

Changed Russian translation according to comment in issue 395

Added German wallet encryption messages translation.

Basically some grammatical fixes of the German translation.

Merge pull request #418 from EricJ2190/master

Update makefile.vc for wallet encryptions.

Added crypter to makefile.vc.

Merge pull request #414 from spiechu/polish-translation

added polish wallet encryption messages

added polish wallet encryption messages

Merge pull request #413 from TheBlueMatt/loadwalletret

Fix bad return values in LoadWallet.

Fix bad return values in LoadWallet.

Merge pull request #412 from sgimenez/db-transactions

Single DB transactions

Single DB transaction for addresses from DNS seeds

Merge pull request #411 from TheBlueMatt/master

Fix Build in GetReservedKey() in wallet.cpp

Fix Build in GetReservedKey() in wallet.cpp

Merge pull request #410 from TheBlueMatt/master

Generate Warning when using default key.

Single DB transaction for all addresses in a message

Cuts disk activity at startup immensely

Generate Warning when using default key.

Merge pull request #409 from TheBlueMatt/master

Fix crashes when a wallet is locked and GetReservedKey() is called

Fix crashes when a wallet is locked and GetReservedKey() is called

Merge pull request #406 from muggenhor/warning-fixes

Warning fixes

Merge pull request #405 from sipa/syncdefault

Bugfix: add autogenerated addresses to address book

fix warning: control reaches end of non-void function [-Wreturn-type]

Signed-off-by: Giel van Schijndel <me@mortis.eu>

fix warning: variable ‘nMinDepth’ set but not used [-Wunused-but-set-variable]

Signed-off-by: Giel van Schijndel <me@mortis.eu>

Merge pull request #404 from TheBlueMatt/newenc

Fix makefile.linux-mingw

Fix makefile.linux-mingw

Bugfix: add autogenerated addresses to address book

doc/README: word wrap into something readable

Bump version to 0.3.25

Yes, we might release as v0.4, but let's just do a simple increment
for now.

Update CWallet::LoadWallet for proper return type.

Merge pull request #399 from muggenhor/warning-fixes

Warning fixes

fix warning: unused function 'SigIllHandlerSSE2' [-Wunused-function]

Only declare & define SigIllHandlerSSE2 when its used.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

fix warning: unused variable 'X' [-Wunused-variable]

Remove several unused variables.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

fix warning: X enumeration values not handled in switch [-Wswitch-enum]

Add default cases to opcode switches to assert that they should never
occur.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

fix warning: comparison of unsigned expression < 0 is always false [-Wtautological-compare]

Don't check for a negative parameter count, because not only will it
never happen, it doesn't make any sense either.

Invalid sockets (as returned by socket(2)) are always exactly -1 (not
just negative as negative file descriptors are technically not
prohibited by POSIX) on POSIX systems.  Since we store them in SOCKET
(unsigned int), however, that really is ~0U (or MAX_UINT) which happens
to be what INVALID_SOCKET is already defined to, so an additional check
for being negative is not only unnecessary (unsigned integers aren't
*ever* negative) its redundant as well (the INVALID_SOCKET comparison is
enough).

Signed-off-by: Giel van Schijndel <me@mortis.eu>

fix warnings: using the result of an assignment as a condition without parentheses [-Wparentheses]

Don't unnecessarily assign to variables within the *boolean* expression
of a conditional.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

fix warnings: expression result unused [-Wunused-value]

In the assert()s take advantage of the fact that string constants
("string") are effectively of type 'const char []', which when used in
an expression yield a non-NULL pointer.

An assertion that should always fail can thus be formulated as:
  assert(!"fail);

An assertion where a text message should be added to the expression can
be written as such:
  assert("message" && expression);

Signed-off-by: Giel van Schijndel <me@mortis.eu>

fix warning on 64bit systems: cast to pointer from integer of different size [-Wint-to-pointer-cast]

Signed-off-by: Giel van Schijndel <me@mortis.eu>

Update makefile.linux-mingw to work with crypter and UPnP fix.

Merge pull request #381 from TheBlueMatt/nminversion

Add minversion to wallet.

Merge pull request #402 from jayschwa/hirez-icon

High resolution Windows icon

Merge pull request #396 from jayschwa/nsis-branding

Add bitcoin.org logos/branding to NSIS installer.

Merge pull request #352 from TheBlueMatt/newenc

Wallet Private Key Encryption (on CWallet)

Increase resolution of Windows icon.

The .ico file has changed in the following ways:
* Added 64x64 layer (max size for "Classic Mode").
* Added 256x256 layer (max size for Vista and 7).
* Removed copies with no alpha channel:
  * Display depths lower than 32-bits are rare nowadays.
  * 8-bit alpha channels in icons has been supported since XP.
  * If the display depth is lowered, they look no better than the
    downsampled versions that Windows automatically generates.

Tested various sizes on both XP and Win 7. It looks fine
(unchanged) on XP and downright sexy on Win 7.

Make an invalid addrIncoming so that old clients crash.

This prevents old clients from opening, and thus corrupting
or otherwise causing harm to encrypted wallets.

Use DB Transactions when encrypting wallet.
This speeds up the encryption process significantly.

Do not use obsolete CPrivKey for passing keys around