From: kibak.yoon <kibak.yoon@samsung.com>
Date: Mon, 10 Apr 2017 09:53:18 +0000 (+0900)
Subject: sensor-recorder: check parameters of some APIs
X-Git-Tag: submit/tizen/20170413.031950~2^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=69a3316337588990e5315b8a5a8ec5c02196debb;p=platform%2Fcore%2Fapi%2Fsensor.git

sensor-recorder: check parameters of some APIs

Change-Id: Ie364d4af76f9c2cb86ee137b878a8153c1c7a8fc
Signed-off-by: kibak.yoon <kibak.yoon@samsung.com>
---

diff --git a/src/sensor_recorder.cpp b/src/sensor_recorder.cpp
index 115b006..9eeaf77 100644
--- a/src/sensor_recorder.cpp
+++ b/src/sensor_recorder.cpp
@@ -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);