Move error handling to Inflate class

Every call wants this check so this should be done inside the class.
HemaZ · Feb 17, 2020 · f6669ff · f6669ff
1 parent fe5aa64
commit f6669ff
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/core/decoders/tight.js b/core/decoders/tight.js
@@ -161,9 +161,6 @@ export default class TightDecoder {
             }
 
             data = this._zlibs[streamId].inflate(data, uncompressedSize);
-            if (data.length != uncompressedSize) {
-                throw new Error("Incomplete zlib block");
-            }
         }
 
         display.blitRgbImage(x, y, width, height, data, 0, false);
@@ -209,9 +206,6 @@ export default class TightDecoder {
             }
 
             data = this._zlibs[streamId].inflate(data, uncompressedSize);
-            if (data.length != uncompressedSize) {
-                throw new Error("Incomplete zlib block");
-            }
         }
 
         // Convert indexed (palette based) image data to RGB

diff --git a/core/inflator.js b/core/inflator.js
@@ -37,6 +37,10 @@ export default class Inflate {
 
         inflate(this.strm, 0); // Flush argument not used.
 
+        if (this.strm.next_out != expected) {
+            throw new Error("Incomplete zlib block");
+        }
+
         return new Uint8Array(this.strm.output.buffer, 0, this.strm.next_out);
     }