You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ja...@apache.org on 2015/08/12 01:41:17 UTC

svn commit: r1695411 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java

Author: jahewson
Date: Tue Aug 11 23:41:17 2015
New Revision: 1695411

URL: http://svn.apache.org/r1695411
Log:
PDFBOX-2929: Avoid  JVM crash when dash array contains zero

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java?rev=1695411&r1=1695410&r2=1695411&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java Tue Aug 11 23:41:17 2015
@@ -536,7 +536,11 @@ public class PageDrawer extends PDFGraph
             for (int i = 0; i < dashArray.length; ++i)
             {
                 // minimum line dash width avoids JVM crash, see PDFBOX-2373
-                dashArray[i] = Math.max(transformWidth(dashArray[i]), 0.016f);
+                float w = transformWidth(dashArray[i]);
+                if (w != 0)
+                {
+                    dashArray[i] = Math.max(w, 0.016f);
+                }
             }
             phaseStart = (int)transformWidth(phaseStart);