Fixed a bug where queue processing could cause a stack overflow. #117

Closed
wants to merge 1 commit into
from

3 participants

@nullobject

Hi there,

I fixed a bug in the async queue processing. The recursion in the #process method has the potential to cause a stack overflow if the queue is large. Using #nextTick resolves this issue.

Thanks for the great library.

@nullobject nullobject Fixed a bug where queue processing could cause a stack overflow.
* Use #nextTick instead of recursively calling #process.
e7ec77c
@MauriceSchoenmakers

had same issue please pull. (note: also if using push with async callback).

@caolan
Owner

Now fixed elsewhere.

@caolan caolan closed this Jan 31, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment