1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

module: fix pointer reference to out-of-scope variable

Browse Source 
Reported by Tom Hughes.
This commit is contained in:
Ben Noordhuis 2011-08-14 01:25:39 +02:00
parent 59b815b097
commit eaac88163a
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/node.cc
View File

@ -1690,6 +1690,7 @@ typedef void (*extInit)(Handle<Object> exports);
// DLOpen is node.dlopen(). Used to load 'module.node' dynamically shared // DLOpen is node.dlopen(). Used to load 'module.node' dynamically shared
// objects. // objects.
Handle<Value> DLOpen(const v8::Arguments& args) { Handle<Value> DLOpen(const v8::Arguments& args) {
node_module_struct compat_mod;
HandleScope scope; HandleScope scope;
if (args.Length() < 2) return Undefined(); if (args.Length() < 2) return Undefined();
@ -1732,10 +1733,13 @@ Handle<Value> DLOpen(const v8::Arguments& args) {
// Get the init() function from the dynamically shared object. // Get the init() function from the dynamically shared object.
node_module_struct *mod = static_cast<node_module_struct *>(dlsym(handle, symstr)); node_module_struct *mod = static_cast<node_module_struct *>(dlsym(handle, symstr));
free(symstr); free(symstr);
symstr = NULL;
// Error out if not found. // Error out if not found.
if (mod == NULL) { if (mod == NULL) {
/* Start Compatibility hack: Remove once everyone is using NODE_MODULE macro */ /* Start Compatibility hack: Remove once everyone is using NODE_MODULE macro */
node_module_struct compat_mod; memset(&compat_mod, 0, sizeof compat_mod);
mod = &compat_mod; mod = &compat_mod;
mod->version = NODE_MODULE_VERSION; mod->version = NODE_MODULE_VERSION;