From ce44735aeb123dcf10fefce6922cb886cab89b86 Mon Sep 17 00:00:00 2001 From: Stephen Paul Weber Date: Wed, 18 Sep 2024 13:37:05 -0500 Subject: [PATCH] Avoid crashes due to setting value outside of range In case of bad value --- .../java/eu/siacs/conversations/entities/Conversation.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/main/java/eu/siacs/conversations/entities/Conversation.java b/src/main/java/eu/siacs/conversations/entities/Conversation.java index 1122f4c52efa5ff15b7877d524758a31e36b5eeb..755ff5013783c33a80acae889a6211e9ac01c3c9 100644 --- a/src/main/java/eu/siacs/conversations/entities/Conversation.java +++ b/src/main/java/eu/siacs/conversations/entities/Conversation.java @@ -2545,7 +2545,12 @@ public class Conversation extends AbstractEntity implements Blockable, Comparabl } if (field.getValues().size() > 0) { - binding.slider.setValue(Float.valueOf(field.getValue().getContent())); + final var val = Float.valueOf(field.getValue().getContent()); + if ((min == null || val >= min) && (max == null || val <= max)) { + binding.slider.setValue(Float.valueOf(field.getValue().getContent())); + } else { + binding.slider.setValue(min == null ? Float.MIN_VALUE : min); + } } else { binding.slider.setValue(min == null ? Float.MIN_VALUE : min); }