src: fix leak in ObjectWrap instance set/getters

Fixes: #891 PR-URL: #899 Fixes: #891 Reviewed-By: Gabriel Schulhof <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
nodejs · Feb 12, 2021 · a575a6e · a575a6e
1 parent b6e844e
commit a575a6e
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,