Call deregister thread tracer from ~Thread

Added a missed call.
cactusdynamics · Jul 18, 2024 · f859259 · f859259
1 parent fd78d5d
commit f859259
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/include/cactus_rt/thread.h b/include/cactus_rt/thread.h
@@ -98,7 +98,7 @@ class Thread {
 
   // The constructors and destructors are needed because we need to delete
   // objects of type Thread polymorphically, through the map in the App class.
-  virtual ~Thread() = default;
+  virtual ~Thread();
 
   // Copy constructors are not allowed
   Thread(const Thread&) = delete;

diff --git a/src/cactus_rt/thread.cc b/src/cactus_rt/thread.cc
@@ -78,4 +78,11 @@ void Thread::Start(int64_t start_monotonic_time_ns, App* app) {
 int Thread::Join() const {
   return pthread_join(thread_, nullptr);
 }
+
+Thread::~Thread() {
+  if (app_ != nullptr) {
+    app_->DeregisterThreadTracer(tracer_);
+  }
+}
+
 }  // namespace cactus_rt