st/mesa: fix missing setting of _ElementSize in new_draw_rasterpos_stage
authorCharmaine Lee <charmainel@vmware.com>
Thu, 26 Apr 2018 16:21:52 +0000 (09:21 -0700)
committerCharmaine Lee <charmainel@vmware.com>
Thu, 26 Apr 2018 17:29:02 +0000 (10:29 -0700)
With this patch, _ElementSize is initialized along with the rest
of the vertex array attributes in new_draw_rasterpos_stage().
This fixes a crash in st_pipe_vertex_format() when running
topogun-1.06-orc-84k-resize trace file with VMware svga driver.

Reviewed-by: Brian Paul <brianp@vmware.com>
src/mesa/state_tracker/st_cb_rasterpos.c

index 4e5417baada2a1344d4ec3d82fbe27842d85d8bc..b73d543653f245e2235df49f4a399750c01c179c 100644 (file)
@@ -40,6 +40,7 @@
 #include "main/macros.h"
 #include "main/feedback.h"
 #include "main/rastpos.h"
+#include "glformats.h"
 
 #include "st_context.h"
 #include "st_atom.h"
@@ -182,6 +183,7 @@ new_draw_rastpos_stage(struct gl_context *ctx, struct draw_context *draw)
 {
    struct rastpos_stage *rs = ST_CALLOC_STRUCT(rastpos_stage);
    GLuint i;
+   GLuint elementSize;
 
    rs->stage.draw = draw;
    rs->stage.next = NULL;
@@ -196,12 +198,15 @@ new_draw_rastpos_stage(struct gl_context *ctx, struct draw_context *draw)
 
    rs->binding.Stride = 0;
    rs->binding.BufferObj = NULL;
+
+   elementSize = _mesa_bytes_per_vertex_attrib(4, GL_FLOAT);
    for (i = 0; i < ARRAY_SIZE(rs->array); i++) {
       rs->attrib[i].Size = 4;
       rs->attrib[i].Type = GL_FLOAT;
       rs->attrib[i].Format = GL_RGBA;
       rs->attrib[i].Ptr = (GLubyte *) ctx->Current.Attrib[i];
       rs->attrib[i].Normalized = GL_TRUE;
+      rs->attrib[i]._ElementSize = elementSize;
       rs->array[i].BufferBinding = &rs->binding;
       rs->array[i].VertexAttrib = &rs->attrib[i];
    }