Fix app event handler size 71/231971/2
authorYunjin Lee <yunjin-.lee@samsung.com>
Mon, 27 Apr 2020 05:31:01 +0000 (14:31 +0900)
committerYunjin Lee <yunjin-.lee@samsung.com>
Mon, 27 Apr 2020 05:39:18 +0000 (14:39 +0900)
- Wrong event handler size can cause stack smashing

Change-Id: Ie051e7dfec9d464cabb6a3c1678dc9ad97638329
Signed-off-by: Yunjin Lee <yunjin-.lee@samsung.com>
ui-popup/src/app_main.c
ui/src/main_privacy_setting.c

index 85c1cdf1c7a505dfb26cc67cf9a096b83c972baa..4fc68fcc9d9c17987c194f1532e6186f3bf60046 100644 (file)
@@ -220,9 +220,9 @@ int main(int argc, char *argv[])
        memset(&ad, 0x0, sizeof(papp_data_s));
 
        ui_app_lifecycle_callback_s event_callback = {0, };
-       app_event_handler_h handlers[2] = {NULL, };
+       app_event_handler_h handler = NULL;
 
-       ui_app_add_event_handler(&handlers[APP_EVENT_LANGUAGE_CHANGED], APP_EVENT_LANGUAGE_CHANGED, ui_app_lang_changed, &ad);
+       ui_app_add_event_handler(&handler, APP_EVENT_LANGUAGE_CHANGED, ui_app_lang_changed, &ad);
 
        event_callback.create = app_create;
        event_callback.terminate = app_terminate;
index 78cb2ac205b5fe052943c331eb353255515130f9..6d6e6fc92ad9a53ae82b2c12024ea51280af8645 100644 (file)
@@ -134,9 +134,9 @@ int main(int argc, char *argv[])
        memset(&ad, 0x0, sizeof(app_data_s));
 
        ui_app_lifecycle_callback_s event_callback = {0, };
-       app_event_handler_h handlers[2] = {NULL, };
+       app_event_handler_h handler = NULL;
 
-       ui_app_add_event_handler(&handlers[APP_EVENT_LANGUAGE_CHANGED], APP_EVENT_LANGUAGE_CHANGED, ui_app_lang_changed, &ad);
+       ui_app_add_event_handler(&handler, APP_EVENT_LANGUAGE_CHANGED, ui_app_lang_changed, &ad);
 
        event_callback.create = app_create;
        event_callback.terminate = app_terminate;