From: Michel Dänzer <daenzer@vmware.com>
Date: Thu, 12 Feb 2009 19:13:09 +0000 (+0100)
Subject: gallium: Fix a couple of potential NULL pointer dereferences.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=5af34758e3bba55cb8c227ae1256818e8f112727;p=mesa.git

gallium: Fix a couple of potential NULL pointer dereferences.

A lot more test programs work.
---

diff --git a/src/gallium/auxiliary/util/u_rect.c b/src/gallium/auxiliary/util/u_rect.c
index 2aceda12f89..6e24e594e46 100644
--- a/src/gallium/auxiliary/util/u_rect.c
+++ b/src/gallium/auxiliary/util/u_rect.c
@@ -173,10 +173,6 @@ util_surface_copy(struct pipe_context *pipe,
    void *dst_map;
    const void *src_map;
 
-   assert(dst_trans->block.size == src_trans->block.size);
-   assert(dst_trans->block.width == src_trans->block.width);
-   assert(dst_trans->block.height == src_trans->block.height);
-
    assert(src->texture && dst->texture);
    if (!src->texture || !dst->texture)
       return;
@@ -196,6 +192,10 @@ util_surface_copy(struct pipe_context *pipe,
                                         PIPE_TRANSFER_WRITE,
                                         dst_x, dst_y, w, h);
 
+   assert(dst_trans->block.size == src_trans->block.size);
+   assert(dst_trans->block.width == src_trans->block.width);
+   assert(dst_trans->block.height == src_trans->block.height);
+
    src_map = pipe->screen->transfer_map(screen, src_trans);
    dst_map = pipe->screen->transfer_map(screen, dst_trans);
 
diff --git a/src/gallium/drivers/softpipe/sp_tile_cache.c b/src/gallium/drivers/softpipe/sp_tile_cache.c
index 8534c7745bb..ccc518eb61b 100644
--- a/src/gallium/drivers/softpipe/sp_tile_cache.c
+++ b/src/gallium/drivers/softpipe/sp_tile_cache.c
@@ -158,11 +158,11 @@ void
 sp_tile_cache_set_surface(struct softpipe_tile_cache *tc,
                           struct pipe_surface *ps)
 {
-   struct pipe_screen *screen = ps->texture->screen;
-
    assert(!tc->texture);
 
    if (tc->transfer) {
+      struct pipe_screen *screen = tc->transfer->texture->screen;
+
       if (ps == tc->surface)
          return;
 
@@ -176,18 +176,22 @@ sp_tile_cache_set_surface(struct softpipe_tile_cache *tc,
 
    tc->surface = ps;
 
-   tc->transfer = screen->get_tex_transfer(screen, ps->texture, ps->face,
-                                           ps->level, ps->zslice,
-                                           PIPE_TRANSFER_READ_WRITE,
-                                           0, 0, ps->width, ps->height);
-
-   tc->depth_stencil = (ps->format == PIPE_FORMAT_S8Z24_UNORM ||
-                        ps->format == PIPE_FORMAT_X8Z24_UNORM ||
-                        ps->format == PIPE_FORMAT_Z24S8_UNORM ||
-                        ps->format == PIPE_FORMAT_Z24X8_UNORM ||
-                        ps->format == PIPE_FORMAT_Z16_UNORM ||
-                        ps->format == PIPE_FORMAT_Z32_UNORM ||
-                        ps->format == PIPE_FORMAT_S8_UNORM);
+   if (ps) {
+      struct pipe_screen *screen = ps->texture->screen;
+
+      tc->transfer = screen->get_tex_transfer(screen, ps->texture, ps->face,
+                                              ps->level, ps->zslice,
+                                              PIPE_TRANSFER_READ_WRITE,
+                                              0, 0, ps->width, ps->height);
+
+      tc->depth_stencil = (ps->format == PIPE_FORMAT_S8Z24_UNORM ||
+                           ps->format == PIPE_FORMAT_X8Z24_UNORM ||
+                           ps->format == PIPE_FORMAT_Z24S8_UNORM ||
+                           ps->format == PIPE_FORMAT_Z24X8_UNORM ||
+                           ps->format == PIPE_FORMAT_Z16_UNORM ||
+                           ps->format == PIPE_FORMAT_Z32_UNORM ||
+                           ps->format == PIPE_FORMAT_S8_UNORM);
+   }
 }
 
 
@@ -235,18 +239,22 @@ sp_tile_cache_set_texture(struct pipe_context *pipe,
                           struct softpipe_tile_cache *tc,
                           struct pipe_texture *texture)
 {
-   struct pipe_screen *screen = texture->screen;
    uint i;
 
    assert(!tc->transfer);
 
    pipe_texture_reference(&tc->texture, texture);
 
-   if (tc->tex_trans_map) {
-      tc->screen->transfer_unmap(tc->screen, tc->tex_trans);
-      tc->tex_trans_map = NULL;
+   if (tc->transfer) {
+      struct pipe_screen *screen = tc->transfer->texture->screen;
+
+      if (tc->tex_trans_map) {
+         tc->screen->transfer_unmap(tc->screen, tc->tex_trans);
+         tc->tex_trans_map = NULL;
+      }
+
+      screen->tex_transfer_release(screen, &tc->tex_trans);
    }
-   screen->tex_transfer_release(screen, &tc->tex_trans);
 
    /* mark as entries as invalid/empty */
    /* XXX we should try to avoid this when the teximage hasn't changed */
@@ -539,9 +547,12 @@ sp_get_cached_tile_tex(struct softpipe_context *sp,
           tc->tex_z != z) {
          /* get new transfer (view into texture) */
 
-	 if (tc->tex_trans_map)
-            tc->screen->transfer_unmap(tc->screen, tc->tex_trans);
-         screen->tex_transfer_release(screen, &tc->tex_trans);
+         if (tc->transfer) {
+            if (tc->tex_trans_map)
+               tc->screen->transfer_unmap(tc->screen, tc->tex_trans);
+
+            screen->tex_transfer_release(screen, &tc->tex_trans);
+         }
 
          tc->tex_trans = screen->get_tex_transfer(screen, tc->texture, face, level, z, 
                                                   PIPE_TRANSFER_READ, 0, 0,