fix: improve Cif destructor by removing redundant argument value freeing and enhance error handling in Pointer creation

Author: DjDeveloperr

NativeScript/ffi/Cif.mm

@@ -191,12 +191,6 @@
     free(argv);
   }
   if (avalues != nullptr) {
-    // Free individual argument values
-    for (int i = 2; i < cif.nargs; i++) {
-      if (avalues[i] != nullptr) {
-        free(avalues[i]);
-      }
-    }
     free(avalues);
   }
   if (shouldFree != nullptr) {

NativeScript/ffi/Interop.mm

@@ -599,8 +599,16 @@ napi_value interop_bufferFromData(napi_env env, napi_callback_info info) {
   bool isInstance = false;
   napi_value jsPointer = get_ref_value(env, bridgeState->pointerClass);
   napi_value result;
-  napi_new_instance(env, jsPointer, 0, nullptr, &result);
+  napi_status status = napi_new_instance(env, jsPointer, 0, nullptr, &result);
+  
+  if (status != napi_ok) {
+    return nullptr;
+  }
+  
   Pointer* ptr = Pointer::unwrap(env, result);
+  if (ptr == nullptr) {
+    return nullptr;
+  }
 
   ptr->data = data;
   return result;

NativeScript/ffi/node_api_util.h

@@ -65,8 +65,8 @@ inline bool napiSupportsThreadsafeFunctions(void* dl) {
   napi_get_last_error_info(env, &error_info);
 
 #define NAPI_THROW_LAST_ERROR \
-  NAPI_ERROR_INFO             \
-  napi_throw_error(env, NULL, error_info->error_message);
+  { NAPI_ERROR_INFO \
+    napi_throw_error(env, NULL, error_info ? error_info->error_message : "Unknown error"); }
 
 #ifndef DEBUG