diff options
author | Sameer Prakash Pradhan <sp.pradhan@samsung.com> | 2018-12-18 18:27:36 +0530 |
---|---|---|
committer | Sameer Prakash Pradhan <sp.pradhan@samsung.com> | 2018-12-20 11:01:44 +0530 |
commit | b455e368f6f5ecdf82bfa47fb0ccc34ac6e98960 (patch) | |
tree | 672b7cc238312922298920bcb24fed8957656861 | |
parent | 025c68c6ac141e042e44c68a50fb387421d89759 (diff) | |
download | email-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-x | email-ipc/email-ipc-api/email-ipc-param-list.c | 2 | ||||
-rwxr-xr-x | email-ipc/email-ipc-api/include/email-ipc-param-list.h | 4 |
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; |