diff options
author | Zack Rusin <[email protected]> | 2013-05-08 23:48:20 -0400 |
---|---|---|
committer | Zack Rusin <[email protected]> | 2013-05-14 03:09:32 -0400 |
commit | 29853ab7b8656cee9b92a53bec43f6e9f1e49691 (patch) | |
tree | 6b4ac7f1accc0358717fe98d6e8b1c5b982baafb /src/gallium/auxiliary/draw/draw_context.h | |
parent | 386327c48f88b052449afa4f41b1090d3fdb5ce9 (diff) |
draw: don't crash on vertex buffer overflow
We would crash when stride was bigger than the size of the buffer.
The correct behavior is to just fetch zero's in this case.
Unfortunatly with user_buffer's there's no way to validate the size
because currently we're just not getting it. Adjust the draw interface
to pass the size along the mapped buffer, which works perfectly
for buffer backed vertex_buffers and, in future, it will allow
us to plumb user_buffer sizes through the same interface.
Signed-off-by: Zack Rusin <[email protected]>
Reviewed-by: José Fonseca <[email protected]>
Reviewed-by: Roland Scheidegger <[email protected]>
Diffstat (limited to 'src/gallium/auxiliary/draw/draw_context.h')
-rw-r--r-- | src/gallium/auxiliary/draw/draw_context.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/gallium/auxiliary/draw/draw_context.h b/src/gallium/auxiliary/draw/draw_context.h index 94fac880c68..c7a40a1b1a2 100644 --- a/src/gallium/auxiliary/draw/draw_context.h +++ b/src/gallium/auxiliary/draw/draw_context.h @@ -215,7 +215,8 @@ void draw_set_indexes(struct draw_context *draw, const void *elements, unsigned elem_size); void draw_set_mapped_vertex_buffer(struct draw_context *draw, - unsigned attr, const void *buffer); + unsigned attr, const void *buffer, + size_t size); void draw_set_mapped_constant_buffer(struct draw_context *draw, |