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 2021/05/21 06:26:27 UTC

svn commit: r1890059 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java

Author: tilman
Date: Fri May 21 06:26:27 2021
New Revision: 1890059

URL: http://svn.apache.org/viewvc?rev=1890059&view=rev
Log:
PDFBOX-5196, PDFBOX-5137: use numFrameComponents instead of numScanComponents to get component count, as suggested by Matej Risek and Stéphane Vallée; closes #110

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java?rev=1890059&r1=1890058&r2=1890059&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java Fri May 21 06:26:27 2021
@@ -224,12 +224,12 @@ public final class JPEGFactory
         try
         {
             XPath xpath = XPathFactory.newInstance().newXPath();
-            String numScanComponents = xpath.evaluate("markerSequence/sos/@numScanComponents", root);
-            if (numScanComponents.isEmpty())
+            String numFrameComponents = xpath.evaluate("markerSequence/sof/@numFrameComponents", root);
+            if (numFrameComponents.isEmpty())
             {
                 return 0;
             }
-            return Integer.parseInt(numScanComponents);
+            return Integer.parseInt(numFrameComponents);
         }
         catch (NumberFormatException ex)
         {