fix(autoware_tensorrt_classifier): fix bugprone-exception-escape (#9732)

kobayu858 · web-flow · commit a499a0a0ed79 · 2024-12-24T17:23:30.000+09:00
fix: bugprone-error

Signed-off-by: kobayu858 &lt;yutaro.kobayashi@tier4.jp&gt;
diff --git a/perception/autoware_tensorrt_classifier/src/tensorrt_classifier.cpp b/perception/autoware_tensorrt_classifier/src/tensorrt_classifier.cpp
@@ -193,9 +193,15 @@ TrtClassifier::TrtClassifier(
 
 TrtClassifier::~TrtClassifier()
 {
-  if (m_cuda) {
-    if (h_img_) CHECK_CUDA_ERROR(cudaFreeHost(h_img_));
-    if (d_img_) CHECK_CUDA_ERROR(cudaFree(d_img_));
+  try {
+    if (m_cuda) {
+      if (h_img_) CHECK_CUDA_ERROR(cudaFreeHost(h_img_));
+      if (d_img_) CHECK_CUDA_ERROR(cudaFree(d_img_));
+    }
+  } catch (const std::exception & e) {
+    std::cerr << "Exception in TrtClassifier destructor: " << e.what() << std::endl;
+  } catch (...) {
+    std::cerr << "Unknown exception in TrtClassifier destructor" << std::endl;
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -193,9 +193,15 @@ TrtClassifier::TrtClassifier(`
`193`	`193`
`194`	`194`	`TrtClassifier::~TrtClassifier()`
`195`	`195`	`{`
`196`		`- if (m_cuda) {`
`197`		`- if (h_img_) CHECK_CUDA_ERROR(cudaFreeHost(h_img_));`
`198`		`- if (d_img_) CHECK_CUDA_ERROR(cudaFree(d_img_));`
	`196`	`+ try {`
	`197`	`+ if (m_cuda) {`
	`198`	`+ if (h_img_) CHECK_CUDA_ERROR(cudaFreeHost(h_img_));`
	`199`	`+ if (d_img_) CHECK_CUDA_ERROR(cudaFree(d_img_));`
	`200`	`+ }`
	`201`	`+ } catch (const std::exception & e) {`
	`202`	`+ std::cerr << "Exception in TrtClassifier destructor: " << e.what() << std::endl;`
	`203`	`+ } catch (...) {`
	`204`	`+ std::cerr << "Unknown exception in TrtClassifier destructor" << std::endl;`
`199`	`205`	`}`
`200`	`206`	`}`
`201`	`207`