aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWladimir J. van der Laan <laanwj@gmail.com>2014-05-07 09:09:13 +0200
committerWladimir J. van der Laan <laanwj@gmail.com>2014-05-12 09:30:46 +0200
commit0a0cd345520382bd726fef50c62864715b03f164 (patch)
treeaeb8a42d8fa0adae4e336d8f7488fa0aff65b568
parent2e2f6855df83a8b1b3d155e8bf2c1d55e4fee4a6 (diff)
rpc: pass errors from async_accept
According to the [boost::asio documentation](http://www.boost.org/doc/libs/1_55_0/doc/html/boost_asio/reference/basic_socket_acceptor/async_accept/overload2.html), the function signature of the handler must be: void handler( const boost::system::error_code& error // Result of operation. ); We were binding *all* the arguments, instead of all but the error, resulting in nullary function that never got the error. Fix this by adding an input argument substitution.
-rw-r--r--src/rpcserver.cpp4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/rpcserver.cpp b/src/rpcserver.cpp
index ac40ea7cf1..44f329dd15 100644
--- a/src/rpcserver.cpp
+++ b/src/rpcserver.cpp
@@ -466,7 +466,7 @@ static void RPCListen(boost::shared_ptr< basic_socket_acceptor<Protocol, SocketA
boost::ref(context),
fUseSSL,
conn,
- boost::asio::placeholders::error));
+ _1));
}
@@ -490,6 +490,8 @@ static void RPCAcceptHandler(boost::shared_ptr< basic_socket_acceptor<Protocol,
if (error)
{
delete conn;
+ // TODO: Actually handle errors
+ LogPrintf("%s: Error: %s\n", __func__, error.message());
}
// Restrict callers by IP. It is important to