From 845584e7172406224071ebeb04838dc757761bcb Mon Sep 17 00:00:00 2001
From: James Dong <jdong@google.com>
Date: Fri, 10 Dec 2010 13:09:05 -0800
Subject: [PATCH] Fixed a failure from timelapse video recording because of
 accidental use of metadata buffers

bug - 3274317

Change-Id: I2c0e9fcd30c9141ce5a85e9d72cbd0baf561458b
---
 media/libstagefright/CameraSource.cpp | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/media/libstagefright/CameraSource.cpp b/media/libstagefright/CameraSource.cpp
index 8c686b9bc7844..fac7b7844d3ad 100644
--- a/media/libstagefright/CameraSource.cpp
+++ b/media/libstagefright/CameraSource.cpp
@@ -485,10 +485,13 @@ status_t CameraSource::init(
     // check earlier by calling mCamera->setParameters().
     CHECK_EQ(OK, mCamera->setPreviewDisplay(mSurface));
 
+    // By default, do not store metadata in video buffers
     mIsMetaDataStoredInVideoBuffers = false;
-    if (storeMetaDataInVideoBuffers &&
-        OK == mCamera->storeMetaDataInBuffers(true)) {
-        mIsMetaDataStoredInVideoBuffers = true;
+    mCamera->storeMetaDataInBuffers(false);
+    if (storeMetaDataInVideoBuffers) {
+        if (OK == mCamera->storeMetaDataInBuffers(true)) {
+            mIsMetaDataStoredInVideoBuffers = true;
+        }
     }
 
     IPCThreadState::self()->restoreCallingIdentity(token);