Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Resource leak in binding.cc #200

Open
Sonnerz opened this issue Dec 16, 2020 · 0 comments
Open

Resource leak in binding.cc #200

Sonnerz opened this issue Dec 16, 2020 · 0 comments

Comments

@Sonnerz
Copy link

Sonnerz commented Dec 16, 2020

/src/binding.cc

#129 CompressWorker* worker = new CompressWorker(
#130      input, callback
#131  );
#132
#133  Nan::AsyncQueueWorker(worker);
#134  return;
...
#174  Nan::AsyncQueueWorker(worker);
#176  return;
...
#210 Nan::AsyncQueueWorker(worker);
#212 return;

There is a resource leak here: worker going out of scope leaks the storage it points to

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant