Protect WriterHistory::remove_change_and_reuse (#5115) (#5121)

mergify[bot] · juanlofer-eprosima · web-flow · commit 44fddaba1929 · 2024-07-31T10:56:16.000+02:00
Signed-off-by: Juan Lopez Fernandez <juanlopez@eprosima.com> (cherry picked from commit 0d996bc) Co-authored-by: juanlofer-eprosima <88179026+juanlofer-eprosima@users.noreply.github.com>
diff --git a/src/cpp/rtps/history/WriterHistory.cpp b/src/cpp/rtps/history/WriterHistory.cpp
@@ -232,6 +232,8 @@ CacheChange_t* WriterHistory::remove_change_and_reuse(
         return nullptr;
     }
 
+    std::lock_guard<RecursiveTimedMutex> guard(*mp_mutex);
+
     // Create a temporary reference change associated to the sequence number
     CacheChange_t ch;
     ch.sequenceNumber = sequence_number;

Original file line number	Diff line number	Diff line change
`@@ -232,6 +232,8 @@ CacheChange_t* WriterHistory::remove_change_and_reuse(`
`232`	`232`	`return nullptr;`
`233`	`233`	`}`
`234`	`234`
	`235`	`+ std::lock_guard<RecursiveTimedMutex> guard(*mp_mutex);`
	`236`	`+`
`235`	`237`	`// Create a temporary reference change associated to the sequence number`
`236`	`238`	`CacheChange_t ch;`
`237`	`239`	`ch.sequenceNumber = sequence_number;`