diff options
author | Samuel Pitoiset <[email protected]> | 2017-09-18 11:42:53 +0200 |
---|---|---|
committer | Samuel Pitoiset <[email protected]> | 2017-09-20 10:27:32 +0200 |
commit | 2ad1f20cd02b1b362d506924a52b7538f9b55d6a (patch) | |
tree | 5aaf7d62ed6ae536875fac3531a924bf75f53468 | |
parent | e5b6cdbf451e175e9408cd1fc5f27578a05bf9fb (diff) |
radv: add some assertions in vkCmdSetViewport()
To check some valid usage requirements.
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Dave Airlie <[email protected]>
-rw-r--r-- | src/amd/vulkan/radv_cmd_buffer.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 07c16cb4fbd..5644c1584e7 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -2497,8 +2497,11 @@ void radv_CmdSetViewport( const VkViewport* pViewports) { RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer); - const uint32_t total_count = firstViewport + viewportCount; + + assert(firstViewport < MAX_VIEWPORTS); + assert(total_count >= 1 && total_count <= MAX_VIEWPORTS); + if (cmd_buffer->state.dynamic.viewport.count < total_count) cmd_buffer->state.dynamic.viewport.count = total_count; |