Add missing lock guards for discovered_names and discovered_namespaces (

#301)

Signed-off-by: ivanpauno <[email protected]>

rmw_fastrtps_shared_cpp/src/rmw_node_info_and_types.cpp

@@ -63,6 +63,8 @@ rmw_ret_t __get_guid_by_name(
     guid = impl->participant->getGuid();
   } else {
     std::set<GUID_t> nodes_in_desired_namespace;
+    std::lock_guard<std::mutex> guard(impl->listener->names_mutex_);
+
     auto namespaces = impl->listener->discovered_namespaces;
     for (auto & guid_to_namespace : impl->listener->discovered_namespaces) {
       if (guid_to_namespace.second == node_namespace) {
@@ -284,13 +286,15 @@ __log_debug_information(const CustomParticipantInfo & impl)
     }
     {
       std::stringstream ss;
+      std::lock_guard<std::mutex> guard(impl.listener->names_mutex_);
       for (auto & node_pair : impl.listener->discovered_names) {
         ss << node_pair.first << " : " << node_pair.second << " ";
       }
       RCUTILS_LOG_DEBUG_NAMED(kLoggerTag, "Discovered names: %s", ss.str().c_str());
     }
     {
       std::stringstream ss;
+      std::lock_guard<std::mutex> guard(impl.listener->names_mutex_);
       for (auto & node_pair : impl.listener->discovered_namespaces) {
         ss << node_pair.first << " : " << node_pair.second << " ";
       }