fix(autoware_universe_utils): fix memory leak of time_keeper (#8425)

fix bug of time_keeper

Signed-off-by: Y.Hisaki <[email protected]>

common/autoware_universe_utils/include/autoware/universe_utils/system/time_keeper.hpp

@@ -67,9 +67,9 @@ class ProcessingTimeNode : public std::enable_shared_from_this<ProcessingTimeNod
   /**
    * @brief Get the parent node
    *
-   * @return std::shared_ptr<ProcessingTimeNode> Shared pointer to the parent node
+   * @return std::weak_ptr<ProcessingTimeNode> Weak pointer to the parent node
    */
-  std::shared_ptr<ProcessingTimeNode> get_parent_node() const;
+  std::weak_ptr<ProcessingTimeNode> get_parent_node() const;
 
   /**
    * @brief Get the child nodes
@@ -101,10 +101,10 @@ class ProcessingTimeNode : public std::enable_shared_from_this<ProcessingTimeNod
   std::string get_name() const;
 
 private:
-  const std::string name_;                                    //!< Name of the node
-  double processing_time_{0.0};                               //!< Processing time of the node
-  std::string comment_;                                       //!< Comment for the node
-  std::shared_ptr<ProcessingTimeNode> parent_node_{nullptr};  //!< Shared pointer to the parent node
+  const std::string name_;                         //!< Name of the node
+  double processing_time_{0.0};                    //!< Processing time of the node
+  std::string comment_;                            //!< Comment for the node
+  std::weak_ptr<ProcessingTimeNode> parent_node_;  //!< Weak pointer to the parent node
   std::vector<std::shared_ptr<ProcessingTimeNode>>
     child_nodes_;  //!< Vector of shared pointers to the child nodes
 };

common/autoware_universe_utils/src/system/time_keeper.cpp

@@ -28,7 +28,7 @@ ProcessingTimeNode::ProcessingTimeNode(const std::string & name) : name_(name)
 std::shared_ptr<ProcessingTimeNode> ProcessingTimeNode::add_child(const std::string & name)
 {
   auto new_child_node = std::make_shared<ProcessingTimeNode>(name);
-  new_child_node->parent_node_ = shared_from_this();
+  new_child_node->parent_node_ = weak_from_this();
   child_nodes_.push_back(new_child_node);
   return new_child_node;
 }
@@ -86,7 +86,7 @@ tier4_debug_msgs::msg::ProcessingTimeTree ProcessingTimeNode::to_msg() const
   return time_tree_msg;
 }
 
-std::shared_ptr<ProcessingTimeNode> ProcessingTimeNode::get_parent_node() const
+std::weak_ptr<ProcessingTimeNode> ProcessingTimeNode::get_parent_node() const
 {
   return parent_node_;
 }
@@ -152,7 +152,7 @@ void TimeKeeper::end_track(const std::string & func_name)
   }
   const double processing_time = stop_watch_.toc(func_name);
   current_time_node_->set_time(processing_time);
-  current_time_node_ = current_time_node_->get_parent_node();
+  current_time_node_ = current_time_node_->get_parent_node().lock();
 
   if (current_time_node_ == nullptr) {
     report();