From 6c8d778646c5f61ef463b70d8d563bd290e068c6 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Kristian=20H=C3=B8gsberg?= Date: Tue, 25 Jun 2013 10:29:30 -0400 Subject: [PATCH] compositor: Remove double free()s in resource destructors With the change to move free()ing of the wl_resource into wayland-server, we now have a few cases where we double free the resource in the destructor. This patch removes those. --- src/compositor.c | 2 -- src/input.c | 1 - src/text-backend.c | 4 ---- 3 files changed, 7 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index af273b9..7b1e503 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -2206,8 +2206,6 @@ subsurface_resource_destroy(struct wl_resource *resource) if (sub) weston_subsurface_destroy(sub); - - free(resource); } static void diff --git a/src/input.c b/src/input.c index ad4512d..7360dcb 100644 --- a/src/input.c +++ b/src/input.c @@ -40,7 +40,6 @@ empty_region(pixman_region32_t *region) static void unbind_resource(struct wl_resource *resource) { wl_list_remove(wl_resource_get_link(resource)); - free(resource); } void diff --git a/src/text-backend.c b/src/text-backend.c index cc7d65d..38728e9 100644 --- a/src/text-backend.c +++ b/src/text-backend.c @@ -533,8 +533,6 @@ unbind_keyboard(struct wl_resource *resource) input_method_context_end_keyboard_grab(context); context->keyboard = NULL; - - free(resource); } static void @@ -746,8 +744,6 @@ unbind_input_method(struct wl_resource *resource) input_method->context = NULL; text_backend->input_method.binding = NULL; - - free(resource); } static void -- 2.7.4