I'd be interested in knowing how you came across this issue since that
functionality should be handled automatically on surface hide. This patch
is not a complete solution and will break some seat methods, so it would be
great if we could get that cleaned up.

On Tue, Jan 16, 2018 at 3:50 AM Carsten Haitzler <ras...@rasterman.com>
wrote:

> raster pushed a commit to branch master.
>
>
> http://git.enlightenment.org/core/efl.git/commit/?id=574ef9a5e53ac2ecabc14fd02be7e55cd90a9b25
>
> commit 574ef9a5e53ac2ecabc14fd02be7e55cd90a9b25
> Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com>
> Date:   Tue Jan 16 14:50:31 2018 +0900
>
>     efl wl - fix segv where garbage ptr to comp surface is still there
>
>     comp surface destruction doesnt clear the pointer to one in the comp
>     struct that then is garbage later. fix.
> ---
>  src/lib/efl_wl/efl_wl.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/lib/efl_wl/efl_wl.c b/src/lib/efl_wl/efl_wl.c
> index 28a0814710..40667cabf9 100644
> --- a/src/lib/efl_wl/efl_wl.c
> +++ b/src/lib/efl_wl/efl_wl.c
> @@ -2321,6 +2321,9 @@ comp_surface_smart_del(Evas_Object *obj)
>  {
>     Comp_Surface *cs = evas_object_smart_data_get(obj);
>
> +   if ((cs->c) && (cs->c->active_surface == cs))
> +     cs->c->active_surface = NULL;
> +
>     array_clear(&cs->input_rects);
>     array_clear(&cs->opaque_rects);
>     eina_tiler_free(cs->opaque);
>
> --
>
>
>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to