[PATCH weston] libweston-desktop: listen for the wl_surface resource destruction

Giulio Camuffo giuliocamuffo at gmail.com
Wed Sep 28 09:13:32 UTC 2016


Weston's desktop shell increments the ref_count on the surfaces so that
when the wl_resource of the wl_surface is destroyed the weston_surface
keeps living. In that case, the weston_desktop_surface keeps living too
but the shell resource doesn't. In the xdg_shell case it is destroyed
by the client with the destroy request, in the wl_shell case it needs
to be destroyed by the server when the wl_surface is destroyed.
That means that a weston_desktop_surface can have no active shell
implementation, and we need to guard that.
---
 libweston-desktop/client.c  |  2 +-
 libweston-desktop/surface.c | 63 +++++++++++++++++++++++++++++++++++----------
 2 files changed, 51 insertions(+), 14 deletions(-)

diff --git a/libweston-desktop/client.c b/libweston-desktop/client.c
index 810b6ba..a8968b8 100644
--- a/libweston-desktop/client.c
+++ b/libweston-desktop/client.c
@@ -186,7 +186,7 @@ weston_desktop_client_ping(struct weston_desktop_client *client)
 	void *implementation_data =
 		weston_desktop_surface_get_implementation_data(surface);
 
-	if (implementation->ping == NULL)
+	if (implementation == NULL || implementation->ping == NULL)
 		return -1;
 
 	if (client->ping_serial != 0)
diff --git a/libweston-desktop/surface.c b/libweston-desktop/surface.c
index 2205107..103f34f 100644
--- a/libweston-desktop/surface.c
+++ b/libweston-desktop/surface.c
@@ -54,6 +54,7 @@ struct weston_desktop_surface {
 	struct weston_position buffer_move;
 	struct wl_listener surface_commit_listener;
 	struct wl_listener surface_destroy_listener;
+	struct wl_listener wl_surface_resource_destroy_listener;
 	struct wl_listener client_destroy_listener;
 	struct wl_list children_list;
 
@@ -130,6 +131,7 @@ weston_desktop_surface_destroy(struct weston_desktop_surface *surface)
 
 	wl_list_remove(&surface->surface_commit_listener.link);
 	wl_list_remove(&surface->surface_destroy_listener.link);
+	wl_list_remove(&surface->wl_surface_resource_destroy_listener.link);
 	wl_list_remove(&surface->client_destroy_listener.link);
 
 	if (!wl_list_empty(&surface->resource_list)) {
@@ -140,7 +142,9 @@ weston_desktop_surface_destroy(struct weston_desktop_surface *surface)
 		}
 	}
 
-	surface->implementation->destroy(surface, surface->implementation_data);
+	if (surface->implementation)
+		surface->implementation->destroy(surface,
+						 surface->implementation_data);
 
 	surface->surface->committed = NULL;
 	surface->surface->committed_private = NULL;
@@ -169,6 +173,9 @@ weston_desktop_surface_surface_committed(struct wl_listener *listener,
 	struct weston_desktop_surface *surface =
 		wl_container_of(listener, surface, surface_commit_listener);
 
+	/* committed cannot be called if the resource for the wl_surface does
+	 * not exist anymore, so we don't need to check if implementation is
+	 * valid here. */
 	if (surface->implementation->committed != NULL)
 		surface->implementation->committed(surface,
 						   surface->implementation_data,
@@ -218,6 +225,20 @@ weston_desktop_surface_resource_destroy(struct wl_resource *resource)
 }
 
 static void
+weston_desktop_wl_surface_resource_destroyed(struct wl_listener *listener,
+					     void *data)
+{
+	struct weston_desktop_surface *surface =
+		wl_container_of(listener, surface,
+				wl_surface_resource_destroy_listener);
+
+	wl_list_init(&surface->wl_surface_resource_destroy_listener.link);
+
+	surface->implementation->destroy(surface, surface->implementation_data);
+	surface->implementation = NULL;
+}
+
+static void
 weston_desktop_surface_committed(struct weston_surface *wsurface,
 				 int32_t sx, int32_t sy)
 {
@@ -277,6 +298,10 @@ weston_desktop_surface_create(struct weston_desktop *desktop,
 		weston_desktop_surface_surface_destroyed;
 	wl_signal_add(&surface->surface->destroy_signal,
 		      &surface->surface_destroy_listener);
+	surface->wl_surface_resource_destroy_listener.notify =
+		weston_desktop_wl_surface_resource_destroyed;
+	wl_resource_add_destroy_listener(wsurface->resource,
+					 &surface->wl_surface_resource_destroy_listener);
 
 	wl_list_init(&surface->client_link);
 	wl_list_init(&surface->resource_list);
@@ -459,7 +484,8 @@ weston_desktop_surface_propagate_layer(struct weston_desktop_surface *surface)
 WL_EXPORT void
 weston_desktop_surface_set_activated(struct weston_desktop_surface *surface, bool activated)
 {
-	if (surface->implementation->set_activated != NULL)
+	if (surface->implementation != NULL &&
+	    surface->implementation->set_activated != NULL)
 		surface->implementation->set_activated(surface,
 						       surface->implementation_data,
 						       activated);
@@ -468,7 +494,8 @@ weston_desktop_surface_set_activated(struct weston_desktop_surface *surface, boo
 WL_EXPORT void
 weston_desktop_surface_set_fullscreen(struct weston_desktop_surface *surface, bool fullscreen)
 {
-	if (surface->implementation->set_fullscreen != NULL)
+	if (surface->implementation != NULL &&
+	    surface->implementation->set_fullscreen != NULL)
 		surface->implementation->set_fullscreen(surface,
 							surface->implementation_data,
 							fullscreen);
@@ -477,7 +504,8 @@ weston_desktop_surface_set_fullscreen(struct weston_desktop_surface *surface, bo
 WL_EXPORT void
 weston_desktop_surface_set_maximized(struct weston_desktop_surface *surface, bool maximized)
 {
-	if (surface->implementation->set_maximized != NULL)
+	if (surface->implementation != NULL &&
+	    surface->implementation->set_maximized != NULL)
 		surface->implementation->set_maximized(surface,
 						       surface->implementation_data,
 						       maximized);
@@ -486,7 +514,8 @@ weston_desktop_surface_set_maximized(struct weston_desktop_surface *surface, boo
 WL_EXPORT void
 weston_desktop_surface_set_resizing(struct weston_desktop_surface *surface, bool resizing)
 {
-	if (surface->implementation->set_resizing != NULL)
+	if (surface->implementation != NULL &&
+	    surface->implementation->set_resizing != NULL)
 		surface->implementation->set_resizing(surface,
 						      surface->implementation_data,
 						      resizing);
@@ -495,7 +524,8 @@ weston_desktop_surface_set_resizing(struct weston_desktop_surface *surface, bool
 WL_EXPORT void
 weston_desktop_surface_set_size(struct weston_desktop_surface *surface, int32_t width, int32_t height)
 {
-	if (surface->implementation->set_size != NULL)
+	if (surface->implementation != NULL &&
+	    surface->implementation->set_size != NULL)
 		surface->implementation->set_size(surface,
 						  surface->implementation_data,
 						  width, height);
@@ -504,7 +534,8 @@ weston_desktop_surface_set_size(struct weston_desktop_surface *surface, int32_t
 WL_EXPORT void
 weston_desktop_surface_close(struct weston_desktop_surface *surface)
 {
-	if (surface->implementation->close != NULL)
+	if (surface->implementation != NULL &&
+	    surface->implementation->close != NULL)
 		surface->implementation->close(surface,
 					       surface->implementation_data);
 }
@@ -606,7 +637,8 @@ weston_desktop_surface_get_pid(struct weston_desktop_surface *surface)
 WL_EXPORT bool
 weston_desktop_surface_get_activated(struct weston_desktop_surface *surface)
 {
-	if (surface->implementation->get_activated == NULL)
+	if (surface->implementation == NULL ||
+	    surface->implementation->get_activated == NULL)
 		return false;
 	return surface->implementation->get_activated(surface,
 						      surface->implementation_data);
@@ -615,7 +647,8 @@ weston_desktop_surface_get_activated(struct weston_desktop_surface *surface)
 WL_EXPORT bool
 weston_desktop_surface_get_resizing(struct weston_desktop_surface *surface)
 {
-	if (surface->implementation->get_resizing == NULL)
+	if (surface->implementation == NULL ||
+	    surface->implementation->get_resizing == NULL)
 		return false;
 	return surface->implementation->get_resizing(surface,
 						     surface->implementation_data);
@@ -624,7 +657,8 @@ weston_desktop_surface_get_resizing(struct weston_desktop_surface *surface)
 WL_EXPORT bool
 weston_desktop_surface_get_maximized(struct weston_desktop_surface *surface)
 {
-	if (surface->implementation->get_maximized == NULL)
+	if (surface->implementation == NULL ||
+	    surface->implementation->get_maximized == NULL)
 		return false;
 	return surface->implementation->get_maximized(surface,
 						      surface->implementation_data);
@@ -633,7 +667,8 @@ weston_desktop_surface_get_maximized(struct weston_desktop_surface *surface)
 WL_EXPORT bool
 weston_desktop_surface_get_fullscreen(struct weston_desktop_surface *surface)
 {
-	if (surface->implementation->get_fullscreen == NULL)
+	if (surface->implementation == NULL ||
+	    surface->implementation->get_fullscreen == NULL)
 		return false;
 	return surface->implementation->get_fullscreen(surface,
 						       surface->implementation_data);
@@ -652,7 +687,8 @@ weston_desktop_surface_get_max_size(struct weston_desktop_surface *surface)
 {
 	struct weston_size size = { 0, 0 };
 
-	if (surface->implementation->get_max_size == NULL)
+	if (surface->implementation == NULL ||
+	    surface->implementation->get_max_size == NULL)
 		return size;
 	return surface->implementation->get_max_size(surface,
 						     surface->implementation_data);
@@ -663,7 +699,8 @@ weston_desktop_surface_get_min_size(struct weston_desktop_surface *surface)
 {
 	struct weston_size size = { 0, 0 };
 
-	if (surface->implementation->get_min_size == NULL)
+	if (surface->implementation == NULL ||
+	    surface->implementation->get_min_size == NULL)
 		return size;
 	return surface->implementation->get_min_size(surface,
 						     surface->implementation_data);
-- 
2.10.0



More information about the wayland-devel mailing list