From f1da69d77a7e858d520b662e19bfd27b64f5ee4e Mon Sep 17 00:00:00 2001 From: Daniel Juyung Seo Date: Tue, 9 Apr 2013 16:58:29 +0900 Subject: [PATCH] elc_player: set internal variable to NULL after deleting the timer. This fixes elc_player crash issue. --- legacy/elementary/ChangeLog | 4 ++++ legacy/elementary/NEWS | 1 + legacy/elementary/src/lib/elc_player.c | 6 +++++- 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/legacy/elementary/ChangeLog b/legacy/elementary/ChangeLog index 02e4e61555..86842047d1 100644 --- a/legacy/elementary/ChangeLog +++ b/legacy/elementary/ChangeLog @@ -1221,3 +1221,7 @@ * Fix layout_example_02. * Fix hoversel_example_01. + +2013-04-09 Daniel Juyung Seo (SeoZ) + + * Fix elc_player crash issue. diff --git a/legacy/elementary/NEWS b/legacy/elementary/NEWS index b0cc27fda2..49b677f023 100644 --- a/legacy/elementary/NEWS +++ b/legacy/elementary/NEWS @@ -200,6 +200,7 @@ Fixes: * Fix ctxpopup geometry when parent is an elm_win. * Quit the scroll animator if the scroller don't have a bounce and reach a edge. * Fix the scroller show by a page if the page size is set and the region_bring_in or region_show is called. + * Fix elc_player crash issue. Removals: diff --git a/legacy/elementary/src/lib/elc_player.c b/legacy/elementary/src/lib/elc_player.c index 7fe372ef87..d622f970fc 100644 --- a/legacy/elementary/src/lib/elc_player.c +++ b/legacy/elementary/src/lib/elc_player.c @@ -171,7 +171,11 @@ _update_slider(void *data, elm_slider_min_max_set(sd->slider, 0, length); elm_slider_value_set(sd->slider, pos); sd->last_update_time = ecore_loop_time_get(); - if (sd->delay_update) ecore_timer_del(sd->delay_update); + if (sd->delay_update) + { + ecore_timer_del(sd->delay_update); + sd->delay_update = NULL; + } } static Eina_Bool