e_mod_effect: Fix possible path of dereferencing null object 47/116747/1 accepted/tizen/3.0/common/20170302.075828 accepted/tizen/3.0/ivi/20170302.034455 accepted/tizen/3.0/mobile/20170302.034152 accepted/tizen/3.0/tv/20170302.034309 accepted/tizen/3.0/wearable/20170302.034405 accepted/tizen/common/20170228.162244 accepted/tizen/ivi/20170302.120816 accepted/tizen/mobile/20170302.120329 accepted/tizen/tv/20170302.120408 accepted/tizen/unified/20170309.034545 accepted/tizen/wearable/20170302.120514 submit/tizen/20170228.130530 submit/tizen_3.0/20170228.132601 submit/tizen_unified/20170308.100410
authorHoyub Lee <hoyub.lee@samsung.com>
Tue, 28 Feb 2017 05:50:38 +0000 (14:50 +0900)
committerHoyub Lee <hoyub.lee@samsung.com>
Tue, 28 Feb 2017 05:50:38 +0000 (14:50 +0900)
Change-Id: I02558e1e6ae17337565e0c29f8c4217c473621d0
Signed-off-by: Hoyub Lee <hoyub.lee@samsung.com>
src/e_mod_effect.c

index eb977fc..a0f7b57 100644 (file)
@@ -239,6 +239,8 @@ _eff_object_setup(E_Client *ec, E_Effect_Group group)
    E_Comp_Config *cfg;
    cfg = e_comp_config_get();
 
+   if (!ec) return;
+
    if (group == E_EFFECT_GROUP_KEYBOARD)
      e_comp_object_effect_set(ec->frame, "keyboard");
    else
@@ -251,7 +253,7 @@ _eff_object_setup(E_Client *ec, E_Effect_Group group)
         while (parent_ec->parent)
           parent_ec = parent_ec->parent;
 
-        if ((parent_ec) && (parent_ec != ec) && (parent_ec->launching)) // Launchig case with parent at same time
+        if ((parent_ec != ec) && (parent_ec->launching)) // Launchig case with parent at same time
           e_comp_object_effect_set(ec->frame, cfg->effect_style);
         else if ((efc) && (e_policy_client_is_home_screen(efc->reverse_ec))) // Home key effect senario
           e_comp_object_effect_set(ec->frame, cfg->effect_style);