Fix a whoopsie

swaywm · Nov 28, 2021 · 9bed12b · 9bed12b
1 parent e9939fc
commit 9bed12b
Showing 1 changed file with 16 additions and 10 deletions.
diff --git a/types/wlr_zext_screencopy_v1.c b/types/wlr_zext_screencopy_v1.c
@@ -212,11 +212,14 @@ static void surface_commit(struct wl_client *client,
 	surface->current_buffer.resource = surface->staged_buffer.resource;
 	surface->staged_buffer.resource = NULL;
 
-	wl_list_remove(&surface->staged_buffer.destroy.link);
-	wl_resource_add_destroy_listener(surface->current_buffer.resource,
-			&surface->current_buffer.destroy);
-	surface->current_buffer.destroy.notify =
-		surface_handle_committed_buffer_destroy;
+	if (surface->current_buffer.resource) {
+		wl_list_remove(&surface->staged_buffer.destroy.link);
+		wl_resource_add_destroy_listener(
+				surface->current_buffer.resource,
+				&surface->current_buffer.destroy);
+		surface->current_buffer.destroy.notify =
+			surface_handle_committed_buffer_destroy;
+	}
 
 	pixman_region32_copy(&surface->current_buffer.damage,
 			&surface->staged_buffer.damage);
@@ -230,11 +233,14 @@ static void surface_commit(struct wl_client *client,
 		surface->staged_cursor_buffer.resource;
 	surface->staged_cursor_buffer.resource = NULL;
 
-	wl_list_remove(&surface->staged_cursor_buffer.destroy.link);
-	wl_resource_add_destroy_listener(surface->current_cursor_buffer.resource,
-			&surface->current_cursor_buffer.destroy);
-	surface->current_cursor_buffer.destroy.notify =
-		surface_handle_committed_cursor_buffer_destroy;
+	if (surface->current_cursor_buffer.resource) {
+		wl_list_remove(&surface->staged_cursor_buffer.destroy.link);
+		wl_resource_add_destroy_listener(
+				surface->current_cursor_buffer.resource,
+				&surface->current_cursor_buffer.destroy);
+		surface->current_cursor_buffer.destroy.notify =
+			surface_handle_committed_cursor_buffer_destroy;
+	}
 
 	pixman_region32_copy(&surface->current_cursor_buffer.damage,
 			&surface->staged_cursor_buffer.damage);