From 1891245e7318bf625bbf67aab08a79fc3e87b61d Mon Sep 17 00:00:00 2001 From: Ben Woosley Date: Thu, 5 Mar 2020 04:44:45 -0500 Subject: refactor: Cast ping values to double before output Note the divisor is a floating point literal so presumably also floating point. --- src/rpc/net.cpp | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) (limited to 'src/rpc') diff --git a/src/rpc/net.cpp b/src/rpc/net.cpp index e8ae458c57..97fe99b8c1 100644 --- a/src/rpc/net.cpp +++ b/src/rpc/net.cpp @@ -165,12 +165,15 @@ static UniValue getpeerinfo(const JSONRPCRequest& request) obj.pushKV("bytesrecv", stats.nRecvBytes); obj.pushKV("conntime", stats.nTimeConnected); obj.pushKV("timeoffset", stats.nTimeOffset); - if (stats.m_ping_usec > 0) - obj.pushKV("pingtime", stats.m_ping_usec / 1e6); - if (stats.m_min_ping_usec < std::numeric_limits::max()) - obj.pushKV("minping", stats.m_min_ping_usec / 1e6); - if (stats.m_ping_wait_usec > 0) - obj.pushKV("pingwait", stats.m_ping_wait_usec / 1e6); + if (stats.m_ping_usec > 0) { + obj.pushKV("pingtime", ((double)stats.m_ping_usec) / 1e6); + } + if (stats.m_min_ping_usec < std::numeric_limits::max()) { + obj.pushKV("minping", ((double)stats.m_min_ping_usec) / 1e6); + } + if (stats.m_ping_wait_usec > 0) { + obj.pushKV("pingwait", ((double)stats.m_ping_wait_usec) / 1e6); + } obj.pushKV("version", stats.nVersion); // Use the sanitized form of subver here, to avoid tricksy remote peers from // corrupting or modifying the JSON output by putting special characters in -- cgit v1.2.3