Re: [PATCH weston v2] compositor: make sure to reset views' pointers to destroyed output

2015-07-14 Thread Daniel Stone
Hi, On 13 July 2015 at 11:26, Pekka Paalanen wrote: > On Sun, 12 Jul 2015 10:52:32 +0300 > Giulio Camuffo wrote: >> @@ -3888,9 +3891,15 @@ WL_EXPORT void >> weston_output_destroy(struct weston_output *output) >> { >> struct wl_resource *resource; >> + struct weston_view *view; >> >>

Re: [PATCH weston v2] compositor: make sure to reset views' pointers to destroyed output

2015-07-13 Thread Pekka Paalanen
On Sun, 12 Jul 2015 10:52:32 +0300 Giulio Camuffo wrote: > When an output is being destroyed reassign the output of the views > that were in it, to be sure not to keep a dangling pointer which could > be used later on by calling weston_surface_assign_output() on the > view's surface. > Also make

Re: [PATCH weston v2] compositor: make sure to reset views' pointers to destroyed output

2015-07-13 Thread Jonas Ådahl
On Sun, Jul 12, 2015 at 10:52:32AM +0300, Giulio Camuffo wrote: > When an output is being destroyed reassign the output of the views > that were in it, to be sure not to keep a dangling pointer which could > be used later on by calling weston_surface_assign_output() on the > view's surface. > Also

[PATCH weston v2] compositor: make sure to reset views' pointers to destroyed output

2015-07-12 Thread Giulio Camuffo
When an output is being destroyed reassign the output of the views that were in it, to be sure not to keep a dangling pointer which could be used later on by calling weston_surface_assign_output() on the view's surface. Also make sure we send wl_surface.leave events to the surfaces that were in tha

Re: [PATCH weston v2] compositor: make sure to reset views' pointers to destroyed output

2015-05-14 Thread Jonas Ådahl
On Thu, Jan 22, 2015 at 05:40:29PM +0200, Giulio Camuffo wrote: > When an output is destroyed reassign the output of the views that > were in it, to be sure not to keep a dangling pointer which could > be used later on by calling weston_surface_assign_output() on the > view's surface. Found this p

[PATCH weston v2] compositor: make sure to reset views' pointers to destroyed output

2015-01-22 Thread Giulio Camuffo
When an output is destroyed reassign the output of the views that were in it, to be sure not to keep a dangling pointer which could be used later on by calling weston_surface_assign_output() on the view's surface. --- src/compositor.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/c