diff options
author | Eino-Ville Talvala <etalvala@google.com> | 2015-02-03 18:40:29 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2015-02-03 18:40:30 +0000 |
commit | 16e8d463bc5c8805ae6815c67875b16d177a9d8a (patch) | |
tree | 1c42641374d9ec3af44b3a79ddcd71d5e7b803cf | |
parent | 434c80e54fb0dbf01941aa7560d5add5e69e0f1d (diff) | |
parent | 19fa0feaef631f92ab812cd1640252ade5fae1a2 (diff) | |
download | frameworks_av-16e8d463bc5c8805ae6815c67875b16d177a9d8a.zip frameworks_av-16e8d463bc5c8805ae6815c67875b16d177a9d8a.tar.gz frameworks_av-16e8d463bc5c8805ae6815c67875b16d177a9d8a.tar.bz2 |
Merge "camera2: fix various BpCameraDeviceUser methods to return correct result."
-rw-r--r-- | camera/camera2/ICameraDeviceUser.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/camera/camera2/ICameraDeviceUser.cpp b/camera/camera2/ICameraDeviceUser.cpp index 35345d2..277b5db 100644 --- a/camera/camera2/ICameraDeviceUser.cpp +++ b/camera/camera2/ICameraDeviceUser.cpp @@ -107,7 +107,7 @@ public: } } - if ((res != NO_ERROR) || (resFrameNumber != NO_ERROR)) { + if ((res < NO_ERROR) || (resFrameNumber != NO_ERROR)) { res = FAILED_TRANSACTION; } return res; @@ -147,7 +147,7 @@ public: resFrameNumber = reply.readInt64(lastFrameNumber); } } - if ((res != NO_ERROR) || (resFrameNumber != NO_ERROR)) { + if ((res < NO_ERROR) || (resFrameNumber != NO_ERROR)) { res = FAILED_TRANSACTION; } return res; @@ -167,7 +167,7 @@ public: status_t resFrameNumber = BAD_VALUE; if (reply.readInt32() != 0) { if (lastFrameNumber != NULL) { - res = reply.readInt64(lastFrameNumber); + resFrameNumber = reply.readInt64(lastFrameNumber); } } if ((res != NO_ERROR) || (resFrameNumber != NO_ERROR)) { @@ -296,7 +296,7 @@ public: status_t resFrameNumber = BAD_VALUE; if (reply.readInt32() != 0) { if (lastFrameNumber != NULL) { - res = reply.readInt64(lastFrameNumber); + resFrameNumber = reply.readInt64(lastFrameNumber); } } if ((res != NO_ERROR) || (resFrameNumber != NO_ERROR)) { |