lwille/node-gphoto2

Memory leak

auterium opened this issue · 4 comments

auterium commented

First of all, thanks for this awesome library, it's been very helpful to me.

I need to be able to listen for disconnection/reconnection of the camera. To do so, I execute multiple times the list() method. After many tests, I found out that calling this method ads ~5MB of ram usage that never gets cleaned up. Here's some code to test the leakage:

var GPhoto2 = require("gphoto2").GPhoto2;
var os = require("os");
var gp = new GPhoto2();

setIntervar(function() {
  gp.list(function(list) {
    console.log(os.freemem());
  });
}, 1000);

Thanks

Sija commented

@lwille Issue is still there, isn't it?

lwille commented

uh oh, I thought that was fixed in your commit.

Sija commented

@lwille nope, this fix was too naïve so I removed it after all.

auterium commented

Thanks guys, I've tested latest version (0.3.0) with node v8.11.1 and it's no longer leaking.

  • 🎉1