commit 8da4cd7d706d9f551bf2d08cd8b61907a10cd38d
author James Dong <jdong@google.com> Sat Aug 04 19:58:07 2012 -0700
committer James Dong <jdong@google.com> Sat Aug 04 20:46:14 2012 -0700
tree d859052f6839fedf654ce8cea7dea2d567ad710f
parent d9f0253353db0b859c0ca2b32eddc24e3e9d4d86

Fix a deadlock in commandSetVideoBufferCountL()

o The lock to be acquired in recordingEnabled() has aleady been acquired in sendCommand()
  before the call to commandSetVideoBufferCountL().

Change-Id: I664d51ef449c9eb3576d5d56f73f29c98444ff3f

services/camera/libcameraservice/Camera2Client.cpp

diff --git a/services/camera/libcameraservice/Camera2Client.cpp b/services/camera/libcameraservice/Camera2Client.cpp
index dfcc8c0..cd74e6d 100644
--- a/services/camera/libcameraservice/Camera2Client.cpp
+++ b/services/camera/libcameraservice/Camera2Client.cpp
@@ -792,8 +792,15 @@
 bool Camera2Client::recordingEnabled() {
     ATRACE_CALL();
     Mutex::Autolock icl(mICameraLock);
+
     if ( checkPid(__FUNCTION__) != OK) return false;
 
+    return recordingEnabledL();
+}
+
+bool Camera2Client::recordingEnabledL() {
+    ATRACE_CALL();
+
     return (mState == RECORD || mState == VIDEO_SNAPSHOT);
 }
 
@@ -1632,7 +1639,7 @@
 }
 
 status_t Camera2Client::commandSetVideoBufferCountL(size_t count) {
-    if (recordingEnabled()) {
+    if (recordingEnabledL()) {
         ALOGE("%s: Camera %d: Error setting video buffer count after "
                 "recording was started", __FUNCTION__, mCameraId);
         return INVALID_OPERATION;