node-api: address coverity warning

[mhdawson]

• address coverity unitialized variable warning

Signed-off-by: Michael Dawson [email protected]

[nodejs-github-bot]

Review requested:

• @nodejs/node-api

[mhdawson]

This is the report. I think technically it may be a false positive, but initializing the variable seems ok as well, so PR does that versus marking as false positive.

/ Create a new napi_env for this specific module.
634  napi_env env = v8impl::NewEnv(context, module_filename);
635
   	3. var_decl: Declaring variable _exports without initializer.
636  napi_value _exports;
637  env->CallIntoModule([&](napi_env env) {
638    _exports = init(env, v8impl::JsValueFromV8LocalValue(exports));
639  });
640
641  // If register function returned a non-null exports object different from
642  // the exports object we passed it, set that as the "exports" property of
643  // the module.
   	
CID 276441 (#1 of 1): Uninitialized pointer read (UNINIT)
4. uninit_use: Using uninitialized value _exports.
644  if (_exports != nullptr &&
645      _exports != v8impl::JsValueFromV8LocalValue(exports)) {
646    napi_value _module = v8impl::JsValueFromV8LocalValue(module);
647    napi_set_named_property(env, _module, "exports", _exports);
648  }

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/48079/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/48094/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/48100/

[mhdawson]

Landed in 0bbef56