Fixed Uninitialized string offset for Excel5

ec-web · Mar 8, 2018 · d5646c3 · d5646c3
1 parent 4626c9d
commit d5646c3
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/Parser/Excel5.php b/src/Parser/Excel5.php
@@ -983,6 +983,10 @@ private function readSst() {
 
         // loop through the Unicode strings (16-bit length)
         for ($i = 0; $i < $nm; ++$i) {
+            if (!isset($recordData[$pos+2])) {
+                break;
+            }
+
             // number of characters in the Unicode string
             $numChars = Format::getUInt2d($recordData, $pos);
             $pos += 2;
@@ -1042,7 +1046,7 @@ private function readSst() {
                 $pos = $limitPos;
 
                 // keep reading the characters
-                while ($charsLeft > 0 && isset($recordData[$pos])) {
+                while ($charsLeft > 0) {
                     // look up next limit position, in case the string span more than one continue record
                     foreach ($spliceOffsets as $spliceOffset) {
                         if ($pos < $spliceOffset) {