summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/svg/qsvghandler.cpp3
-rw-r--r--tests/auto/qsvgrenderer/tst_qsvgrenderer.cpp8
2 files changed, 11 insertions, 0 deletions
diff --git a/src/svg/qsvghandler.cpp b/src/svg/qsvghandler.cpp
index 98a87e1..842af02 100644
--- a/src/svg/qsvghandler.cpp
+++ b/src/svg/qsvghandler.cpp
@@ -1533,6 +1533,9 @@ static void pathArc(QPainterPath &path,
qreal y,
qreal curx, qreal cury)
{
+ if (!rx || !ry)
+ return;
+
qreal sin_th, cos_th;
qreal a00, a01, a10, a11;
qreal x0, y0, x1, y1, xc, yc;
diff --git a/tests/auto/qsvgrenderer/tst_qsvgrenderer.cpp b/tests/auto/qsvgrenderer/tst_qsvgrenderer.cpp
index 686c854..f76a1b2 100644
--- a/tests/auto/qsvgrenderer/tst_qsvgrenderer.cpp
+++ b/tests/auto/qsvgrenderer/tst_qsvgrenderer.cpp
@@ -82,6 +82,7 @@ private slots:
void smallFont();
void styleSheet();
void duplicateStyleId();
+ void oss_fuzz_23731();
#ifndef QT_NO_COMPRESS
void testGzLoading();
@@ -1605,5 +1606,12 @@ void tst_QSvgRenderer::duplicateStyleId()
renderer.render(&painter);
}
+void tst_QSvgRenderer::oss_fuzz_23731()
+{
+ // when configured with "-sanitize undefined", this resulted in:
+ // "runtime error: division by zero"
+ QSvgRenderer().load(QByteArray("<svg><path d=\"A4------\">"));
+}
+
QTEST_MAIN(tst_QSvgRenderer)
#include "tst_qsvgrenderer.moc"