diff options
| author | Wladimir J. van der Laan <[email protected]> | 2014-05-07 09:09:13 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2014-05-12 09:30:46 +0200 |
| commit | 0a0cd345520382bd726fef50c62864715b03f164 (patch) | |
| tree | aeb8a42d8fa0adae4e336d8f7488fa0aff65b568 /src/rpcserver.cpp | |
| parent | Merge pull request #4175 (diff) | |
| download | discoin-0a0cd345520382bd726fef50c62864715b03f164.tar.xz discoin-0a0cd345520382bd726fef50c62864715b03f164.zip | |
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.
Diffstat (limited to 'src/rpcserver.cpp')
| -rw-r--r-- | src/rpcserver.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/rpcserver.cpp b/src/rpcserver.cpp index ac40ea7cf..44f329dd1 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 |