[PATCH 1/6] srccompositor.c: set surface->plane from destroyed plane to NULL

Xiong Zhang xiong.y.zhang at intel.com
Thu Oct 17 04:10:48 CEST 2013


In drm backend, the cursor_surface->plane point to drm_output->cursor_plane.
when this output is removed, drm_output->cursor_plane is destroyed, but
cursor_surface->plane still point to destroyed plane. So once mouse move to this
cursor_surface and system will repaint this cursor_surface, segment fault will occure in
weston_surface_damage_below() function.

Eech client APP has its own cursor surface, if mouse focus isn't on this app's cursor surface,
this app's cursor surface won't be on any cursor_layer and compositor->surface_list.But once
moving mouse to this APP, this APP's cursor surface will be in compositor->surface_list. In
order to track these hiden cursor surface,plane should track all the surfaces belonged to it,
when plane is destroyed, all surface's plane on destroyed plane is set to NULL.

bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=69777

v2: set surface->plane to NULL whose plane point to unplugged output, then
    change weston_surface_damage_below() to do nothing if surface->plane is NULL (Kristian)

Signed-off-by: Xiong Zhang <xiong.y.zhang at intel.com>
---
 src/compositor.c | 22 +++++++++++++++++++---
 src/compositor.h |  2 ++
 2 files changed, 21 insertions(+), 3 deletions(-)

diff --git a/src/compositor.c b/src/compositor.c
index 376ddfd..990a378 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -362,6 +362,7 @@ weston_surface_create(struct weston_compositor *compositor)
 
 	wl_list_init(&surface->link);
 	wl_list_init(&surface->layer_link);
+	wl_list_init(&surface->plane_link);
 
 	surface->compositor = compositor;
 	surface->alpha = 1.0;
@@ -377,7 +378,7 @@ weston_surface_create(struct weston_compositor *compositor)
 	surface->pending.buffer_transform = surface->buffer_transform;
 	surface->pending.buffer_scale = surface->buffer_scale;
 	surface->output = NULL;
-	surface->plane = &compositor->primary_plane;
+	surface->plane = NULL;
 	surface->pending.newly_attached = 0;
 
 	pixman_region32_init(&surface->damage);
@@ -566,7 +567,10 @@ weston_surface_move_to_plane(struct weston_surface *surface,
 		return;
 
 	weston_surface_damage_below(surface);
+	if (surface->plane)
+		wl_list_remove(&surface->plane_link);
 	surface->plane = plane;
+	wl_list_insert(plane->surface_list.prev, &surface->plane_link);
 	weston_surface_damage(surface);
 }
 
@@ -578,8 +582,9 @@ weston_surface_damage_below(struct weston_surface *surface)
 	pixman_region32_init(&damage);
 	pixman_region32_subtract(&damage, &surface->transform.boundingbox,
 				 &surface->clip);
-	pixman_region32_union(&surface->plane->damage,
-			      &surface->plane->damage, &damage);
+	if (surface->plane)
+		pixman_region32_union(&surface->plane->damage,
+				&surface->plane->damage, &damage);
 	pixman_region32_fini(&damage);
 }
 
@@ -1125,6 +1130,9 @@ weston_surface_destroy(struct weston_surface *surface)
 
 	weston_surface_set_transform_parent(surface, NULL);
 
+	if (surface->plane)
+		wl_list_remove(&surface->plane_link);
+
 	free(surface);
 }
 
@@ -2614,14 +2622,22 @@ weston_plane_init(struct weston_plane *plane, int32_t x, int32_t y)
 	/* Init the link so that the call to wl_list_remove() when releasing
 	 * the plane without ever stacking doesn't lead to a crash */
 	wl_list_init(&plane->link);
+	wl_list_init(&plane->surface_list);
 }
 
 WL_EXPORT void
 weston_plane_release(struct weston_plane *plane)
 {
+	struct weston_surface *surface, *next;
+
 	pixman_region32_fini(&plane->damage);
 	pixman_region32_fini(&plane->clip);
 
+	wl_list_for_each_safe(surface, next, &plane->surface_list, plane_link) {
+		wl_list_remove(&surface->plane_link);
+		surface->plane = NULL;
+	}
+
 	wl_list_remove(&plane->link);
 }
 
diff --git a/src/compositor.h b/src/compositor.h
index a19d966..bb00c79 100644
--- a/src/compositor.h
+++ b/src/compositor.h
@@ -508,6 +508,7 @@ struct weston_plane {
 	pixman_region32_t damage;
 	pixman_region32_t clip;
 	int32_t x, y;
+	struct wl_list surface_list;
 	struct wl_list link;
 };
 
@@ -724,6 +725,7 @@ struct weston_surface {
 	pixman_region32_t input;
 	struct wl_list link;
 	struct wl_list layer_link;
+	struct wl_list plane_link;
 	float alpha;                     /* part of geometry, see below */
 	struct weston_plane *plane;
 	int32_t ref_count;
-- 
1.8.3.2



More information about the wayland-devel mailing list