src: fix leak in ObjectWrap instance set/getters

KevinEady · mhdawson · commit a575a6ec6065 · 2021-02-12T16:50:11.000-05:00
Fixes: #891 PR-URL: #899 Fixes: #891 Reviewed-By: Gabriel Schulhof <gabriel.schulhof@intel.com> Reviewed-By: Michael Dawson <midawson@redhat.com>
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,
