src: fix leak in ObjectWrap instance set/getters

Fixes: nodejs/node-addon-api#891 PR-URL: nodejs/node-addon-api#899 Fixes: nodejs/node-addon-api#891 Reviewed-By: Gabriel Schulhof <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Marlyfleitas · Feb 12, 2021 · a3f8aaa · a3f8aaa
1 parent 1176396
commit a3f8aaa
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/napi-inl.h b/napi-inl.h
@@ -3262,7 +3262,7 @@ inline void InstanceWrap<T>::AttachPropData(napi_env env,
                                        napi_value value,
                                        const napi_property_descriptor* prop) {
   napi_status status;
-  if (prop->method != nullptr && !(prop->attributes & napi_static)) {
+  if (!(prop->attributes & napi_static)) {
     if (prop->method == T::InstanceVoidMethodCallbackWrapper) {
       status = Napi::details::AttachData(env,
                     value,