aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPieter Wuille <pieter.wuille@gmail.com>2015-03-05 04:01:22 -0800
committerWladimir J. van der Laan <laanwj@gmail.com>2015-03-09 11:13:35 +0100
commitd5d89980284179436a5f3009a5346581fdaa9a01 (patch)
tree3a90734c4c2e0853054dd7fa030deb6fb27d1779
parent786cf72c7ff5d4042e7a7d010e96e53b0a597371 (diff)
downloadbitcoin-d5d89980284179436a5f3009a5346581fdaa9a01.tar.xz
Limit message sizes before transfer
This introduces a fixed limit for the size of p2p messages, and enforces it before download. Rebased-From: ba04c4a7801e7d68a5e84035b919e5c3626eb7a7 Github-Pull: #5843
-rw-r--r--src/net.cpp5
-rw-r--r--src/net.h2
2 files changed, 7 insertions, 0 deletions
diff --git a/src/net.cpp b/src/net.cpp
index 5c8ff79702..7ed9d65fb2 100644
--- a/src/net.cpp
+++ b/src/net.cpp
@@ -574,6 +574,11 @@ bool CNode::ReceiveMsgBytes(const char *pch, unsigned int nBytes)
if (handled < 0)
return false;
+ if (msg.in_data && msg.hdr.nMessageSize > MAX_PROTOCOL_MESSAGE_LENGTH) {
+ LogPrint("net", "Oversized message from peer=%i, disconnecting", GetId());
+ return false;
+ }
+
pch += handled;
nBytes -= handled;
diff --git a/src/net.h b/src/net.h
index a475be0b33..79d8baeb6d 100644
--- a/src/net.h
+++ b/src/net.h
@@ -46,6 +46,8 @@ static const int TIMEOUT_INTERVAL = 20 * 60;
static const unsigned int MAX_INV_SZ = 50000;
/** The maximum number of new addresses to accumulate before announcing. */
static const unsigned int MAX_ADDR_TO_SEND = 1000;
+/** Maximum length of incoming protocol messages (no message over 2 MiB is currently acceptable). */
+static const unsigned int MAX_PROTOCOL_MESSAGE_LENGTH = 2 * 1024 * 1024;
/** -listen default */
static const bool DEFAULT_LISTEN = true;
/** -upnp default */