sensor-recorder: check parameters of some APIs 48/124148/1
authorkibak.yoon <kibak.yoon@samsung.com>
Mon, 10 Apr 2017 09:53:18 +0000 (18:53 +0900)
committerkibak.yoon <kibak.yoon@samsung.com>
Mon, 10 Apr 2017 09:53:18 +0000 (18:53 +0900)
Change-Id: Ie364d4af76f9c2cb86ee137b878a8153c1c7a8fc
Signed-off-by: kibak.yoon <kibak.yoon@samsung.com>
src/sensor_recorder.cpp

index 115b006..9eeaf77 100644 (file)
@@ -94,6 +94,9 @@ int sensor_recorder_start(sensor_type_e type, sensor_recorder_option_h option)
 {
        int ret = SENSOR_ERROR_NONE;
 
+       if (type <= SENSOR_ALL)
+               return SENSOR_ERROR_INVALID_PARAMETER;
+
        ret = ctx_sensor_rec_start(sensor_keys[type].c_str(),
                        reinterpret_cast<ctx_sensor_rec_option_h>(option));
 
@@ -104,6 +107,9 @@ int sensor_recorder_stop(sensor_type_e type)
 {
        int ret = SENSOR_ERROR_NONE;
 
+       if (type <= SENSOR_ALL)
+               return SENSOR_ERROR_INVALID_PARAMETER;
+
        ret = ctx_sensor_rec_stop(sensor_keys[type].c_str());
 
        return convert_error(ret);
@@ -180,6 +186,9 @@ int sensor_recorder_read(sensor_type_e type, sensor_recorder_query_h query, sens
 {
        int ret = SENSOR_ERROR_NONE;
 
+       if (sensor_keys.find(type) == sensor_keys.end())
+               return SENSOR_ERROR_INVALID_PARAMETER;
+
        ret = ctx_sensor_rec_read(sensor_keys[type].c_str(),
                        reinterpret_cast<ctx_sensor_rec_query_h>(query), (ctx_sensor_rec_data_cb)cb, user_data);
 
@@ -190,6 +199,9 @@ int sensor_recorder_read_sync(sensor_type_e type, sensor_recorder_query_h query,
 {
        int ret = SENSOR_ERROR_NONE;
 
+       if (sensor_keys.find(type) == sensor_keys.end())
+               return SENSOR_ERROR_INVALID_PARAMETER;
+
        ret = ctx_sensor_rec_read_sync(sensor_keys[type].c_str(),
                        reinterpret_cast<ctx_sensor_rec_query_h>(query), (ctx_sensor_rec_data_cb)cb, user_data);