CameraHAL: CTS fixes related to #testMeteringAreas

- The status of "parseAreas()" should always be checked
  in order to return proper error code in case of invalid
  input arguments.
- If the input metering areas are more than the supported
  ones, then always return an error.
- 'parseAreas()' expects the buffer size, not the string
  length.
- Refactors "parseFocusAreas()' to the more general
  'parseAreas()'.

Change-Id: I4c70bcaa3c676f525b721916cc39c66db5e0fc17

Signed-off-by: Emilian Peev <epeev@mm-sol.com>

Signed-off-by: Sundar Raman <sunds@ti.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/camera/CameraHalUtilClasses.cpp b/camera/CameraHalUtilClasses.cpp
index b0d937c..c8e5eff 100644
--- a/camera/CameraHalUtilClasses.cpp
+++ b/camera/CameraHalUtilClasses.cpp
@@ -197,9 +197,9 @@ status_t CameraArea::checkArea(ssize_t top,
     return NO_ERROR;

 }

 

-status_t CameraArea::parseFocusArea(const char *area,

-                               size_t areaLength,

-                               Vector< sp<CameraArea> > &areas)

+status_t CameraArea::parseAreas(const char *area,

+                                size_t areaLength,

+                                Vector< sp<CameraArea> > &areas)

 {

     status_t ret = NO_ERROR;

     char *ctx;