[20180] TCP Client&Server Participant Decision-Making

src/cpp/fastdds/publisher/DataWriterImpl.cpp

@@ -1210,6 +1210,7 @@ const DataWriterQos& DataWriterImpl::get_qos() const
 ReturnCode_t DataWriterImpl::set_listener(
         DataWriterListener* listener)
 {
+    std::lock_guard<std::mutex> scoped_lock(listener_mutex_);
     listener_ = listener;
     return ReturnCode_t::RETCODE_OK;
 }
@@ -1974,6 +1975,7 @@ bool DataWriterImpl::can_qos_be_updated(
 DataWriterListener* DataWriterImpl::get_listener_for(
         const StatusMask& status)
 {
+    std::lock_guard<std::mutex> scoped_lock(listener_mutex_);
     if (listener_ != nullptr &&
             user_datawriter_->get_status_mask().is_active(status))
     {

src/cpp/fastdds/publisher/DataWriterImpl.hpp

@@ -410,6 +410,9 @@ class DataWriterImpl : protected rtps::IReaderDataFilter
     //! DataWriterListener
     DataWriterListener* listener_ = nullptr;
 
+    //! Mutex to protect listener_
+    std::mutex listener_mutex_;
+
     //!History
     DataWriterHistory history_;
 

src/cpp/rtps/attributes/RTPSParticipantAttributes.cpp

@@ -208,7 +208,6 @@ static void setup_transports_large_data(
 
         auto tcp_transport = create_tcpv4_transport(att);
         att.userTransports.push_back(tcp_transport);
-        att.properties.properties().emplace_back("fastdds.tcp_transport.non_blocking_send", "true");
 
         Locator_t tcp_loc;
         tcp_loc.kind = LOCATOR_KIND_TCPv4;
@@ -245,7 +244,6 @@ static void setup_transports_large_datav6(
 
         auto tcp_transport = create_tcpv6_transport(att);
         att.userTransports.push_back(tcp_transport);
-        att.properties.properties().emplace_back("fastdds.tcp_transport.non_blocking_send", "true");
 
         Locator_t tcp_loc;
         tcp_loc.kind = LOCATOR_KIND_TCPv6;
@@ -263,6 +261,7 @@ static void setup_transports_large_datav6(
     {
         Locator_t pdp_locator;
         pdp_locator.kind = LOCATOR_KIND_UDPv6;
+        IPLocator::setIPv6(pdp_locator, "ff1e::ffff:efff:1");
         att.builtin.metatrafficMulticastLocatorList.push_back(pdp_locator);
     }
 }

src/cpp/rtps/transport/TCPChannelResource.cpp

@@ -84,7 +84,7 @@ ResponseCode TCPChannelResource::process_bind_request(
     if (connection_status_.compare_exchange_strong(expected, eConnectionStatus::eEstablished))
     {
         locator_ = IPLocator::toPhysicalLocator(locator);
-        EPROSIMA_LOG_INFO(RTCP_MSG, "Connection Stablished");
+        EPROSIMA_LOG_INFO(RTCP_MSG, "Connection Established");
         return RETCODE_OK;
     }
     else if (expected == eConnectionStatus::eEstablished)
@@ -139,9 +139,7 @@ void TCPChannelResource::add_logical_port(
             pending_logical_output_ports_.emplace_back(port);
             if (connection_established())
             {
-                scopedLock.unlock();
                 TCPTransactionId id = rtcp_manager->sendOpenLogicalPortRequest(this, port);
-                scopedLock.lock();
                 negotiating_logical_ports_[id] = port;
             }
         }

src/cpp/rtps/transport/TCPChannelResourceBasic.cpp

@@ -105,6 +105,7 @@ void TCPChannelResourceBasic::disconnect()
 {
     if (eConnecting < change_status(eConnectionStatus::eDisconnected) && alive())
     {
+        std::lock_guard<std::mutex> read_lock(read_mutex_);
         auto socket = socket_;
 
         std::error_code ec;

src/cpp/rtps/transport/TCPSenderResource.hpp

@@ -32,14 +32,14 @@ class TCPSenderResource : public fastrtps::rtps::SenderResource
 
     TCPSenderResource(
             TCPTransportInterface& transport,
-            std::shared_ptr<TCPChannelResource>& channel)
+            eprosima::fastrtps::rtps::Locator_t& locator)
         : fastrtps::rtps::SenderResource(transport.kind())
-        , channel_(channel)
+        , locator_(locator)
     {
         // Implementation functions are bound to the right transport parameters
         clean_up = [this, &transport]()
                 {
-                    transport.CloseOutputChannel(channel_);
+                    transport.CloseOutputChannel(locator_);
                 };
 
         send_lambda_ = [this, &transport](
@@ -49,7 +49,7 @@ class TCPSenderResource : public fastrtps::rtps::SenderResource
             fastrtps::rtps::LocatorsIterator* destination_locators_end,
             const std::chrono::steady_clock::time_point&) -> bool
                 {
-                    return transport.send(data, dataSize, channel_, destination_locators_begin,
+                    return transport.send(data, dataSize, locator_, destination_locators_begin,
                                    destination_locators_end);
                 };
     }
@@ -62,9 +62,9 @@ class TCPSenderResource : public fastrtps::rtps::SenderResource
         }
     }
 
-    std::shared_ptr<TCPChannelResource>& channel()
+    fastrtps::rtps::Locator_t& locator()
     {
-        return channel_;
+        return locator_;
     }
 
     static TCPSenderResource* cast(
@@ -102,7 +102,7 @@ class TCPSenderResource : public fastrtps::rtps::SenderResource
     TCPSenderResource& operator =(
             const SenderResource&) = delete;
 
-    std::shared_ptr<TCPChannelResource> channel_;
+    fastrtps::rtps::Locator_t locator_;
 };
 
 } // namespace rtps