From: Yegor Kozlov <yegor@apache.org>
Date: Mon, 19 Dec 2011 09:06:10 +0000 (+0000)
Subject: Bugzilla 52348:  Avoid exception when creating cell style in a workbook that has... 
X-Git-Tag: REL_3_8_FINAL~99
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1b65bb839a01ed8f830a475f8c668ff5b2a51e24;p=poi.git

Bugzilla 52348:  Avoid exception when creating cell style in a workbook that has an empty xf table

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1220659 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/src/documentation/content/xdocs/status.xml b/src/documentation/content/xdocs/status.xml
index f9609297ca..735eedfbae 100644
--- a/src/documentation/content/xdocs/status.xml
+++ b/src/documentation/content/xdocs/status.xml
@@ -34,6 +34,7 @@
 
     <changes>
         <release version="3.8-beta6" date="2012-??-??">
+           <action dev="poi-developers" type="fix">52348 - Avoid exception when creating cell style in a workbook that has an empty xf table</action>
            <action dev="poi-developers" type="fix">52219 - fixed XSSFSimpleShape to set rich text attributes from XSSFRichtextString</action>
            <action dev="poi-developers" type="fix">52314 - enhanced SheetUtil.getColumnWidth</action>
         </release>
diff --git a/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCellStyle.java b/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCellStyle.java
index 51ffe6e06e..d633300ae6 100644
--- a/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCellStyle.java
+++ b/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCellStyle.java
@@ -68,15 +68,15 @@ public class XSSFCellStyle implements CellStyle {
 
     /**
      * Creates a Cell Style from the supplied parts
-     * @param cellXfId The main XF for the cell
-     * @param cellStyleXfId Optional, style xf
+     * @param cellXfId The main XF for the cell. Must be a valid 0-based index into the XF table
+     * @param cellStyleXfId Optional, style xf. A value of <code>-1</code> means no xf.
      * @param stylesSource Styles Source to work off
      */
     public XSSFCellStyle(int cellXfId, int cellStyleXfId, StylesTable stylesSource, ThemesTable theme) {
         _cellXfId = cellXfId;
         _stylesSource = stylesSource;
         _cellXf = stylesSource.getCellXfAt(this._cellXfId);
-        _cellStyleXf = stylesSource.getCellStyleXfAt(cellStyleXfId);
+        _cellStyleXf = cellStyleXfId == -1 ? null : stylesSource.getCellStyleXfAt(cellStyleXfId);
         _theme = theme;
     }
 
diff --git a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFCellStyle.java b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFCellStyle.java
index a9c968c7e3..11b724f4ab 100644
--- a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFCellStyle.java
+++ b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFCellStyle.java
@@ -691,4 +691,19 @@ public class TestXSSFCellStyle extends TestCase {
        assertEquals(fmtClone.getFormat("Test##"), reload.getDataFormat());
        assertFalse(fmtClone.getFormat("Test##") == fmt.getFormat("Test##"));
    }
+
+    /**
+     * Avoid ArrayIndexOutOfBoundsException  when creating cell style
+     * in a workbook that has an empty xf table.
+     */
+    public void testBug52348() {
+        XSSFWorkbook workbook = XSSFTestDataSamples.openSampleWorkbook("52348.xlsx");
+        StylesTable st = workbook.getStylesSource();
+        assertEquals(0, st._getStyleXfsSize());
+        
+        
+        XSSFCellStyle style = workbook.createCellStyle(); // no exception at this point
+        assertNull(style.getStyleXf());
+    }
+
 }
diff --git a/test-data/spreadsheet/52348.xlsx b/test-data/spreadsheet/52348.xlsx
new file mode 100644
index 0000000000..356490d8da
Binary files /dev/null and b/test-data/spreadsheet/52348.xlsx differ