From 65d7986ceac6e35426749ac7e05bbd2a38949db4 Mon Sep 17 00:00:00 2001
From: Igor Murashkin <iam@google.com>
Date: Wed, 27 Mar 2013 11:07:06 -0700
Subject: ProCamera: Fix memory leak in consumeFrameMetadata

Bug: 8402301
Change-Id: I975990185f45bb8552712271b9ac457af08a170c
---
 camera/ProCamera.cpp | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

(limited to 'camera/ProCamera.cpp')

diff --git a/camera/ProCamera.cpp b/camera/ProCamera.cpp
index 396b009..fec5461 100644
--- a/camera/ProCamera.cpp
+++ b/camera/ProCamera.cpp
@@ -103,7 +103,7 @@ void ProCamera::onResultReceived(int32_t frameId, camera_metadata* result) {
     {
         Mutex::Autolock al(mWaitMutex);
         mMetadataReady = true;
-        mLatestMetadata = tmp;
+        mLatestMetadata = tmp; // make copy
         mWaitCondition.broadcast();
     }
 
@@ -312,8 +312,6 @@ void ProCamera::onFrameAvailable(int streamId) {
     sp<ProCameraListener> listener = mListener;
     StreamInfo& stream = getStreamInfo(streamId);
 
-    CpuConsumer::LockedBuffer buf;
-
     if (listener.get() != NULL) {
         listener->onFrameAvailable(streamId, stream.cpuConsumer);
     }
@@ -421,7 +419,7 @@ CameraMetadata ProCamera::consumeFrameMetadata() {
 
     // Destructive: Subsequent calls return empty metadatas
     CameraMetadata tmp = mLatestMetadata;
-    mLatestMetadata.release();
+    mLatestMetadata.clear();
 
     return tmp;
 }
-- 
cgit v1.1