how to fix memory leak in setTimeout closure?

Question

Can someone please explain how to fix this memory leak in nodejs:

var foo = function (data, cb) {
    cb(data);
};

setInterval(function() {
    foo('asdf', function(data) {
        console.log(data);
    });
}, 500);

because the memory is increasing. Also I tried to comment the console.log(), the memory is increasing a lot slower but still increasing. — Keloo, May 19 '14 at 13:24

score 3 · Accepted Answer · edited May 23 '17 at 12:06

3

Actually I agree with @vkurchatkin, there is no memory leak. I tried to run this code with small variations and my results: node processes had no memory incrementation. But my terminal emulator rxvt does. You should check yours.

Usefull topic: How to prevent memory leaks in node.js?

edited May 23 '17 at 12:06

Community

1
1

answered May 19 '14 at 13:44

zarkone

1,335
10
16

Agreed. Running the code in the question for several minutes showed no memory leak on mine... – Matthew Bakaitis May 19 '14 at 13:50
I tested it on a linux server. `top` terminal, and there is no memory increasing. I was confused by windows terminal ). – Keloo May 19 '14 at 14:07

how to fix memory leak in setTimeout closure?

1 Answers1