fix: Remove logging that could occur while a thread is suspended (#2014)

Remove some logging that could happen when threads fail to be suspended or resumed. Co-authored-by: Philipp Hofmann <philipp.hofmann@sentry.io>
getsentry · Jul 28, 2022 · 3b70b5b · 3b70b5b
1 parent 3fc26db
commit 3b70b5b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@
 
 - Add sampling configuration for profiling (#2004)
 
+### Fixes
+
+- Remove logging that could occur while a thread is suspended (#2014)
+
 ## 7.22.0
 
 ### Features

diff --git a/Sources/Sentry/SentryThreadHandle.cpp b/Sources/Sentry/SentryThreadHandle.cpp
@@ -240,7 +240,7 @@ namespace profiling {
         if (handle_ == THREAD_NULL) {
             return false;
         }
-        return SENTRY_PROF_LOG_KERN_RETURN(thread_suspend(handle_)) == KERN_SUCCESS;
+        return thread_suspend(handle_) == KERN_SUCCESS;
     }
 
     bool
@@ -249,7 +249,7 @@ namespace profiling {
         if (handle_ == THREAD_NULL) {
             return false;
         }
-        return SENTRY_PROF_LOG_KERN_RETURN(thread_resume(handle_)) == KERN_SUCCESS;
+        return thread_resume(handle_) == KERN_SUCCESS;
     }
 
     bool