mesa: fix memory leak when generating mipmaps for compressed textures
authorBrian Paul <brianp@vmware.com>
Thu, 1 Oct 2009 20:55:13 +0000 (14:55 -0600)
committerBrian Paul <brianp@vmware.com>
Thu, 1 Oct 2009 20:55:13 +0000 (14:55 -0600)
src/mesa/main/mipmap.c

index 3dca09d9f289ae39612f90b7cc999d1d2f1ec230..4d3e62572d5149304147eaf50efbfe2cecba63f8 100644 (file)
@@ -1510,7 +1510,9 @@ _mesa_generate_mipmap(GLcontext *ctx, GLenum target,
 
    /* Find convertFormat - the format that do_row() will process */
    if (srcImage->IsCompressed) {
-      /* setup for compressed textures */
+      /* setup for compressed textures - need to allocate temporary
+       * image buffers to hold uncompressed images.
+       */
       GLuint row;
       GLint  components, size;
       GLchan *dst;
@@ -1587,11 +1589,7 @@ _mesa_generate_mipmap(GLcontext *ctx, GLenum target,
                                          &dstWidth, &dstHeight, &dstDepth);
       if (!nextLevel) {
          /* all done */
-         if (srcImage->IsCompressed) {
-            _mesa_free((void *) srcData);
-            _mesa_free(dstData);
-         }
-         return;
+         break;
       }
 
       /* get dest gl_texture_image */
@@ -1682,6 +1680,12 @@ _mesa_generate_mipmap(GLcontext *ctx, GLenum target,
       }
 
    } /* loop over mipmap levels */
+
+   if (srcImage->IsCompressed) {
+      /* free uncompressed image buffers */
+      _mesa_free((void *) srcData);
+      _mesa_free(dstData);
+   }
 }