Bug 57456: Fix reading XLS with empty SSTRecord where Excel puts some random number in unique-strings-count-field

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

1 files changed, 7 insertions, 0 deletions

diff --git a/src/java/org/apache/poi/hssf/record/SSTRecord.java b/src/java/org/apache/poi/hssf/record/SSTRecord.java
index aadd58619b..abbf909681 100644
--- a/src/java/org/apache/poi/hssf/record/SSTRecord.java
+++ b/src/java/org/apache/poi/hssf/record/SSTRecord.java
@@ -243,7 +243,14 @@ public final class SSTRecord extends ContinuableRecord {
         field_1_num_strings = in.readInt();
         field_2_num_unique_strings = in.readInt();
         field_3_strings = new IntMapper<UnicodeString>();
+        
         deserializer = new SSTDeserializer(field_3_strings);
+        // Bug 57456: some Excel Sheets send 0 as field=1, but have some random number in field_2,
+        // we should not try to read the strings in this case.
+        if(field_1_num_strings == 0) {
+            field_2_num_unique_strings = 0;
+            return;
+        }
         deserializer.manufactureStrings( field_2_num_unique_strings, in );
     }