summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSameer Prakash Pradhan <sp.pradhan@samsung.com>2018-12-18 18:27:36 +0530
committerSameer Prakash Pradhan <sp.pradhan@samsung.com>2018-12-20 11:01:44 +0530
commitb455e368f6f5ecdf82bfa47fb0ccc34ac6e98960 (patch)
tree672b7cc238312922298920bcb24fed8957656861
parent025c68c6ac141e042e44c68a50fb387421d89759 (diff)
downloademail-service-b455e368f6f5ecdf82bfa47fb0ccc34ac6e98960.tar.gz
email-service-b455e368f6f5ecdf82bfa47fb0ccc34ac6e98960.tar.bz2
email-service-b455e368f6f5ecdf82bfa47fb0ccc34ac6e98960.zip
out of boundry check for param_list count
Signed-off-by: Sameer Prakash Pradhan <sp.pradhan@samsung.com> Change-Id: Ie5cd4bd4c348b34548797dc98b967dc40ae2ba73
-rwxr-xr-xemail-ipc/email-ipc-api/email-ipc-param-list.c2
-rwxr-xr-xemail-ipc/email-ipc-api/include/email-ipc-param-list.h4
2 files changed, 4 insertions, 2 deletions
diff --git a/email-ipc/email-ipc-api/email-ipc-param-list.c b/email-ipc/email-ipc-api/email-ipc-param-list.c
index f3aefb9..f1f0a11 100755
--- a/email-ipc/email-ipc-api/email-ipc-param-list.c
+++ b/email-ipc/email-ipc-api/email-ipc-param-list.c
@@ -213,7 +213,7 @@ EXPORT_API int emipc_sum_param_list_length(emipc_param_list *param_list)
EXPORT_API bool emipc_add_param_to_param_list(emipc_param_list *param_list, void *data, int len)
{
EM_DEBUG_FUNC_BEGIN();
- if (emipc_set_param(&(param_list->params[param_list->param_count]), data, len)) {
+ if ( param_list->param_count < MAXPARAM_COUNT && emipc_set_param(&(param_list->params[param_list->param_count]), data, len)) {
param_list->param_count++;
EM_SAFE_FREE(param_list->byte_stream);
return true;
diff --git a/email-ipc/email-ipc-api/include/email-ipc-param-list.h b/email-ipc/email-ipc-api/include/email-ipc-param-list.h
index 54a65cd..5849a6b 100755
--- a/email-ipc/email-ipc-api/include/email-ipc-param-list.h
+++ b/email-ipc/email-ipc-api/include/email-ipc-param-list.h
@@ -26,6 +26,8 @@
#include "email-ipc-param.h"
#include "email-types.h"
+#define MAXPARAM_COUNT 10
+
typedef enum {
eSTREAM_APIID = 0,
eSTREAM_RESID,
@@ -37,7 +39,7 @@ typedef enum {
typedef struct {
int param_count;
- emipc_param params[10];
+ emipc_param params[MAXPARAM_COUNT];
unsigned char *byte_stream;
} emipc_param_list;