diff options
author | dcherniev <dcherniev@github.com> | 2016-11-29 17:24:00 +0200 |
---|---|---|
committer | dcherniev <dcherniev@github.com> | 2016-11-29 17:24:00 +0200 |
commit | c0ae93ef89de99ba5e3f7b99fd6ba471c30f160c (patch) | |
tree | 6702075e4a52e7fd54e099a9de120050456e7f0c /src/components/transport_manager/src/transport_adapter | |
parent | 34fab90226b663bdeb0316b1a42fc4ed22bfacd0 (diff) | |
download | sdl_core-c0ae93ef89de99ba5e3f7b99fd6ba471c30f160c.tar.gz |
Fix issue of TransportManager UT
Moved initialization of info.connection field right after connection was created.
Added check if DisconnectDevice method would try to send Disconnect to invalid connection.
Related to: APPLINK-30284
Diffstat (limited to 'src/components/transport_manager/src/transport_adapter')
-rw-r--r-- | src/components/transport_manager/src/transport_adapter/threaded_socket_connection.cc | 1 | ||||
-rw-r--r-- | src/components/transport_manager/src/transport_adapter/transport_adapter_impl.cc | 8 |
2 files changed, 5 insertions, 4 deletions
diff --git a/src/components/transport_manager/src/transport_adapter/threaded_socket_connection.cc b/src/components/transport_manager/src/transport_adapter/threaded_socket_connection.cc index 5e9f3d941d..63cfb3ea51 100644 --- a/src/components/transport_manager/src/transport_adapter/threaded_socket_connection.cc +++ b/src/components/transport_manager/src/transport_adapter/threaded_socket_connection.cc @@ -162,7 +162,6 @@ TransportAdapter::Error ThreadedSocketConnection::Disconnect() { void ThreadedSocketConnection::threadMain() { LOG4CXX_AUTO_TRACE(logger_); - controller_->ConnectionCreated(this, device_handle(), application_handle()); ConnectError* connect_error = NULL; if (!Establish(&connect_error)) { LOG4CXX_ERROR(logger_, "Connection Establish failed"); diff --git a/src/components/transport_manager/src/transport_adapter/transport_adapter_impl.cc b/src/components/transport_manager/src/transport_adapter/transport_adapter_impl.cc index f0fddfb4f5..f1c047fe58 100644 --- a/src/components/transport_manager/src/transport_adapter/transport_adapter_impl.cc +++ b/src/components/transport_manager/src/transport_adapter/transport_adapter_impl.cc @@ -281,9 +281,11 @@ TransportAdapter::Error TransportAdapterImpl::DisconnectDevice( j != to_disconnect.end(); ++j) { ConnectionInfo info = *j; - if (OK != info.connection->Disconnect()) { - error = FAIL; - LOG4CXX_ERROR(logger_, "Error on disconnect " << error); + if (info.connection) { + if (OK != info.connection->Disconnect()) { + error = FAIL; + LOG4CXX_ERROR(logger_, "Error on disconnect " << error); + } } } |