diff --git a/include/fastdds/rtps/transport/shared_mem/SharedMemTransportDescriptor.h b/include/fastdds/rtps/transport/shared_mem/SharedMemTransportDescriptor.h index a6e939443ba..a7ff8b300fc 100644 --- a/include/fastdds/rtps/transport/shared_mem/SharedMemTransportDescriptor.h +++ b/include/fastdds/rtps/transport/shared_mem/SharedMemTransportDescriptor.h @@ -30,17 +30,17 @@ class TransportInterface; */ typedef struct SharedMemTransportDescriptor : public TransportDescriptorInterface { - virtual ~SharedMemTransportDescriptor() + virtual ~SharedMemTransportDescriptor() { - + } virtual TransportInterface* create_transport() const override; uint32_t min_send_buffer_size() const override { - return 0; + return segment_size_; } - + RTPS_DllAPI SharedMemTransportDescriptor(); RTPS_DllAPI SharedMemTransportDescriptor( @@ -50,7 +50,7 @@ typedef struct SharedMemTransportDescriptor : public TransportDescriptorInterfac { return segment_size_; } - + RTPS_DllAPI void segment_size( uint32_t segment_size) { @@ -58,8 +58,8 @@ typedef struct SharedMemTransportDescriptor : public TransportDescriptorInterfac } virtual uint32_t max_message_size() const override - { - return maxMessageSize; + { + return maxMessageSize; } RTPS_DllAPI void max_message_size( diff --git a/src/cpp/rtps/network/ReceiverResource.cpp b/src/cpp/rtps/network/ReceiverResource.cpp index 108d84133e7..1762c30f340 100644 --- a/src/cpp/rtps/network/ReceiverResource.cpp +++ b/src/cpp/rtps/network/ReceiverResource.cpp @@ -17,25 +17,25 @@ #include #include -#define IDSTRING "(ID:" << std::this_thread::get_id() <<") "<< +#define IDSTRING "(ID:" << std::this_thread::get_id() << ") " << using namespace std; using namespace eprosima::fastdds::rtps; -namespace eprosima{ -namespace fastrtps{ -namespace rtps{ +namespace eprosima { +namespace fastrtps { +namespace rtps { ReceiverResource::ReceiverResource( - TransportInterface& transport, - const Locator_t& locator, - uint32_t max_recv_buffer_size) - : Cleanup(nullptr) - , LocatorMapsToManagedChannel(nullptr) - , mValid(false) - , mtx() - , receiver(nullptr) - , max_message_size_(max_recv_buffer_size) + TransportInterface& transport, + const Locator_t& locator, + uint32_t max_recv_buffer_size) + : Cleanup(nullptr) + , LocatorMapsToManagedChannel(nullptr) + , mValid(false) + , mtx() + , receiver(nullptr) + , max_message_size_(max_recv_buffer_size) { // Internal channel is opened and assigned to this resource. mValid = transport.OpenInputChannel(locator, this, max_message_size_); @@ -45,12 +45,18 @@ ReceiverResource::ReceiverResource( } // Implementation functions are bound to the right transport parameters - Cleanup = [&transport, locator]() { transport.CloseInputChannel(locator); }; + Cleanup = [&transport, locator]() + { + transport.CloseInputChannel(locator); + }; LocatorMapsToManagedChannel = [&transport, locator](const Locator_t& locatorToCheck) -> bool - { return transport.DoInputLocatorsMatch(locator, locatorToCheck); }; + { + return locator.kind == locatorToCheck.kind && transport.DoInputLocatorsMatch(locator, locatorToCheck); + }; } -ReceiverResource::ReceiverResource(ReceiverResource&& rValueResource) +ReceiverResource::ReceiverResource( + ReceiverResource&& rValueResource) { Cleanup.swap(rValueResource.Cleanup); LocatorMapsToManagedChannel.swap(rValueResource.LocatorMapsToManagedChannel); @@ -61,7 +67,8 @@ ReceiverResource::ReceiverResource(ReceiverResource&& rValueResource) max_message_size_ = rValueResource.max_message_size_; } -bool ReceiverResource::SupportsLocator(const Locator_t& localLocator) +bool ReceiverResource::SupportsLocator( + const Locator_t& localLocator) { if (LocatorMapsToManagedChannel) { @@ -70,22 +77,31 @@ bool ReceiverResource::SupportsLocator(const Locator_t& localLocator) return false; } -void ReceiverResource::RegisterReceiver(MessageReceiver* rcv) +void ReceiverResource::RegisterReceiver( + MessageReceiver* rcv) { std::unique_lock lock(mtx); if (receiver == nullptr) + { receiver = rcv; + } } -void ReceiverResource::UnregisterReceiver(MessageReceiver* rcv) +void ReceiverResource::UnregisterReceiver( + MessageReceiver* rcv) { std::unique_lock lock(mtx); if (receiver == rcv) + { receiver = nullptr; + } } -void ReceiverResource::OnDataReceived(const octet * data, const uint32_t size, - const Locator_t & localLocator, const Locator_t & remoteLocator) +void ReceiverResource::OnDataReceived( + const octet* data, + const uint32_t size, + const Locator_t& localLocator, + const Locator_t& remoteLocator) { (void)localLocator; diff --git a/src/cpp/rtps/participant/RTPSParticipantImpl.cpp b/src/cpp/rtps/participant/RTPSParticipantImpl.cpp index a010b2b9c28..4ab9d0533fc 100644 --- a/src/cpp/rtps/participant/RTPSParticipantImpl.cpp +++ b/src/cpp/rtps/participant/RTPSParticipantImpl.cpp @@ -1122,7 +1122,6 @@ bool RTPSParticipantImpl::assignEndpoint2LocatorList( one of the supported Locators is needed to make the match, and the case of new ListenResources being created has been removed since its the NetworkFactory the one that takes care of Resource creation. */ - LocatorList_t finalList; for (auto lit = list.begin(); lit != list.end(); ++lit) { //Iteration of all Locators within the Locator list passed down as argument diff --git a/src/cpp/rtps/reader/StatefulReader.cpp b/src/cpp/rtps/reader/StatefulReader.cpp index e7a053c9796..dc8cc3802b2 100644 --- a/src/cpp/rtps/reader/StatefulReader.cpp +++ b/src/cpp/rtps/reader/StatefulReader.cpp @@ -219,6 +219,12 @@ bool StatefulReader::matched_writer_add( matched_writers_pool_.pop_back(); } + SequenceNumber_t initial_sequence; + add_persistence_guid(wdata.guid(), wdata.persistence_guid()); + initial_sequence = get_last_notified(wdata.guid()); + + wp->start(wdata, initial_sequence); + if (!is_same_process) { for (const Locator_t& locator : wp->remote_locators_shrinked()) @@ -227,12 +233,6 @@ bool StatefulReader::matched_writer_add( } } - SequenceNumber_t initial_sequence; - add_persistence_guid(wdata.guid(), wdata.persistence_guid()); - initial_sequence = get_last_notified(wdata.guid()); - - wp->start(wdata, initial_sequence); - if (is_datasharing) { if (datasharing_listener_->add_datasharing_writer(wdata.guid(), diff --git a/test/blackbox/common/BlackboxTestsPubSubBasic.cpp b/test/blackbox/common/BlackboxTestsPubSubBasic.cpp index f19bc1e3fcb..9b30bbd9a40 100644 --- a/test/blackbox/common/BlackboxTestsPubSubBasic.cpp +++ b/test/blackbox/common/BlackboxTestsPubSubBasic.cpp @@ -275,6 +275,7 @@ TEST_P(PubSubBasic, PubSubMoreThan256Unacknowledged) reader.startReception(expected_data); reader.block_for_all(); + EXPECT_TRUE(writer.waitForAllAcked(std::chrono::seconds(10))); } TEST_P(PubSubBasic, PubSubAsReliableHelloworldMulticastDisabled) @@ -308,6 +309,7 @@ TEST_P(PubSubBasic, PubSubAsReliableHelloworldMulticastDisabled) ASSERT_TRUE(data.empty()); // Block reader until reception finished or timeout. reader.block_for_all(); + EXPECT_TRUE(writer.waitForAllAcked(std::chrono::seconds(10))); } TEST_P(PubSubBasic, ReceivedDynamicDataWithNoSizeLimit)