From 13e04977ac19a3f8aacd755bab506d0424d06c08 Mon Sep 17 00:00:00 2001 From: Jianpeng Ma Date: Tue, 30 Apr 2019 15:29:08 +0800 Subject: [PATCH] test/crimson: fix a compiler error. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit When with -DWITH_SEASTAR=ON, met the following bug: /home/ceph/src/test/crimson/test_alien_echo.cc:164:13: error: ‘client’ was not declared in this scope client.msgr->set_require_authorizer(false); ^~~~~~ /home/ceph/src/test/crimson/test_alien_echo.cc:164:13: note: suggested alternative: ‘client_t’ client.msgr->set_require_authorizer(false); ^~~~~~ client_t /home/ceph/src/test/crimson/test_alien_echo.cc: In instantiation of ‘seastar_echo(entity_addr_t, echo_role, unsigned int):::: [with auto:48 = seastar_pingpong::Client; auto:47 = ceph::net::Messenger*]’: /usr/include/c++/7/type_traits:2428:26: required by substitution of ‘template static std::__result_of_success()((declval<_Args>)()...)), std::__invoke_other> std::__result_of_other_impl::_S_test(int) [with _Fn = seastar_echo(entity_addr_t, echo_role, unsigned int):: [with auto:47 = ceph::net::Messenger*]::; _Args = {seastar_pingpong::Client&}]’ /usr/include/c++/7/type_traits:2439:55: required from ‘struct std::__result_of_impl [with auto:47 = ceph::net::Messenger*]::, seastar_pingpong::Client&>’ /usr/include/c++/7/type_traits:2444:12: required from ‘struct std::__invoke_result [with auto:47 = ceph::net::Messenger*]::, seastar_pingpong::Client&>’ /usr/include/c++/7/type_traits:2457:12: required from ‘class std::result_of [with auto:47 = ceph::net::Messenger*]::(seastar_pingpong::Client&)>’ /home//ceph/src/seastar/include/seastar/core/do_with.hh:93:17: required from ‘auto seastar::do_with(T&&, F&&) [with T = seastar_pingpong::Client; F = seastar_echo(entity_addr_t, echo_role, unsigned int):: [with auto:47 = ceph::net::Messenger*]::]’ /home/ceph/src/test/crimson/test_alien_echo.cc:184:32: required from ‘seastar_echo(entity_addr_t, echo_role, unsigned int):: [with auto:47 = ceph::net::Messenger*]’ /usr/include/c++/7/type_traits:2428:26: required by substitution of ‘template static std::__result_of_success()((declval<_Args>)()...)), std::__invoke_other> std::__result_of_other_impl::_S_test(int) [with _Fn = seastar_echo(entity_addr_t, echo_role, unsigned int)::; _Args = {ceph::net::Messenger*&&}]’ /usr/include/c++/7/type_traits:2439:55: required from ‘struct std::__result_of_impl, ceph::net::Messenger*&&>’ /usr/include/c++/7/type_traits:2444:12: required from ‘struct std::__invoke_result, ceph::net::Messenger*&&>’ /usr/include/c++/7/type_traits:2457:12: required from ‘class std::result_of(ceph::net::Messenger*&&)>’ /home//ceph/src/seastar/include/seastar/core/future.hh:980:30: required by substitution of ‘template requires CanApply Result seastar::future::then(Func&&) [with Func = seastar_echo(entity_addr_t, echo_role, unsigned int)::; Result = ]’ /home//ceph/src/test/crimson/test_alien_echo.cc:207:8: required from here /home/ceph/src/test/crimson/test_alien_echo.cc:190:13: error: base operand of ‘->’ has non-pointer type ‘ceph::net::Messenger’ client.msgr->set_require_authorizer(false); ^~~~~~ Signed-off-by: Jianpeng Ma --- src/test/crimson/test_alien_echo.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/crimson/test_alien_echo.cc b/src/test/crimson/test_alien_echo.cc index 5b98350ca1b..a368b93daab 100644 --- a/src/test/crimson/test_alien_echo.cc +++ b/src/test/crimson/test_alien_echo.cc @@ -161,7 +161,7 @@ seastar_echo(const entity_addr_t addr, echo_role role, unsigned count) server.msgr.set_default_policy(ceph::net::SocketPolicy::stateless_server(0)); server.msgr.set_policy_throttler(entity_name_t::TYPE_OSD, &server.byte_throttler); - client.msgr->set_require_authorizer(false); + server.msgr.set_require_authorizer(false); server.msgr.set_auth_client(&server.dummy_auth); server.msgr.set_auth_server(&server.dummy_auth); return server.msgr.bind(entity_addrvec_t{addr}) @@ -187,7 +187,7 @@ seastar_echo(const entity_addr_t addr, echo_role role, unsigned count) client.msgr.set_default_policy(ceph::net::SocketPolicy::lossy_client(0)); client.msgr.set_policy_throttler(entity_name_t::TYPE_OSD, &client.byte_throttler); - client.msgr->set_require_authorizer(false); + client.msgr.set_require_authorizer(false); client.msgr.set_auth_client(&client.dummy_auth); client.msgr.set_auth_server(&client.dummy_auth); return client.msgr.start(&client.dispatcher) -- 2.39.5