search

toyobayashi / wz

MapleStory wz reader for Node.js and browser.
wz-mu.vercel.app
38 stars 8 forks source link

fix "Source is too large" error #1

Closed ghlin closed 3 years ago

ghlin commented 3 years ago

C# 的MemoryStream()构造函数参数为数组的大小, 而 https://github.com/toyobayashi/wz/blob/96e9fa36bc9d566c2029dec7fac8e587500fd369/src/util/WzMutableKey.ts#L49 subarray的参数为结束元素的索引.

类似地,在 https://github.com/toyobayashi/wz/blob/96e9fa36bc9d566c2029dec7fac8e587500fd369/src/util/WzMutableKey.ts#L58 和 https://github.com/toyobayashi/wz/blob/96e9fa36bc9d566c2029dec7fac8e587500fd369/src/util/WzMutableKey.ts#L63 可能引发RangeError.

toyobayashi commented 3 years ago

LGTM