From 08a3497223fbfd6c88b81361e5a956e3b37cda4a Mon Sep 17 00:00:00 2001 From: Dylan Baker Date: Tue, 9 Sep 2025 20:56:04 +0000 Subject: [PATCH] anv: add assertion that tes and tcs data is non-null It doesn't make any sense ot have TCS but not TES (or vice versa), but coverity doesn't realize that. Add an assertion that they are both non-null before we start reading them. Fixes: 50fd669294f ("anv: prep work for separate tessellation shaders") CID: 1665360 CID: 1665327 Reviewed-by: Lionel Landwerlin Part-of: --- src/intel/vulkan/genX_gfx_state.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/intel/vulkan/genX_gfx_state.c b/src/intel/vulkan/genX_gfx_state.c index 8e698a323a0..7dccc5f4f40 100644 --- a/src/intel/vulkan/genX_gfx_state.c +++ b/src/intel/vulkan/genX_gfx_state.c @@ -2495,6 +2495,7 @@ cmd_buffer_flush_gfx_runtime_state(struct anv_gfx_dynamic_state *hw_state, if ((tcs_dynamic || tes_dynamic) && ((gfx->dirty & (ANV_CMD_DIRTY_HS | ANV_CMD_DIRTY_DS)) || BITSET_TEST(dyn->dirty, MESA_VK_DYNAMIC_TS_PATCH_CONTROL_POINTS))) { + assert(tcs_prog_data != NULL && tes_prog_data != NULL); SET(TESS_CONFIG, tess_config, intel_tess_config(dyn->ts.patch_control_points, tcs_prog_data->instances,