You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2022/12/30 13:10:04 UTC

svn commit: r1906283 - /pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java

Author: tilman
Date: Fri Dec 30 13:10:04 2022
New Revision: 1906283

URL: http://svn.apache.org/viewvc?rev=1906283&view=rev
Log:
PDFBOX-5554: sonar fix

Modified:
    pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java

Modified: pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java?rev=1906283&r1=1906282&r2=1906283&view=diff
==============================================================================
--- pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java (original)
+++ pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java Fri Dec 30 13:10:04 2022
@@ -171,14 +171,20 @@ public class TextToPDF implements Callab
                     }
                 }
             }
-            InputStream is = new FileInputStream(infile);
-            if (hasUtf8BOM)
+            try (InputStream is = new FileInputStream(infile))
             {
-                is.skip(3);
-            }
-            try (Reader reader = new InputStreamReader(is, charset))
-            {
-                createPDFFromText(doc, reader);
+                if (hasUtf8BOM)
+                {
+                    long skipped = is.skip(3);
+                    if (skipped != 3)
+                    {
+                        throw new IOException("Could not skip 3 bytes, size changed?!");
+                    }
+                }
+                try (Reader reader = new InputStreamReader(is, charset))
+                {
+                    createPDFFromText(doc, reader);
+                }
             }
             doc.save(outfile);
         }