summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSamuel Pitoiset <[email protected]>2017-09-18 11:42:53 +0200
committerSamuel Pitoiset <[email protected]>2017-09-20 10:27:32 +0200
commit2ad1f20cd02b1b362d506924a52b7538f9b55d6a (patch)
tree5aaf7d62ed6ae536875fac3531a924bf75f53468
parente5b6cdbf451e175e9408cd1fc5f27578a05bf9fb (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.c5
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;