[PATCH 4/6] compositor: first destroy backend routines and then display

Tiago Vignatti vignatti at freedesktop.org
Mon Dec 19 02:30:26 PST 2011


From: Tiago Vignatti <tiago.vignatti at intel.com>

I caught this when an evdev device fd was trying to trigger the main event
loop, which was already free'd and causing an invalid read.

Signed-off-by: Tiago Vignatti <tiago.vignatti at intel.com>
---
 compositor/compositor.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/compositor/compositor.c b/compositor/compositor.c
index fc75689..a87c658 100644
--- a/compositor/compositor.c
+++ b/compositor/compositor.c
@@ -2092,9 +2092,9 @@ int main(int argc, char *argv[])
 
 	if (ec->has_bind_display)
 		ec->unbind_display(ec->display, display);
-	wl_display_destroy(display);
 
 	ec->destroy(ec);
+	wl_display_destroy(display);
 
 	return 0;
 }
-- 
1.7.5.4



More information about the wayland-devel mailing list