You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ce...@apache.org on 2026/01/24 13:47:51 UTC

(poi) 03/05: Avoid NPE when retrieving sheets by name with invalid name

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

centic pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/poi.git

commit 04f4c1fa7424f12b12f1e513950f9e7fa13c625d
Author: Dominik Stadler <do...@gmx.at>
AuthorDate: Fri Jan 23 07:13:50 2026 +0100

    Avoid NPE when retrieving sheets by name with invalid name
---
 .../src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java     | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java b/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
index a62bbfb739..a063b4d8a9 100644
--- a/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
+++ b/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
@@ -1218,6 +1218,10 @@ public class XSSFWorkbook extends POIXMLDocument implements Workbook, Date1904Su
      */
     @Override
     public XSSFSheet getSheet(String name) {
+        if (name == null) {
+            throw new IllegalArgumentException("Encountered an empty name when looking up sheets by name");
+        }
+
         for (XSSFSheet sheet : sheets) {
             if (name.equalsIgnoreCase(sheet.getSheetName())) {
                 return sheet;


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org