commit b0800519b0dce9852ad77176a163cbd16e50a2c1
author Robert Sesek <rsesek@chromium.org> Thu Aug 30 21:13:44 2018
committer Commit Bot <commit-bot@chromium.org> Tue Sep 04 13:43:24 2018
tree 2b4c478e0e906bc5874cab198257975f98c493b4
parent 46d65e2c2cc0ab79828627a4024c396fe6b45b32

Do not leak the Mach thread port in GetThreadCpuTimeNanos().

Bug: chromium:879307
Change-Id: Ia6b5b3ea4684354d8a21dc85e43f67166832cc19
Reviewed-on: https://webrtc-review.googlesource.com/96980
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#24554}

rtc_base/cpu_time.cc

diff --git a/rtc_base/cpu_time.cc b/rtc_base/cpu_time.cc
index f25b506..de4a6bd 100644
--- a/rtc_base/cpu_time.cc
+++ b/rtc_base/cpu_time.cc
@@ -16,6 +16,7 @@
 #include <time.h>
 #elif defined(WEBRTC_MAC)
 #include <mach/mach_init.h>
+#include <mach/mach_port.h>
 #include <mach/thread_act.h>
 #include <mach/thread_info.h>
 #include <sys/resource.h>
@@ -81,10 +82,13 @@
     RTC_LOG_ERR(LS_ERROR) << "clock_gettime() failed.";
   }
 #elif defined(WEBRTC_MAC)
+  mach_port_t thread_port = mach_thread_self();
   thread_basic_info_data_t info;
   mach_msg_type_number_t count = THREAD_BASIC_INFO_COUNT;
-  if (thread_info(mach_thread_self(), THREAD_BASIC_INFO, (thread_info_t)&info,
-                  &count) == KERN_SUCCESS) {
+  kern_return_t kr =
+      thread_info(thread_port, THREAD_BASIC_INFO, (thread_info_t)&info, &count);
+  mach_port_deallocate(mach_task_self(), thread_port);
+  if (kr == KERN_SUCCESS) {
     return info.user_time.seconds * kNumNanosecsPerSec +
            info.user_time.microseconds * kNumNanosecsPerMicrosec;
   } else {