fix svace issues 13/141813/1 accepted/tizen_3.0_ivi accepted/tizen/3.0/common/20170803.140841 accepted/tizen/3.0/ivi/20170803.075656 accepted/tizen/3.0/mobile/20170803.075631 accepted/tizen/3.0/tv/20170803.075619 accepted/tizen/3.0/wearable/20170803.075644 submit/tizen_3.0/20170802.102112
authorJengHyun Kang <jhyuni.kang@samsung.com>
Tue, 1 Aug 2017 12:02:08 +0000 (21:02 +0900)
committerJengHyun Kang <jhyuni.kang@samsung.com>
Tue, 1 Aug 2017 12:02:08 +0000 (21:02 +0900)
Change-Id: Ia18a9d6c6b8a6e19f3998c9344744c0b75409bcf

src/e_mod_keyrouter_events.c
src/e_mod_keyrouter_list.c
src/e_mod_main_wl.c

index 2a6ed84..77b669a 100644 (file)
@@ -40,6 +40,7 @@ _e_keyrouter_event_generate_key(Ecore_Event_Key *ev, int type, struct wl_client
 
    len = sizeof(Ecore_Event_Key) + strlen(ev->key) + strlen(ev->keyname) + ((ev->compose) ? strlen(ev->compose) : 0) + 3;
    ev_cpy = calloc(1, len);
+   if (!ev_cpy) return;
    memcpy(ev_cpy, ev, len);
    ev_cpy->data = send_surface;
 
index 81c8cf1..0e97f4d 100644 (file)
@@ -524,13 +524,16 @@ _e_keyrouter_build_register_list(void)
                   if (!krt->HardKeys[*ddata].registered_ptr)
                     {
                        node = E_NEW(E_Keyrouter_Key_List_Node, 1);
-                       node->surface = surface;
-                       node->wc = NULL;
-                       node->focused = EINA_FALSE;
-                       node->status = E_KRT_CSTAT_ALIVE;
-                       krt->HardKeys[*ddata].registered_ptr = node;
-
-                       KLDBG("%d key's register wl_surface is %p", *ddata, surface);
+                       if (node)
+                         {
+                            node->surface = surface;
+                            node->wc = NULL;
+                            node->focused = EINA_FALSE;
+                            node->status = E_KRT_CSTAT_ALIVE;
+                            krt->HardKeys[*ddata].registered_ptr = node;
+
+                            KLDBG("%d key's register wl_surface is %p", *ddata, surface);
+                         }
                     }
                }
 
@@ -647,8 +650,11 @@ _e_keyrouter_add_registered_surface_in_list(struct wl_resource *surface, int key
              if (EINA_TRUE == key_finded) break;
 
              key_added = E_NEW(int, 1);
-             *key_added = key;
-             rwin_info->keys = eina_list_append(rwin_info->keys, key_added);
+             if (key_added)
+               {
+                  *key_added = key;
+                  rwin_info->keys = eina_list_append(rwin_info->keys, key_added);
+               }
              surface_finded = EINA_TRUE;
 
              KLDBG("Registered Key(%d) is added to wl_surface (%p)", key, surface);
@@ -659,8 +665,14 @@ _e_keyrouter_add_registered_surface_in_list(struct wl_resource *surface, int key
    if (EINA_FALSE == surface_finded)
      {
         rwin_added = E_NEW(E_Keyrouter_Registered_Window_Info, 1);
+        if (!rwin_added) return;
         rwin_added->surface = surface;
         key_added = E_NEW(int, 1);
+        if (!key_added)
+          {
+             free(rwin_added);
+             return;
+          }
         *key_added = key;
         rwin_added->keys = eina_list_append(rwin_added->keys, key_added);
         krt->registered_window_list = eina_list_append(krt->registered_window_list, rwin_added);
index bfefbd2..990dfaf 100644 (file)
@@ -1009,7 +1009,6 @@ _e_keyrouter_keygrab_print(void *data, const char *log_path)
                        fprintf(log_fl, "                    -- Client Information --\n");
                        fprintf(log_fl, "                        = connected fd: %d\n", wl_client_get_fd(kdata->wc));
                     }
-                  break;
                }
             }