From: Marek Olšák <maraeo@gmail.com>
Date: Mon, 21 Jan 2013 14:48:55 +0000 (+0100)
Subject: st/dri: add null-pointer check, remove duplicated local variable
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=fffe3e090878175269af8672d20c14a0ccab78a9;p=mesa.git

st/dri: add null-pointer check, remove duplicated local variable

Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
Tested-by: Michel Dänzer <michel.daenzer@amd.com>
---

diff --git a/src/gallium/state_trackers/dri/common/dri_drawable.c b/src/gallium/state_trackers/dri/common/dri_drawable.c
index 2ea54781d17..d817a8c2178 100644
--- a/src/gallium/state_trackers/dri/common/dri_drawable.c
+++ b/src/gallium/state_trackers/dri/common/dri_drawable.c
@@ -446,6 +446,7 @@ dri_flush(__DRIcontext *cPriv,
 
    /* Flush the context and throttle if needed. */
    if (dri_screen(ctx->sPriv)->throttling_enabled &&
+       drawable &&
        (reason == __DRI2_THROTTLE_SWAPBUFFER ||
         reason == __DRI2_THROTTLE_FLUSHFRONT)) {
       /* Throttle.
@@ -458,19 +459,19 @@ dri_flush(__DRIcontext *cPriv,
        * pushes that fence on the queue. This requires that the st_context_iface
        * flush method returns a fence even if there are no commands to flush.
        */
-      struct dri_drawable *draw = dri_drawable(dPriv);
-      struct pipe_screen *screen = draw->screen->base.screen;
+      struct pipe_screen *screen = drawable->screen->base.screen;
       struct pipe_fence_handle *fence;
 
-      fence = swap_fences_pop_front(draw);
+      fence = swap_fences_pop_front(drawable);
       if (fence) {
          (void) screen->fence_finish(screen, fence, PIPE_TIMEOUT_INFINITE);
          screen->fence_reference(screen, &fence, NULL);
       }
 
       ctx->st->flush(ctx->st, flush_flags, &fence);
+
       if (fence) {
-         swap_fences_push_back(draw, fence);
+         swap_fences_push_back(drawable, fence);
          screen->fence_reference(screen, &fence, NULL);
       }
    }