You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by ta...@apache.org on 2025/01/28 16:33:51 UTC

(tika) branch TIKA-4337b created (now 84e2c5c4c)

This is an automated email from the ASF dual-hosted git repository.

tallison pushed a change to branch TIKA-4337b
in repository https://gitbox.apache.org/repos/asf/tika.git


      at 84e2c5c4c TIKA-4337 -- check for empty string in array index 1

This branch includes the following new commits:

     new 84e2c5c4c TIKA-4337 -- check for empty string in array index 1

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



(tika) 01/01: TIKA-4337 -- check for empty string in array index 1

Posted by ta...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

tallison pushed a commit to branch TIKA-4337b
in repository https://gitbox.apache.org/repos/asf/tika.git

commit 84e2c5c4cf12a782128902ce955891ad3f7246bb
Author: tallison <ta...@apache.org>
AuthorDate: Tue Jan 28 11:33:36 2025 -0500

    TIKA-4337 -- check for empty string in array index 1
---
 .../tika/parser/microsoft/ooxml/xps/XPSPageContentHandler.java       | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/xps/XPSPageContentHandler.java b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/xps/XPSPageContentHandler.java
index e6464639f..c6114ab23 100644
--- a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/xps/XPSPageContentHandler.java
+++ b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/xps/XPSPageContentHandler.java
@@ -32,6 +32,7 @@ import org.xml.sax.helpers.DefaultHandler;
 import org.apache.tika.metadata.Metadata;
 import org.apache.tika.metadata.TikaCoreProperties;
 import org.apache.tika.sax.XHTMLContentHandler;
+import org.apache.tika.utils.StringUtils;
 
 
 /**
@@ -247,10 +248,10 @@ class XPSPageContentHandler extends DefaultHandler {
             for (String indexString : indicesString.split(";", -1)) {
                 // We only want to extract the advance which will be the second comma separated value
                 String[] commaSplit = indexString.split(",", -1);
-                if (commaSplit.length < 2) {
+                if (commaSplit.length < 2 || StringUtils.isBlank(commaSplit[1])) {
                     indices.add(new GlyphIndex(0.0f));
                 } else {
-                    // Advance is measured in hundreths so divide by 100
+                    // Advance is measured in hundredths so divide by 100
                     float advance = Float.parseFloat(commaSplit[1]) / 100.0f;
                     indices.add(new GlyphIndex(advance));
                 }