From 350ae815c213e17e6a00919cd9be5f8a275380d7 Mon Sep 17 00:00:00 2001 From: "Jesper K. Pedersen" Date: Thu, 11 Dec 2025 15:05:15 +0100 Subject: [PATCH] Fix crash when converting a QPointF to a QPoint that was NAN The code actually checked that the number was valid, but only after the conversion. --- src/KDChart/Cartesian/DiagramFlavors/KDChartNormalPlotter_p.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/KDChart/Cartesian/DiagramFlavors/KDChartNormalPlotter_p.cpp b/src/KDChart/Cartesian/DiagramFlavors/KDChartNormalPlotter_p.cpp index ef6d925b..57ebc78d 100644 --- a/src/KDChart/Cartesian/DiagramFlavors/KDChartNormalPlotter_p.cpp +++ b/src/KDChart/Cartesian/DiagramFlavors/KDChartNormalPlotter_p.cpp @@ -145,7 +145,7 @@ void NormalPlotter::paint(PaintContext *ctx) m_private->addLabel(&lpc, sourceIndex, nullptr, pts, Position::NorthWest, Position::NorthWest, point.value); - const bool lineValid = a.toPoint() != b.toPoint() && PaintingHelpers::isFinite(a); + const bool lineValid = PaintingHelpers::isFinite(a) && a.toPoint() != b.toPoint(); if (lineValid) { // data line lineList.append(LineAttributesInfo(sourceIndex, a, b));