From 9e74d0aea7614eaf238ef07261129026572337e7 Mon Sep 17 00:00:00 2001 From: Vaxry Date: Mon, 15 Sep 2025 12:44:12 +0100 Subject: renderer: clamp blur:passes 1-8 fixes some UB and dumb things ref #11707 --- src/render/pass/Pass.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src/render/pass/Pass.cpp') diff --git a/src/render/pass/Pass.cpp b/src/render/pass/Pass.cpp index f7ee6a71..3e14d787 100644 --- a/src/render/pass/Pass.cpp +++ b/src/render/pass/Pass.cpp @@ -294,7 +294,10 @@ float CRenderPass::oneBlurRadius() { // TODO: is this exact range correct? static auto PBLURSIZE = CConfigValue("decoration:blur:size"); static auto PBLURPASSES = CConfigValue("decoration:blur:passes"); - return *PBLURPASSES > 10 ? pow(2, 15) : std::clamp(*PBLURSIZE, sc(1), sc(40)) * pow(2, *PBLURPASSES); // is this 2^pass? I don't know but it works... I think. + + const auto BLUR_PASSES = std::clamp(*PBLURPASSES, sc(1), sc(8)); + + return std::clamp(*PBLURSIZE, sc(1), sc(40)) * pow(2, BLUR_PASSES); // is this 2^pass? I don't know but it works... I think. } void CRenderPass::removeAllOfType(const std::string& type) { -- cgit v1.2.3