commit 582153b68e076c0f2ae71392f7553181c466e849
author Albert Astals Cid <aacid@kde.org> Mon Jan 08 23:34:13 2018 +0100
committer Albert Astals Cid <aacid@kde.org> Mon Jan 08 23:34:13 2018 +0100
tree ea97aacca6821e1120bb785e6251d4ebeda87bf4
parent dea6e03352084361e7c89cff7adaa53110c4e91a

Lexer: Check curStr is actually a Stream before doing Stream operations

Bug #104518

poppler/Lexer.cc

diff --git a/poppler/Lexer.cc b/poppler/Lexer.cc
index 70e0053..3089da3 100644
--- a/poppler/Lexer.cc
+++ b/poppler/Lexer.cc
@@ -97,12 +97,14 @@
   strPtr = 0;
   if (streams->getLength() > 0) {
     curStr = streams->get(strPtr);
-    curStr.streamReset();
+    if (curStr.isStream()) {
+      curStr.streamReset();
+    }
   }
 }
 
 Lexer::~Lexer() {
-  if (!curStr.isNone()) {
+  if (curStr.isStream()) {
     curStr.streamClose();
   }
   if (freeArray) {
@@ -120,7 +122,7 @@
   }
 
   c = EOF;
-  while (!curStr.isNone() && (c = curStr.streamGetChar()) == EOF) {
+  while (curStr.isStream() && (c = curStr.streamGetChar()) == EOF) {
     if (comesFromLook == gTrue) {
       return EOF;
     } else {
@@ -128,8 +130,10 @@
       curStr = Object();
       ++strPtr;
       if (strPtr < streams->getLength()) {
-        curStr = streams->get(strPtr);
-        curStr.streamReset();
+	curStr = streams->get(strPtr);
+	if (curStr.isStream()) {
+	  curStr.streamReset();
+	}
       }
     }
   }