From: Jihoon Kim <jihoon48.kim@samsung.com>
Date: Wed, 22 Jun 2016 00:49:37 +0000 (+0900)
Subject: Fix dereference of NULL issue
X-Git-Tag: submit/tizen/20160622.104551~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6450fb3066845faaf0424eba08a63efee51209a4;p=platform%2Fcore%2Fuifw%2Fe-mod-tizen-wl-textinput.git

Fix dereference of NULL issue

Return value of a function 'vconf_get_str' is dereferenced at e_mod_main.c:157 without checking, but it is usually checked for this function.

Change-Id: Iaf5ac8527bbf8d399c37597f10674cfceeb6e031
---

diff --git a/src/e_mod_main.c b/src/e_mod_main.c
index cf71967..b4d0289 100644
--- a/src/e_mod_main.c
+++ b/src/e_mod_main.c
@@ -152,11 +152,14 @@ _display_language_changed_cb(keynode_t *key, void* data)
 
    /* Just in case we did not find any matching language string */
    g_keymap_index = 0;
-   for (loop = 0; loop < sizeof(g_keymap_info) / sizeof(struct _E_Input_Method_Keymap_Info); loop++)
+   if (language)
      {
-        if (strncmp(language, g_keymap_info[loop].language, strlen(g_keymap_info[loop].language)) == 0)
+        for (loop = 0; loop < sizeof(g_keymap_info) / sizeof(struct _E_Input_Method_Keymap_Info); loop++)
           {
-             g_keymap_index = loop;
+             if (strncmp(language, g_keymap_info[loop].language, strlen(g_keymap_info[loop].language)) == 0)
+               {
+                  g_keymap_index = loop;
+               }
           }
      }
    /* We do not want to change the current keymap related behavior in TV profile for now */