Bug 66425: Avoid exceptions found via poi-fuzz

Prevent a NullPointerException Fixes https://issues.oss-fuzz.com/issues/379574870 git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1923059 13f79535-47bb-0310-9956-ffa450edef68
apache · Jan 11, 2025 · 57afb34 · 57afb34
1 parent edcbd87
commit 57afb34
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/poi/src/main/java/org/apache/poi/sl/extractor/SlideShowExtractor.java b/poi/src/main/java/org/apache/poi/sl/extractor/SlideShowExtractor.java
@@ -364,6 +364,10 @@ private static String replaceTextCap(TextRun tr) {
         // PowerPoint seems to store files with \r as the line break
         // The messes things up on everything but a Mac, so translate them to \n
         String txt = tr.getRawText();
+        if (txt == null) {
+            return "";
+        }
+
         txt = txt.replace('\r', '\n');
         txt = txt.replace((char) 0x0B, sep);
 

diff --git a/test-data/slideshow/clusterfuzz-testcase-minimized-POIXSLFFuzzer-4986044400861184.pptx b/test-data/slideshow/clusterfuzz-testcase-minimized-POIXSLFFuzzer-4986044400861184.pptx
diff --git a/test-data/spreadsheet/stress.xls b/test-data/spreadsheet/stress.xls