Mesa (master): intel: Fix improper freeing of texture data in TFP.

Eric Anholt anholt at kemper.freedesktop.org
Mon Sep 26 22:38:35 UTC 2011


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

Author: Eric Anholt <eric at anholt.net>
Date:   Wed Sep 21 15:17:36 2011 -0700

intel: Fix improper freeing of texture data in TFP.

If there happened to be ->Data present, we assertion failed instead of
handling it correctly.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=35234
Acked-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/mesa/drivers/dri/intel/intel_tex_image.c |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/src/mesa/drivers/dri/intel/intel_tex_image.c b/src/mesa/drivers/dri/intel/intel_tex_image.c
index 15196c0..30e1776 100644
--- a/src/mesa/drivers/dri/intel/intel_tex_image.c
+++ b/src/mesa/drivers/dri/intel/intel_tex_image.c
@@ -686,10 +686,7 @@ intel_set_texture_image_region(struct gl_context *ctx,
 			      region->width, region->height, 1,
 			      0, internalFormat, format);
 
-   if (intel_image->mt) {
-      intel_miptree_release(intel, &intel_image->mt);
-      assert(!image->Data);
-   }
+   ctx->Driver.FreeTextureImageBuffer(ctx, image);
 
    intel_image->mt = intel_miptree_create_for_region(intel, target,
 						     image->TexFormat,




More information about the mesa-commit mailing list