ffi: move DynamicLibrary disposer to native layer

Renegade334 · aduh95 · commit 5d7805e43373 · 2026-05-23T00:05:43.000+02:00
Signed-off-by: Renegade334 <contact.9a5d6388@renegade334.me.uk> PR-URL: #63459 Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Paolo Insogna <paolo@cowtech.it>
diff --git a/lib/ffi.js b/lib/ffi.js
@@ -56,10 +56,6 @@ const {
 
 require('internal/ffi-shared-buffer');
 
-DynamicLibrary.prototype[SymbolDispose] = function() {
-  this.close();
-};
-
 function checkFFIPermission() {
   if (!permission.isEnabled()) {
     return;
diff --git a/src/node_ffi.cc b/src/node_ffi.cc
@@ -1102,6 +1102,7 @@ Local<FunctionTemplate> DynamicLibrary::GetConstructorTemplate(
         static_cast<PropertyAttribute>(ReadOnly));
 
     SetProtoMethod(isolate, tmpl, "close", DynamicLibrary::Close);
+    SetProtoDispose(isolate, tmpl, DynamicLibrary::Close);
     SetProtoMethod(isolate, tmpl, "getFunction", DynamicLibrary::GetFunction);
     SetProtoMethod(isolate, tmpl, "getFunctions", DynamicLibrary::GetFunctions);
     SetProtoMethod(isolate, tmpl, "getSymbol", DynamicLibrary::GetSymbol);
