summaryrefslogtreecommitdiff
path: root/src/components/connection_handler
diff options
context:
space:
mode:
authorAnton Hrytsevich <ahrytsevich>2015-12-25 16:06:19 +0200
committerAnton Hrytsevich <ahrytsevich>2015-12-25 16:06:19 +0200
commit8304f61a7b4a6bd4257d0105ce9ef166abef0d2a (patch)
treeb3c41ef4cb762a9a5cd9152b6e3cc9e689f54e55 /src/components/connection_handler
parent968ac8303084222fcf3be8c42f03f0db843c59d8 (diff)
downloadsdl_core-8304f61a7b4a6bd4257d0105ce9ef166abef0d2a.tar.gz
Make if statment more wide after review
Make more wide if statment. It include map search.
Diffstat (limited to 'src/components/connection_handler')
-rw-r--r--src/components/connection_handler/src/connection_handler_impl.cc34
1 files changed, 18 insertions, 16 deletions
diff --git a/src/components/connection_handler/src/connection_handler_impl.cc b/src/components/connection_handler/src/connection_handler_impl.cc
index f1976c0c40..45215be7c9 100644
--- a/src/components/connection_handler/src/connection_handler_impl.cc
+++ b/src/components/connection_handler/src/connection_handler_impl.cc
@@ -796,23 +796,25 @@ void ConnectionHandlerImpl::CloseSession(ConnectionHandle connection_handle,
protocol_handler_->SendEndSession(connection_handle, session_id);
}
- SessionMap::const_iterator session_map_itr = session_map.find(session_id);
- if (session_map_itr != session_map.end() && connection_handler_observer_) {
- const uint32_t session_key = KeyFromPair(connection_id, session_id);
- const Session &session = session_map_itr->second;
- const ServiceList &service_list = session.service_list;
-
- ServiceList::const_iterator service_list_itr = service_list.begin();
- for (;service_list_itr != service_list.end(); ++service_list_itr) {
- const protocol_handler::ServiceType service_type =
- service_list_itr->service_type;
- connection_handler_observer_->OnServiceEndedCallback(session_key,
- service_type,
- close_reason);
+ if ( connection_handler_observer_ ) {
+ SessionMap::const_iterator session_map_itr = session_map.find(session_id);
+ if (session_map_itr != session_map.end()) {
+ const uint32_t session_key = KeyFromPair(connection_id, session_id);
+ const Session &session = session_map_itr->second;
+ const ServiceList &service_list = session.service_list;
+
+ ServiceList::const_iterator service_list_itr = service_list.begin();
+ for (;service_list_itr != service_list.end(); ++service_list_itr) {
+ const protocol_handler::ServiceType service_type =
+ service_list_itr->service_type;
+ connection_handler_observer_->OnServiceEndedCallback(session_key,
+ service_type,
+ close_reason);
+ }
+ } else {
+ LOG4CXX_ERROR(logger_, "Session with id: " << session_id << " not found");
+ return;
}
- } else {
- LOG4CXX_ERROR(logger_, "Session with id: " << session_id << " not found");
- return;
}
LOG4CXX_DEBUG(logger_, "Session with id: " << session_id << " has been closed successfully");
}