diff options
| author | Tom Stellard <[email protected]> | 2014-05-08 21:05:02 -0400 |
|---|---|---|
| committer | Tom Stellard <[email protected]> | 2014-05-09 04:32:54 -0400 |
| commit | c650033b863c3ac8388ff1b192b476f63cf81202 (patch) | |
| tree | 23fd7ec68e64d944b0d22945c549cebc823bc308 /src/gallium/auxiliary/pipe-loader | |
| parent | 23e9f06569f58062820ab058d8057e9325042105 (diff) | |
pipe-loader: Don't destroy the winsys in the sw loader
The screen takes ownership of the winsys, and is responsible for
destroying it. Users of pipe-loader should make sure they destory
and screens they've created to avoid memory leaks.
This fixes a crash in clover introduced by
ce6c17c0833032e91a2d1b34f9eb80c738a854a2 where the pipe-loader was
destroying the winsys while a screen was still using it.
Cc: "10.2" <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
Diffstat (limited to 'src/gallium/auxiliary/pipe-loader')
| -rw-r--r-- | src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c b/src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c index 08d43538efe..fa317f22aae 100644 --- a/src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c +++ b/src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c @@ -145,9 +145,6 @@ pipe_loader_sw_release(struct pipe_loader_device **dev) { struct pipe_loader_sw_device *sdev = pipe_loader_sw_device(*dev); - if (sdev->ws && sdev->ws->destroy) - sdev->ws->destroy(sdev->ws); - if (sdev->lib) util_dl_close(sdev->lib); |