Refs #21082. Fix release order in WriterHistory destructor.

MiguelCompany · MiguelCompany · commit 60154a1f0b2f · 2024-06-19T10:21:52.000+02:00
Signed-off-by: Miguel Company &lt;miguelcompany@eprosima.com&gt;
diff --git a/src/cpp/rtps/history/WriterHistory.cpp b/src/cpp/rtps/history/WriterHistory.cpp
@@ -114,7 +114,11 @@ WriterHistory::WriterHistory(
 
 WriterHistory::~WriterHistory()
 {
-    // TODO Auto-generated destructor stub
+    // As releasing the change pool will delete the cache changes it owns,
+    // the payload pool may be called to release their payloads, so we should
+    // ensure that the payload pool is destroyed after the change pool.
+    change_pool_.reset();
+    payload_pool_.reset();
 }
 
 const std::shared_ptr<IPayloadPool>& WriterHistory::get_payload_pool() const

Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,11 @@ WriterHistory::WriterHistory(`
`114`	`114`
`115`	`115`	`WriterHistory::~WriterHistory()`
`116`	`116`	`{`
`117`		`- // TODO Auto-generated destructor stub`
	`117`	`+ // As releasing the change pool will delete the cache changes it owns,`
	`118`	`+ // the payload pool may be called to release their payloads, so we should`
	`119`	`+ // ensure that the payload pool is destroyed after the change pool.`
	`120`	`+ change_pool_.reset();`
	`121`	`+ payload_pool_.reset();`
`118`	`122`	`}`
`119`	`123`
`120`	`124`	`const std::shared_ptr<IPayloadPool>& WriterHistory::get_payload_pool() const`