compositor: do not free output region twice in weston_output_set_transform()

Ilia Bozhinov ammen99 at gmail.com
Wed Mar 14 08:59:52 UTC 2018


This is already done when weston_output_init_geometry() is called.
Actually this is a fix for 8564a0d, because without this patch, the
compositor sometimes crashes when setting output transform

Signed-off-by: Ilia Bozhinov <ammen99 at gmail.com>
---
 libweston/compositor.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/libweston/compositor.c b/libweston/compositor.c
index aec937bb..0079c848 100644
--- a/libweston/compositor.c
+++ b/libweston/compositor.c
@@ -4705,9 +4705,6 @@ weston_output_set_transform(struct weston_output *output,
 	pixman_region32_init(&old_region);
 	pixman_region32_copy(&old_region, &output->region);
 
-	pixman_region32_fini(&output->region);
-	pixman_region32_fini(&output->previous_damage);
-
 	weston_output_init_geometry(output, output->x, output->y);
 
 	output->dirty = 1;
-- 
2.14.3



More information about the wayland-devel mailing list