diff options
author | Eino-Ville Talvala <etalvala@google.com> | 2012-10-06 13:50:31 -0700 |
---|---|---|
committer | Eino-Ville Talvala <etalvala@google.com> | 2012-10-06 13:50:31 -0700 |
commit | 4d410ede659448557398c84552eb0bf50ef09cbc (patch) | |
tree | ac0bc2a73208f4703ce37f2232d49618e9ed2ba9 /services | |
parent | f6e09d200c3e043737adcdadd52310ebbd1b19da (diff) | |
download | frameworks_av-4d410ede659448557398c84552eb0bf50ef09cbc.zip frameworks_av-4d410ede659448557398c84552eb0bf50ef09cbc.tar.gz frameworks_av-4d410ede659448557398c84552eb0bf50ef09cbc.tar.bz2 |
Camera2: Fix off-by-one error in ZSL selection.
Bug: 7258126
Change-Id: Ie6da4645c46a7bb0dbd11d7c6183e904f41f74fd
Diffstat (limited to 'services')
-rw-r--r-- | services/camera/libcameraservice/camera2/ZslProcessor.cpp | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/services/camera/libcameraservice/camera2/ZslProcessor.cpp b/services/camera/libcameraservice/camera2/ZslProcessor.cpp index 5208574..1937955 100644 --- a/services/camera/libcameraservice/camera2/ZslProcessor.cpp +++ b/services/camera/libcameraservice/camera2/ZslProcessor.cpp @@ -254,11 +254,14 @@ status_t ZslProcessor::pushToReprocess(int32_t requestId) { if (mZslQueueTail != mZslQueueHead) { CameraMetadata request; size_t index = mZslQueueTail; - while (request.isEmpty() && index != mZslQueueHead) { - request = mZslQueue[index].frame; + while (index != mZslQueueHead) { + if (!mZslQueue[index].frame.isEmpty()) { + request = mZslQueue[index].frame; + break; + } index = (index + 1) % kZslBufferDepth; } - if (request.isEmpty()) { + if (index == mZslQueueHead) { ALOGV("%s: ZSL queue has no valid frames to send yet.", __FUNCTION__); return NOT_ENOUGH_DATA; |