Protect WriterHistory::remove_change_and_reuse (#5115)

juanlofer-eprosima · web-flow · commit 0d996bc3eaf7 · 2024-07-30T18:15:19.000+02:00
Signed-off-by: Juan Lopez Fernandez &lt;juanlopez@eprosima.com&gt;
diff --git a/src/cpp/rtps/history/WriterHistory.cpp b/src/cpp/rtps/history/WriterHistory.cpp
@@ -332,6 +332,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
`@@ -332,6 +332,8 @@ CacheChange_t* WriterHistory::remove_change_and_reuse(`
`332`	`332`	`return nullptr;`
`333`	`333`	`}`
`334`	`334`
	`335`	`+ std::lock_guard<RecursiveTimedMutex> guard(*mp_mutex);`
	`336`	`+`
`335`	`337`	`// Create a temporary reference change associated to the sequence number`
`336`	`338`	`CacheChange_t ch;`
`337`	`339`	`ch.sequenceNumber = sequence_number;`