coverity#1394372 Division or modulo by zero

Change-Id: I1d8d134bf423778a76f7dc6fc5a5e4f8c30c6c3d

1 files changed, 17 insertions, 12 deletions

diff --git a/vcl/source/fontsubset/sft.cxx b/vcl/source/fontsubset/sft.cxx
index d82c0828d601..8e9ccb9bee89 100644
--- a/vcl/source/fontsubset/sft.cxx
+++ b/vcl/source/fontsubset/sft.cxx
@@ -614,18 +614,23 @@ static int GetCompoundTTOutline(TrueTypeFont *ttf, sal_uInt32 glyphID, ControlPo
         if (abs3 < 0) abs3 = -abs3;
         if (abs3 <= 33) n *= 2;
 
-        for (i=0; i<np; i++) {
-            F16Dot16 t;
-            ControlPoint cp;
-            cp.flags = nextComponent[i].flags;
-            const sal_uInt16 x = nextComponent[i].x;
-            const sal_uInt16 y = nextComponent[i].y;
-            t = fixedMulDiv(a, x << 16, m) + fixedMulDiv(c, y << 16, m) + (e << 16);
-            cp.x = (sal_Int16)(fixedMul(t, m) >> 16);
-            t = fixedMulDiv(b, x << 16, n) + fixedMulDiv(d, y << 16, n) + (f << 16);
-            cp.y = (sal_Int16)(fixedMul(t, n) >> 16);
-
-            myPoints.push_back( cp );
+        SAL_WARN_IF(np && !m, "vcl.fonts", "Parsing error in " << OUString::createFromAscii(ttf->fname) <<
+                     ": " << "divide by zero");
+
+        if (m != 0) {
+            for (i=0; i<np; i++) {
+                F16Dot16 t;
+                ControlPoint cp;
+                cp.flags = nextComponent[i].flags;
+                const sal_uInt16 x = nextComponent[i].x;
+                const sal_uInt16 y = nextComponent[i].y;
+                t = fixedMulDiv(a, x << 16, m) + fixedMulDiv(c, y << 16, m) + (e << 16);
+                cp.x = (sal_Int16)(fixedMul(t, m) >> 16);
+                t = fixedMulDiv(b, x << 16, n) + fixedMulDiv(d, y << 16, n) + (f << 16);
+                cp.y = (sal_Int16)(fixedMul(t, n) >> 16);
+
+                myPoints.push_back( cp );
+            }
         }
 
         free(nextComponent);