out of boundry check for param_list count 99/195999/1
authorSameer Prakash Pradhan <sp.pradhan@samsung.com>
Tue, 18 Dec 2018 12:57:36 +0000 (18:27 +0530)
committerSameer Prakash Pradhan <sp.pradhan@samsung.com>
Thu, 20 Dec 2018 13:27:53 +0000 (13:27 +0000)
Signed-off-by: Sameer Prakash Pradhan <sp.pradhan@samsung.com>
Change-Id: Ie5cd4bd4c348b34548797dc98b967dc40ae2ba73
(cherry picked from commit b455e368f6f5ecdf82bfa47fb0ccc34ac6e98960)

email-ipc/email-ipc-api/email-ipc-param-list.c
email-ipc/email-ipc-api/include/email-ipc-param-list.h

index 7cf7c23..8c06dec 100755 (executable)
@@ -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;
index 54a65cd..5849a6b 100755 (executable)
@@ -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;