Mesa (master): st/dri: prevent leak of dri option default values

Aaron Watry awatry at kemper.freedesktop.org
Thu Jan 16 16:10:22 UTC 2014


Module: Mesa
Branch: master
Commit: ce3528896b37c7d8ef051780e29ea9588fada9da
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ce3528896b37c7d8ef051780e29ea9588fada9da

Author: Aaron Watry <awatry at gmail.com>
Date:   Wed Dec 18 14:40:49 2013 -0600

st/dri: prevent leak of dri option default values

v2: Change comment style

CC: "10.0" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>

---

 src/gallium/state_trackers/dri/common/dri_screen.c |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/gallium/state_trackers/dri/common/dri_screen.c b/src/gallium/state_trackers/dri/common/dri_screen.c
index 7410dbe..b338f64 100644
--- a/src/gallium/state_trackers/dri/common/dri_screen.c
+++ b/src/gallium/state_trackers/dri/common/dri_screen.c
@@ -360,6 +360,12 @@ dri_destroy_option_cache(struct dri_screen * screen)
    }
 
    free(screen->optionCache.values);
+
+   /* Default values are copied to screen->optionCache->values in
+    * initOptionCache. The info field, however, is a pointer copy, so don't free
+    * that twice.
+    */
+   free(screen->optionCacheDefaults.values);
 }
 
 void




More information about the mesa-commit mailing list