diff options
author | MarcoFalke <falke.marco@gmail.com> | 2021-02-01 14:57:34 +0000 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2021-02-11 12:42:40 +0100 |
commit | 08dada84565ea5f49127123e356c82a150626f3c (patch) | |
tree | f118994b26354a51f0209c8c758c393aec464416 /src/rpc | |
parent | 95218ee95cdb4046ee7d622eac822e74d94314c7 (diff) | |
download | bitcoin-08dada84565ea5f49127123e356c82a150626f3c.tar.xz |
util: Disallow negative mocktime
Signed-off-by: practicalswift <practicalswift@users.noreply.github.com>
Github-Pull: #21043
Rebased-From: 3ddbf22ed179a2db733af4b521bec5d2b13ebf4b
Diffstat (limited to 'src/rpc')
-rw-r--r-- | src/rpc/misc.cpp | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/src/rpc/misc.cpp b/src/rpc/misc.cpp index 0c982317f5..00f7445cfa 100644 --- a/src/rpc/misc.cpp +++ b/src/rpc/misc.cpp @@ -360,13 +360,13 @@ static RPCHelpMan signmessagewithprivkey() static RPCHelpMan setmocktime() { return RPCHelpMan{"setmocktime", - "\nSet the local time to given timestamp (-regtest only)\n", - { - {"timestamp", RPCArg::Type::NUM, RPCArg::Optional::NO, UNIX_EPOCH_TIME + "\n" - " Pass 0 to go back to using the system time."}, - }, - RPCResult{RPCResult::Type::NONE, "", ""}, - RPCExamples{""}, + "\nSet the local time to given timestamp (-regtest only)\n", + { + {"timestamp", RPCArg::Type::NUM, RPCArg::Optional::NO, UNIX_EPOCH_TIME + "\n" + "Pass 0 to go back to using the system time."}, + }, + RPCResult{RPCResult::Type::NONE, "", ""}, + RPCExamples{""}, [&](const RPCHelpMan& self, const JSONRPCRequest& request) -> UniValue { if (!Params().IsMockableChain()) { @@ -381,7 +381,10 @@ static RPCHelpMan setmocktime() LOCK(cs_main); RPCTypeCheck(request.params, {UniValue::VNUM}); - int64_t time = request.params[0].get_int64(); + const int64_t time{request.params[0].get_int64()}; + if (time < 0) { + throw JSONRPCError(RPC_INVALID_PARAMETER, strprintf("Mocktime can not be negative: %s.", time)); + } SetMockTime(time); if (request.context.Has<NodeContext>()) { for (const auto& chain_client : request.context.Get<NodeContext>().chain_clients) { |