Snap for 7069213 from 2cffede25703367337461fa633b19ebfb69837f2 to rvc-d2-releaseandroid-11.0.0_r45android-11.0.0_r44android-11.0.0_r42android-11.0.0_r41android11-d2-release

Change-Id: I53f00160b5f3965c2c3ef25d65269c323273225f

1 files changed, 9 insertions, 0 deletions

diff --git a/common/hal/hidl_service/hidl_profiler.cc b/common/hal/hidl_service/hidl_profiler.cc
index 3c7e40d..e7e9ab9 100644
--- a/common/hal/hidl_service/hidl_profiler.cc
+++ b/common/hal/hidl_service/hidl_profiler.cc
@@ -17,6 +17,7 @@
 //#define LOG_NDEBUG 0
 #define LOG_TAG "GCH_HidlProfiler"
 #include <log/log.h>
+#include <mutex>
 #include <utility>
 
 #include "hidl_profiler.h"
@@ -57,6 +58,8 @@ struct HidlProfiler {
 };
 
 std::unique_ptr<HidlProfiler> gHidlProfiler = nullptr;
+// Mutex to make all API functions mutually exclusive.
+std::mutex api_mutex;
 
 void StartNewConnector() {
   if (gHidlProfiler != nullptr && gHidlProfiler->profiler != nullptr) {
@@ -80,6 +83,7 @@ void EndProfiler() {
 }  // anonymous namespace
 
 std::unique_ptr<HidlProfilerItem> OnCameraOpen() {
+  std::lock_guard lock(api_mutex);
   gHidlProfiler = std::make_unique<HidlProfiler>();
   if (gHidlProfiler == nullptr || gHidlProfiler->profiler == nullptr) {
     ALOGE("%s: gHidlProfiler or profiler is nullptr.", __FUNCTION__);
@@ -94,6 +98,7 @@ std::unique_ptr<HidlProfilerItem> OnCameraOpen() {
 }
 
 std::unique_ptr<HidlProfilerItem> OnCameraFlush() {
+  std::lock_guard lock(api_mutex);
   EndConnector();
   if (gHidlProfiler == nullptr) {
     gHidlProfiler = std::make_unique<HidlProfiler>();
@@ -110,6 +115,7 @@ std::unique_ptr<HidlProfilerItem> OnCameraFlush() {
 }
 
 std::unique_ptr<HidlProfilerItem> OnCameraClose() {
+  std::lock_guard lock(api_mutex);
   EndConnector();
   if (gHidlProfiler == nullptr) {
     gHidlProfiler = std::make_unique<HidlProfiler>();
@@ -126,6 +132,7 @@ std::unique_ptr<HidlProfilerItem> OnCameraClose() {
 }
 
 std::unique_ptr<HidlProfilerItem> OnCameraStreamConfigure() {
+  std::lock_guard lock(api_mutex);
   EndConnector();
   if (gHidlProfiler == nullptr) {
     gHidlProfiler = std::make_unique<HidlProfiler>();
@@ -146,6 +153,7 @@ std::unique_ptr<HidlProfilerItem> OnCameraStreamConfigure() {
 }
 
 void OnFirstFrameRequest() {
+  std::lock_guard lock(api_mutex);
   EndConnector();
   if (gHidlProfiler != nullptr && gHidlProfiler->profiler != nullptr) {
     gHidlProfiler->profiler->Start(
@@ -156,6 +164,7 @@ void OnFirstFrameRequest() {
 }
 
 void OnFirstFrameResult() {
+  std::lock_guard lock(api_mutex);
   if (gHidlProfiler != nullptr && gHidlProfiler->profiler != nullptr) {
     gHidlProfiler->profiler->End(
         "First frame", google::camera_common::Profiler::kInvalidRequestId);