From 9e3cae38d04d3e1a8934eef6b2aea18e192f39b8 Mon Sep 17 00:00:00 2001 From: Mu-Woong Lee Date: Fri, 1 Jul 2016 15:45:00 +0900 Subject: [PATCH] sensor: set the maximum retention period (1 month) Change-Id: Id37d909394ad1db7b8aafad4f673f7e9ef326f67 Signed-off-by: Mu-Woong Lee --- src/sensor/SensorProvider.cpp | 3 +++ src/sensor/TypesInternal.h | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/sensor/SensorProvider.cpp b/src/sensor/SensorProvider.cpp index e205246..32f0584 100644 --- a/src/sensor/SensorProvider.cpp +++ b/src/sensor/SensorProvider.cpp @@ -119,6 +119,9 @@ int SensorProvider::__addClient(std::string pkgId, int retentionPeriod, Json opt Json tmp; int ret; + /* Validate the retention period */ + IF_FAIL_RETURN(retentionPeriod > 0 && retentionPeriod <= MAX_RETENTION_PERIOD, ERR_INVALID_PARAMETER); + /* Check if the app already started Sensor recording */ ret = __clientInfo.get(getSubject(), pkgId, tmp); IF_FAIL_RETURN(ret != ERR_NONE, ERR_ALREADY_STARTED); diff --git a/src/sensor/TypesInternal.h b/src/sensor/TypesInternal.h index 8d84066..1894520 100644 --- a/src/sensor/TypesInternal.h +++ b/src/sensor/TypesInternal.h @@ -33,7 +33,7 @@ #define SECONDS_PER_HOUR 3600 #define SECONDS_PER_DAY 86400 -/* Default Parameters */ +#define MAX_RETENTION_PERIOD 2678400 /* 1 month (31 days) */ #define DEFAULT_RETENTION SECONDS_PER_HOUR /* 1 hour */ #define DEFAULT_QUERY_PERIOD SECONDS_PER_DAY /* 1 day */ -- 2.34.1