From f30a018c5aecbded0adfb609c5bf180473149387 Mon Sep 17 00:00:00 2001 From: Lukas Matena Date: Mon, 22 Nov 2021 15:22:23 +0100 Subject: [PATCH] Clamp radius in variable layer height dialog, negative values led to a crash --- src/slic3r/GUI/GLCanvas3D.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/slic3r/GUI/GLCanvas3D.cpp b/src/slic3r/GUI/GLCanvas3D.cpp index e14b4e600..1c7fc4a61 100644 --- a/src/slic3r/GUI/GLCanvas3D.cpp +++ b/src/slic3r/GUI/GLCanvas3D.cpp @@ -271,8 +271,10 @@ void GLCanvas3D::LayersEditing::render_overlay(const GLCanvas3D& canvas) const ImGui::SetCursorPosX(widget_align); ImGui::PushItemWidth(imgui.get_style_scaling() * 120.0f); int radius = (int)m_smooth_params.radius; - if (ImGui::SliderInt("##1", &radius, 1, 10)) + if (ImGui::SliderInt("##1", &radius, 1, 10)) { + radius = std::clamp(radius, 1, 10); m_smooth_params.radius = (unsigned int)radius; + } ImGui::SetCursorPosX(text_align); ImGui::AlignTextToFramePadding();