inflateRaw on nodejs : RangeError: Offset/length out of range about pako HOT 10 CLOSED

Could you check it first with pako 0.2 ? If possible - write a test example directly for pako (add fixture + extend one of tests in your fork). It already has rawDeflate test https://github.com/nodeca/pako/blob/master/test/inflate.js#L20-L22 , probably it will be enougth to just add original (not compressed) file into test/fixtures/samples folder.

from pako.

I get the same error with pako's master (just tested with pako 0.2, same result).
I don't have time to create the test right now but I will tonight !

from pako.

Hm... i've downloaded original apachepoi_46535.xlsx, placed to fixtures, but all tests, involved it, passed. Do you operate with arrays or strings? Probably, something wrong with strings format? Inflate consider input string as binary, not utf8-encoded.

Will be waiting for reduced example. Code in test folder is quite simple, but if you need any help about using sources - let me know.

from pako.

Now a comment without time pressure !

I added failing unit tests to show to issue. The files sheet2.compressed to sheet4.compressed come from the file apachepoi_46535.xlsx (entry xl/worksheets/sheet2.xml to xl/worksheets/sheet4.xml). They are the compressed content of the entries. I've extracted them with JSZip (in nodejs, manipulating Buffers). The files sheet{2..4}.data are the extracted files (with unzip).

from pako.

Seems i didn't understood the root of problem before. Does pako fails to unpack content, prepared in another zipper (not js-zip)?

from pako.

Sorry if I wasn't clear.
The xlsx file comes from the tests of Apache POI, here. I don't know which tool generated it but I'm sure it's not JSZip :)

When unpacking the content with pako and inflateRaw, I get an error. When I use unzip (to extract or to test the xlsx file) everything is fine.

I tested other xlsx files but that's the only file that trigger the bug.

from pako.

Ok, now i understand why you provided blobs in first message. Thanks.

/cc @andr83

from pako.

@dduponchel pako had bug - blocks with fixed dictionaries caused overwrite of dynamic tables.

https://github.com/nodeca/pako/tree/issue-22 - this branch contains fix, but we not yet decided about code clearness. Will finish in 1-2 days with release.

from pako.

Fixed. 0.2.1 released.

from pako.

Thanks !

from pako.