Mesa (master): r300g: fix use of uninitialised variables.

Dave Airlie airlied at kemper.freedesktop.org
Sat Dec 26 22:17:08 PST 2009


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

Author: Dave Airlie <airlied at redhat.com>
Date:   Sun Dec 27 16:13:41 2009 +1000

r300g: fix use of uninitialised variables.

These buffers were getting dereferenced later.

---

 src/gallium/winsys/drm/radeon/core/radeon_drm.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/gallium/winsys/drm/radeon/core/radeon_drm.c b/src/gallium/winsys/drm/radeon/core/radeon_drm.c
index dec7c06..05194fc 100644
--- a/src/gallium/winsys/drm/radeon/core/radeon_drm.c
+++ b/src/gallium/winsys/drm/radeon/core/radeon_drm.c
@@ -206,7 +206,7 @@ static boolean radeon_shared_handle_from_texture(struct drm_api *api,
     int retval, fd;
     struct drm_gem_flink flink;
     struct radeon_pipe_buffer* radeon_buffer;
-    struct pipe_buffer *buffer;
+    struct pipe_buffer *buffer = NULL;
 
     if (!radeon_buffer_from_texture(api, texture, &buffer, stride)) {
         return FALSE;
@@ -239,7 +239,7 @@ static boolean radeon_local_handle_from_texture(struct drm_api *api,
                                                 unsigned *stride,
                                                 unsigned *handle)
 {
-    struct pipe_buffer *buffer;
+    struct pipe_buffer *buffer = NULL;
     if (!radeon_buffer_from_texture(api, texture, &buffer, stride)) {
         return FALSE;
     }



More information about the mesa-commit mailing list