replace throw-error with error-log to avoid UAF mentioned in #4175 (#…

…4180)

* replace throw-error with error-log to avoid UAF

Signed-off-by: GoesM <GoesM@buaa.edu.cn>

* fix typo

Signed-off-by: GoesM <GoesM@buaa.edu.cn>

---------

Signed-off-by: GoesM <GoesM@buaa.edu.cn>
Co-authored-by: GoesM <GoesM@buaa.edu.cn>

nav2_util/include/nav2_util/simple_action_server.hpp

@@ -315,7 +315,7 @@ class SimpleActionServer
       if (steady_clock::now() - start_time >= server_timeout_) {
         terminate_all();
         if (completion_callback_) {completion_callback_();}
-        throw std::runtime_error("Action callback is still running and missed deadline to stop");
+        error_msg("Action callback is still running and missed deadline to stop");
       }
     }