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 2025/09/27 09:16:49 UTC
svn commit: r1928778 - pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf
Author: tilman
Date: Sat Sep 27 09:16:49 2025
New Revision: 1928778
Log:
PDFBOX-6078: avoid ArrayIndexOutOfBoundsException
Modified:
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java
Modified: pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java
==============================================================================
--- pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java Sat Sep 27 04:20:03 2025 (r1928777)
+++ pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java Sat Sep 27 09:16:49 2025 (r1928778)
@@ -799,9 +799,17 @@ public class GlyphSubstitutionTable exte
private int applyFeature(FeatureRecord featureRecord, int gid)
{
int lookupResult = gid;
+ LookupTable[] lookups = lookupListTable.getLookups();
for (int lookupListIndex : featureRecord.getFeatureTable().getLookupListIndices())
{
- LookupTable lookupTable = lookupListTable.getLookups()[lookupListIndex];
+ if (lookupListIndex < 0 || lookupListIndex >= lookups.length)
+ {
+ LOG.warn("Skipping GSUB feature '" + featureRecord.getFeatureTag() +
+ "' with invalid lookupListIndex " + lookupListIndex +
+ " (len: " + lookups.length + ")");
+ continue;
+ }
+ LookupTable lookupTable = lookups[lookupListIndex];
if (lookupTable.getLookupType() != 1)
{
LOG.warn("Skipping GSUB feature '" + featureRecord.getFeatureTag()