Mesa (7.11): i965: Fix Coverity wrong sizeof argument defect.

Ian Romanick idr at kemper.freedesktop.org
Thu Jan 19 20:57:12 UTC 2012


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

Author: Vinson Lee <vlee at vmware.com>
Date:   Thu Nov  3 00:36:27 2011 -0700

i965: Fix Coverity wrong sizeof argument defect.

NOTE: This is a candidate for stable release branches.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=42542
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
(cherry picked from commit 3e18ad7fd7026e31fd92fe5cf7097d08e5b9a15a)

---

 src/mesa/drivers/dri/i965/gen7_viewport_state.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/gen7_viewport_state.c b/src/mesa/drivers/dri/i965/gen7_viewport_state.c
index 838ad3a..be6a324 100644
--- a/src/mesa/drivers/dri/i965/gen7_viewport_state.c
+++ b/src/mesa/drivers/dri/i965/gen7_viewport_state.c
@@ -36,7 +36,7 @@ prepare_sf_clip_viewport(struct brw_context *brw)
    const GLfloat *v = ctx->Viewport._WindowMap.m;
    struct gen7_sf_clip_viewport *vp;
 
-   vp = brw_state_batch(brw, sizeof(vp), 64, &brw->sf.vp_offset);
+   vp = brw_state_batch(brw, sizeof(*vp), 64, &brw->sf.vp_offset);
    /* Also assign to clip.vp_offset in case something uses it. */
    brw->clip.vp_offset = brw->sf.vp_offset;
 




More information about the mesa-commit mailing list