Hi Roman,

On 18 April 2018 at 20:34, Roman Gilg <subd...@gmail.com> wrote:

>
> I believe this check should be ORed with xwl_window->window == window.
> Otherwise in case the top parent window is unrealized first,
> xwl_window->present_window != window holds here and afterwards
> xwl_present_cleanup won't be called again since the xwl_window is
> already gone. I.e. an existing frame callback wouldn't be destroyed in
> this case.
>

Sure, I've sent a new patch to do that, but I reckon this can wait after
the release, we might leak a frame callback in some cases (although
valgrind did not really complain yet), but we won't crash anymore.

Cheers,
Olivier
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to