You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2022/06/02 05:47:09 UTC

svn commit: r1901535 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java

Author: lehmi
Date: Thu Jun  2 05:47:09 2022
New Revision: 1901535

URL: http://svn.apache.org/viewvc?rev=1901535&view=rev
Log:
PDFBOX-5143: ensure to copy all bytes

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java?rev=1901535&r1=1901534&r2=1901535&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java Thu Jun  2 05:47:09 2022
@@ -90,7 +90,13 @@ public class CFFParser
         // TODO do we need to store the source data of the font? It isn't used at all
         byte[] bytes = new byte[(int) randomAccessRead.length()];
         randomAccessRead.seek(0);
-        randomAccessRead.read(bytes);
+        int remainingBytes = bytes.length;
+        int amountRead;
+        while ((amountRead = randomAccessRead.read(bytes, bytes.length - remainingBytes,
+                remainingBytes)) > 0)
+        {
+            remainingBytes -= amountRead;
+        }
         randomAccessRead.seek(0);
         this.source = new CFFBytesource(bytes);
         return parse(new DataInputRandomAccessRead(randomAccessRead));