Fix deference after NULL issue 02/119202/1
authorJihoon Kim <jihoon48.kim@samsung.com>
Thu, 16 Mar 2017 03:48:10 +0000 (12:48 +0900)
committerJihoon Kim <jihoon48.kim@samsung.com>
Thu, 16 Mar 2017 03:48:19 +0000 (12:48 +0900)
After having been compared to NULL value at ise.cpp:671, pointer 'g_ui' is dereferenced at ise.cpp:677.

Change-Id: Id43fed950a453c1cc0ddeeae01ed7c5b81fcd885
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>
src/ise.cpp

index 5d24d46..238dde4 100644 (file)
@@ -668,13 +668,14 @@ void ise_show_help_popup(sclulong keyEvent)
     sclint scr_h = 0;
     SclRectangle rectangle = {0};
 
-    if (g_ui)
+    if (g_ui) {
         g_ui->get_screen_resolution(&scr_w, &scr_h);
 
-    if (keyEvent == MVK_space) {
-        g_ui->get_button_geometry("SPACE_KEY", &rectangle);
-    } else {
-        g_ui->get_button_geometry("CM_KEY", &rectangle);
+        if (keyEvent == MVK_space) {
+            g_ui->get_button_geometry("SPACE_KEY", &rectangle);
+        } else {
+            g_ui->get_button_geometry("CM_KEY", &rectangle);
+        }
     }
 
     _guide_popup_window = open_message_popup(NATIVE_WINDOW_CAST(ime_get_main_window()));