Hi Daniel,

On Thu, Apr 19, 2018 at 5:24 PM, Daniel Stone <[email protected]> wrote:

> [...]
> The callback is passed as an argument to the function directly, so you
> could just destroy that instead.
>

Sure, but that wouldn't solve the problem which is the callback occurs
after the “xwl_window” is freed, meaning that we access freed memory here.
In other words, we could do that (and I agree it would be more elegant) but
that would probably crash on the next line or so.

Removing all frame callbacks when un-realizing the window as done with the
other patch that was merged (thanks Adam!) is safer.

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

Reply via email to