PDFBOX-6086: avoid IndexOutOfBoundsException

THausherr · THausherr · commit 9b2b9fcc4152 · 2025-10-21T20:01:53.000Z
git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1929257 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/fontbox/src/main/java/org/apache/fontbox/cff/Type1CharStringParser.java b/fontbox/src/main/java/org/apache/fontbox/cff/Type1CharStringParser.java
@@ -135,7 +135,7 @@ private void processCallSubr(List<byte[]> subrs, List<Object> sequence) throws I
             LOG.warn("CALLSUBR is ignored, operand: {}, subrs.size(): {} in glyph '{}' of font {}",
                     operand, subrs.size(), currentGlyph, fontName);
             // remove all parameters (there can be more than one)
-            while (sequence.get(sequence.size() - 1) instanceof Integer)
+            while (!sequence.isEmpty() && sequence.get(sequence.size() - 1) instanceof Integer)
             {
                 sequence.remove(sequence.size() - 1);
             }

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ private void processCallSubr(List<byte[]> subrs, List<Object> sequence) throws I`
`135`	`135`	`LOG.warn("CALLSUBR is ignored, operand: {}, subrs.size(): {} in glyph '{}' of font {}",`
`136`	`136`	`operand, subrs.size(), currentGlyph, fontName);`
`137`	`137`	`// remove all parameters (there can be more than one)`
`138`		`- while (sequence.get(sequence.size() - 1) instanceof Integer)`
	`138`	`+ while (!sequence.isEmpty() && sequence.get(sequence.size() - 1) instanceof Integer)`
`139`	`139`	`{`
`140`	`140`	`sequence.remove(sequence.size() - 1);`
`141`	`141`	`}`