efl ui slider - fix crashes with fmt callback data and class data
authorCarsten Haitzler (Rasterman) <raster@rasterman.com>
Fri, 24 Nov 2017 14:45:14 +0000 (23:45 +0900)
committerCarsten Haitzler (Rasterman) <raster@rasterman.com>
Fri, 24 Nov 2017 14:51:38 +0000 (23:51 +0900)
the format free cb was passed the class data (sd) as the callback data
but this was freed by the time _indi_default_format_free_cb was called
causing an invalid access. fix this by looking it up from the obj
which is probably invalid at this point...

src/lib/elementary/efl_ui_slider.c

index 1bc9ec9..b5da71e 100644 (file)
@@ -1421,7 +1421,7 @@ static void
 _indi_default_format_cb(void *data, Eina_Strbuf *str, const Eina_Value value)
 {
    const Eina_Value_Type *type = eina_value_type_get(&value);
-   Efl_Ui_Slider_Data *sd = data;
+   Efl_Ui_Slider_Data *sd = efl_data_scope_get(data, EFL_UI_SLIDER_CLASS);
    double v;
 
    if (type != EINA_VALUE_TYPE_DOUBLE) return;
@@ -1433,7 +1433,7 @@ _indi_default_format_cb(void *data, Eina_Strbuf *str, const Eina_Value value)
 static void
 _indi_default_format_free_cb(void *data)
 {
-   Efl_Ui_Slider_Data *sd = data;
+   Efl_Ui_Slider_Data *sd = efl_data_scope_get(data, EFL_UI_SLIDER_CLASS);
 
    if (sd && sd->indi_template)
      {
@@ -1451,7 +1451,7 @@ _efl_ui_slider_part_efl_ui_format_format_string_set(Eo *obj, void *_pd EINA_UNUS
    if (!template) return;
    eina_stringshare_replace(&sd->indi_template, template);
 
-   efl_ui_format_cb_set(efl_part(pd->obj, "indicator"), sd, _indi_default_format_cb, _indi_default_format_free_cb);
+   efl_ui_format_cb_set(efl_part(pd->obj, "indicator"), pd->obj, _indi_default_format_cb, _indi_default_format_free_cb);
 }
 
 EOLIAN static const char *