summaryrefslogtreecommitdiffstats
path: root/services/camera
diff options
context:
space:
mode:
authorEino-Ville Talvala <etalvala@google.com>2012-10-10 17:22:53 -0700
committerAndroid Git Automerger <android-git-automerger@android.com>2012-10-10 17:22:53 -0700
commit142dbcfe437524bc0e8d546d2d43cab2a3ea625b (patch)
treea550ea67fe8070fa5141ecea2691ae4a47f4e2a1 /services/camera
parente6ff6cd2fe5c897803d8dfc2d575c24db38acd88 (diff)
parent33ef5038d780b4cc441eec867224824655e19095 (diff)
downloadframeworks_av-142dbcfe437524bc0e8d546d2d43cab2a3ea625b.zip
frameworks_av-142dbcfe437524bc0e8d546d2d43cab2a3ea625b.tar.gz
frameworks_av-142dbcfe437524bc0e8d546d2d43cab2a3ea625b.tar.bz2
am f5230955: am 907e5651: Camera2: Don\'t let unlocks happen during recording.
* commit 'f52309555013e9909b248374f96d22dae29f6454': Camera2: Don't let unlocks happen during recording.
Diffstat (limited to 'services/camera')
-rw-r--r--services/camera/libcameraservice/Camera2Client.cpp8
1 files changed, 6 insertions, 2 deletions
diff --git a/services/camera/libcameraservice/Camera2Client.cpp b/services/camera/libcameraservice/Camera2Client.cpp
index cd9da42..0ab4e8c 100644
--- a/services/camera/libcameraservice/Camera2Client.cpp
+++ b/services/camera/libcameraservice/Camera2Client.cpp
@@ -454,9 +454,13 @@ status_t Camera2Client::unlock() {
ALOGV("%s: Camera %d: Unlock call from pid %d; current client pid %d",
__FUNCTION__, mCameraId, getCallingPid(), mClientPid);
- // TODO: Check for uninterruptable conditions
-
if (mClientPid == getCallingPid()) {
+ SharedParameters::Lock l(mParameters);
+ if (l.mParameters.state == Parameters::RECORD ||
+ l.mParameters.state == Parameters::VIDEO_SNAPSHOT) {
+ ALOGD("Not allowed to unlock camera during recording.");
+ return INVALID_OPERATION;
+ }
mClientPid = 0;
mCameraClient.clear();
mSharedCameraClient.clear();