diff options
| author | Dave Airlie <airlied@redhat.com> | 2012-08-24 12:56:18 +1000 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2012-09-04 16:15:52 +1000 |
| commit | 20f601a0fbc01a4f8ab85e54ee45f78f7817cdb5 (patch) | |
| tree | 33975c09e3a636f51fa004309d14d2c59ff629d6 | |
| parent | 148583d62b84832369e9df39b3e527b99ca96761 (diff) | |
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 <keithp@keithp.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
| -rw-r--r-- | hw/xfree86/modes/xf86Crtc.c | 2 |
1 files changed, 0 insertions, 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++) { |