jeremycx / node-LDAP

LDAP binding for node.js
MIT License
221 stars 43 forks source link

Assertion `loop->watchers[w->fd] == w' failed. #33

Closed emilsedgh closed 11 years ago

emilsedgh commented 11 years ago

node-ldap builds fine using Node 0.10 now, however, this assertion happens. Previous discussion: https://github.com/joyent/libuv/issues/838

I investigated the issue, assertion happens the second time we try to bind using simplebind function of LDAP.cc it happens on LDAPConnection::SetIO

jeremycx commented 11 years ago

Yeah, thanks for the extra legwork - there were a few cases where I was recycling the handle, thought I caught them all, but obviously not. I'll look at this today.

jeremycx commented 11 years ago

Quick question: are you using the current version from github? It looks like I already fixed the issue about 2 weeks ago, unless I'm missing something...

Before I dive in and go crazy, I'm thinking npm might be out of date.

emilsedgh commented 11 years ago

Im using the latest version on npm, which is 1.1.5 Let me try installing the github version.

emilsedgh commented 11 years ago

Its indeed fixed in master. Thanks a lot.