nan.h error on node v0.10.22

[kuno]

This is error was found in module geoip, the original issue can be found in here.

Quoted from original issue

We are experiencing a new error (very infrequently) since upgrading to node v0.10.22. The following message is displayed and the process exits immediately.

node: /opt/pre-publish/star/node_modules/orion/node_modules/geoip/node_modules/nan/nan.h:809: bool _NanGetExternalParts(v8::Handle<v8::Value>, const char**, size_t*): Assertion `val->IsString()' failed.
Aborted

I suspect it may have something to do with the infamous WalMart memory leak on closed handles fix.

[kkoopa]

This is likely not a bug in NAN.
The assertion tells that _NanGetExternalParts was called with a non-String handle. This in turn means that NanFromV8String was called with a non-String handle. This should only happen if a geoip::*::lookup, geoip::*::lookupSync or geoip::check was called with a non-String argument.

[kuno]

Closing.