Clean up various memory leaks

Author: any1

src/buffer.c

@@ -538,6 +538,7 @@ void wv_buffer_pool_reconfig(struct wv_buffer_pool* pool,
 	if (old_node != config->node) {
 		if (pool->gbm)
 			gbm_device_destroy(pool->gbm);
+		pool->gbm = NULL;
 		if (pool->gbm_fd > 0)
 			close(pool->gbm_fd);
 		pool->gbm_fd = -1;

src/ext-image-copy-capture.c

@@ -95,6 +95,10 @@ static void ext_image_copy_capture_deinit_session(struct ext_image_copy_capture*
 	if (self->cursor)
 		ext_image_copy_capture_cursor_session_v1_destroy(self->cursor);
 	self->cursor = NULL;
+
+	if (self->buffer)
+		wv_buffer_pool_release(self->pool, self->buffer);
+	self->buffer = NULL;
 }
 
 static int ext_image_copy_capture_init_session(struct ext_image_copy_capture* self)
@@ -591,14 +595,8 @@ void ext_image_copy_capture_destroy(struct screencopy* ptr)
 	aml_stop(aml_get_default(), self->timer);
 	aml_unref(self->timer);
 
-	if (self->frame)
-		ext_image_copy_capture_frame_v1_destroy(self->frame);
-	if (self->cursor)
-		ext_image_copy_capture_cursor_session_v1_destroy(self->cursor);
-	if (self->session)
-		ext_image_copy_capture_session_v1_destroy(self->session);
-	if (self->buffer)
-		wv_buffer_pool_release(self->pool, self->buffer);
+	ext_image_copy_capture_deinit_session(self);
+
 	wv_buffer_pool_destroy(self->pool);
 	free(self);
 }

src/main.c

@@ -418,16 +418,13 @@ static void wayland_detach(struct wayvnc* self)
 		zwp_linux_dmabuf_v1_destroy(zwp_linux_dmabuf);
 	zwp_linux_dmabuf = NULL;
 
-	if (screencopy_manager) {
-		screencopy_stop(self->screencopy);
-		screencopy_destroy(self->screencopy);
-		self->screencopy = NULL;
-		screencopy_stop(self->cursor_sc);
-		screencopy_destroy(self->cursor_sc);
-		self->cursor_sc = NULL;
-		zwlr_screencopy_manager_v1_destroy(screencopy_manager);
-	}
-	screencopy_manager = NULL;
+	screencopy_stop(self->screencopy);
+	screencopy_destroy(self->screencopy);
+	self->screencopy = NULL;
+
+	screencopy_stop(self->cursor_sc);
+	screencopy_destroy(self->cursor_sc);
+	self->cursor_sc = NULL;
 
 	if (xdg_output_manager)
 		zxdg_output_manager_v1_destroy(xdg_output_manager);
@@ -477,6 +474,9 @@ static void wayland_detach(struct wayvnc* self)
 		aml_unref(self->capture_retry_timer);
 	self->capture_retry_timer = NULL;
 
+	if (self->transient_seat_manager)
+		ext_transient_seat_manager_v1_destroy(self->transient_seat_manager);
+
 	wl_registry_destroy(self->registry);
 	self->registry = NULL;
 

src/screencopy.c

@@ -356,6 +356,7 @@ static void wlr_screencopy_destroy(struct screencopy* ptr)
 	self->front = NULL;
 
 	wv_buffer_pool_destroy(self->pool);
+	free(self);
 }
 
 struct screencopy_impl wlr_screencopy_impl = {