Memory leak in decodeBuffer

[chebum]

The following code creates a new function reference and never frees it.

heic2any/src/heic2any.ts

Lines 202 to 212 in 3222e59

	((window as any).__heic2any__worker as Worker).addEventListener(
	"message",
	(message) => {
	if (message.data.id === id) {
	if (message.data.error) {
	return reject(message.data.error);
	}
	return resolve(message.data.imageDataArr);
	}
	}
	);

Subsequent calls to heic2any will create a new instance of the message handler. Both handlers will be called. Converting Nth image will create Nth event handler and all N handler will receive message notifications.

Proposed fix: chebum@c6fab0a

Fixed heic2any.js is available here: https://github.com/chebum/heic2any/tree/master/dist

The fix also includes pull request #27

[Hajzenberg]

Fixed heic2any.js is available here: https://github.com/chebum/heic2any/tree/master/dist

Is there npm of this fork?

[chebum]

Fixed heic2any.js is available here: https://github.com/chebum/heic2any/tree/master/dist

Is there npm of this fork?

You can install it as below:

npm install --save github:chebum/heic2any

[tlloydukdev]

thanks for this awesome contribution. as time has gone on now, I see HEIC files taking up 500MB to convert in the browser Memory tab. the end result is the browser is restarted on iOS/Android. desktop browser copes, I guess it has larger memory resource available