From 5ce91bb0fd4fe8eb09f1a728e3b84cc1ff4244eb Mon Sep 17 00:00:00 2001 From: staz <74452129+alyanser@users.noreply.github.com> Date: Tue, 21 Mar 2023 19:01:24 +0000 Subject: Added overflow check for blur radius (#1847) * internal: added overflow check for blur radius --------- Co-authored-by: vaxerski <43317083+vaxerski@users.noreply.github.com> --- src/events/Monitors.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/events/Monitors.cpp') diff --git a/src/events/Monitors.cpp b/src/events/Monitors.cpp index 0412f43e..0ea64e67 100644 --- a/src/events/Monitors.cpp +++ b/src/events/Monitors.cpp @@ -241,7 +241,8 @@ void Events::listener_monitorFrame(void* owner, void* data) { // TODO: can this be optimized? static auto* const PBLURSIZE = &g_pConfigManager->getConfigValuePtr("decoration:blur_size")->intValue; static auto* const PBLURPASSES = &g_pConfigManager->getConfigValuePtr("decoration:blur_passes")->intValue; - const auto BLURRADIUS = *PBLURSIZE * pow(2, *PBLURPASSES); // is this 2^pass? I don't know but it works... I think. + const auto BLURRADIUS = + *PBLURPASSES > 10 ? pow(2, 15) : std::clamp(*PBLURSIZE, (int64_t)1, (int64_t)40) * pow(2, *PBLURPASSES); // is this 2^pass? I don't know but it works... I think. // now, prep the damage, get the extended damage region wlr_region_expand(&damage, &damage, BLURRADIUS); // expand for proper blurring -- cgit v1.2.3