From dc76a5ea5369e6fff43b3ad1d4cc43d0cb735d65 Mon Sep 17 00:00:00 2001 From: JunsuChoi Date: Wed, 19 Sep 2018 19:42:15 +0900 Subject: [PATCH] ctxpopup: Add return value check for elm_layout_theme_set Using elm_layout_theme_set without checking return value. so add return value check and log message. Change-Id: I849f04464df71b6e5bbd0a677b9f0031a70aaac6 --- src/lib/elementary_tizen/elc_ctxpopup.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/lib/elementary_tizen/elc_ctxpopup.c b/src/lib/elementary_tizen/elc_ctxpopup.c index a1b5244..0cf7ccb 100644 --- a/src/lib/elementary_tizen/elc_ctxpopup.c +++ b/src/lib/elementary_tizen/elc_ctxpopup.c @@ -1488,7 +1488,8 @@ _elm_ctxpopup_efl_ui_widget_theme_apply(Eo *obj, Elm_Ctxpopup_Data *sd) if (sd->scr) { - elm_layout_theme_set(sd->scr, "scroller", "ctxpopup", elm_widget_style_get(obj)); + if (!elm_layout_theme_set(sd->scr, "scroller", "ctxpopup", elm_widget_style_get(obj))) + CRI("Failed to set layout!"); if (sd->horizontal) elm_scroller_policy_set(sd->scr, ELM_SCROLLER_POLICY_AUTO, ELM_SCROLLER_POLICY_OFF); @@ -2013,7 +2014,8 @@ _list_new(Evas_Object *obj) ELM_WIDGET_DATA_GET_OR_RETURN(obj, wd); //scroller sd->scr = elm_scroller_add(obj); - elm_layout_theme_set(sd->scr, "scroller", "ctxpopup", elm_widget_style_get(obj)); + if (!elm_layout_theme_set(sd->scr, "scroller", "ctxpopup", elm_widget_style_get(obj))) + CRI("Failed to set layout!"); evas_object_size_hint_align_set(sd->scr, EVAS_HINT_FILL, EVAS_HINT_FILL); _elm_widget_color_class_parent_set(sd->scr, obj); -- 2.7.4