diff options
author | Nicolai Hähnle <[email protected]> | 2017-02-22 14:00:29 +0100 |
---|---|---|
committer | Nicolai Hähnle <[email protected]> | 2017-03-22 12:12:29 +0100 |
commit | c11dcfb5e9b051b9036949b3e40a9dc15138bd97 (patch) | |
tree | 848b95a9ab1fdc56f1c56a2a9f327aac9f6708ee /src/mesa/tnl/t_vb_texmat.c | |
parent | c2dfff280ba7d0857e350581496daa17cccacb84 (diff) |
mesa/main: fix MultiDrawElements[BaseVertex] validation of primcount
primcount must be a GLsizei as in the signature for MultiDrawElements
or bad things can happen.
Furthermore, an error should be flagged when primcount is negative.
Curiously, this code used to work somewhat correctly even when primcount
was negative, because the loop that checks count[i] would iterate out of
bounds and almost certainly hit a negative value at some point.
Found by an ASAN error in
GL45-CTS.gtf32.GL3Tests.draw_elements_base_vertex.draw_elements_base_vertex_primcount
Note that the OpenGL spec seems to have s/primcount/drawcount/ at some
point, and the code still reflects the old language.
v2: provide the correct spec quotes (pointed out by Ian)
Cc: [email protected]
Reviewed-by: Marek Olšák <[email protected]> (v1)
Reviewed-by: Ian Romanick <[email protected]>
Diffstat (limited to 'src/mesa/tnl/t_vb_texmat.c')
0 files changed, 0 insertions, 0 deletions