From fad1f716dd831fba4c380bce1cf76bffc11be6d6 Mon Sep 17 00:00:00 2001 From: Rhys Perry Date: Fri, 3 Feb 2023 17:27:52 +0000 Subject: [PATCH] aco: fix out-of-bounds access when moving s_mem(real)time across SMEM Signed-off-by: Rhys Perry Reviewed-by: Georg Lehmann Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/8224 Cc: mesa-stable Part-of: --- src/amd/compiler/aco_scheduler.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/amd/compiler/aco_scheduler.cpp b/src/amd/compiler/aco_scheduler.cpp index f097814..4ab13fe 100644 --- a/src/amd/compiler/aco_scheduler.cpp +++ b/src/amd/compiler/aco_scheduler.cpp @@ -679,7 +679,7 @@ schedule_SMEM(sched_ctx& ctx, Block* block, std::vector& registe current->operands[0].size() == 4)) break; /* don't move descriptor loads below buffer loads */ - if (candidate->format == Format::SMEM && current->operands[0].size() == 4 && + if (candidate->isSMEM() && !candidate->operands.empty() && current->operands[0].size() == 4 && candidate->operands[0].size() == 2) break; -- 2.7.4