From 20f601a0fbc01a4f8ab85e54ee45f78f7817cdb5 Mon Sep 17 00:00:00 2001 From: Dave Airlie Date: Fri, 24 Aug 2012 12:56:18 +1000 Subject: xf86/crtc: don't free config->name This is set by pre_init not screen init, so if we free it here and then recycle the server, we lose all the providers. I think we need to wrap FreeScreen here to do this properly, will investigate for 1.14 most likely, safer to just leak this on server exit for now. Reviewed-by: Keith Packard Signed-off-by: Dave Airlie --- hw/xfree86/modes/xf86Crtc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/hw/xfree86/modes/xf86Crtc.c b/hw/xfree86/modes/xf86Crtc.c index 1947c5be9..e3eaf5f4c 100644 --- a/hw/xfree86/modes/xf86Crtc.c +++ b/hw/xfree86/modes/xf86Crtc.c @@ -728,8 +728,6 @@ xf86CrtcCloseScreen(ScreenPtr screen) xf86RandR12CloseScreen(screen); - free(config->name); - screen->CloseScreen(screen); for (o = 0; o < config->num_output; o++) { -- cgit v1.2.3