fix: don't try to free blocks that not owned by the BlockWrapper

Author: rigor789

NativeScript/runtime/DataWrapper.h

@@ -556,25 +556,32 @@ class AnonymousFunctionWrapper: public BaseDataWrapper {
 
 class BlockWrapper: public BaseDataWrapper {
 public:
-    BlockWrapper(void* block, const TypeEncoding* typeEncoding)
+    BlockWrapper(void* block, const TypeEncoding* typeEncoding, bool ownsBlock)
         : block_(block),
-          typeEncoding_(typeEncoding) {
+          typeEncoding_(typeEncoding),
+          ownsBlock_(ownsBlock) {
     }
-
+    
     const WrapperType Type() {
         return WrapperType::Block;
     }
 
     void* Block() {
         return this->block_;
     }
-
+    
     const TypeEncoding* Encodings() {
         return this->typeEncoding_;
     }
+    
+    bool OwnsBlock() {
+        return this->ownsBlock_;
+    }
+
 private:
     void* block_;
     const TypeEncoding* typeEncoding_;
+    bool ownsBlock_;
 };
 
 class FunctionReferenceTypeWrapper: public BaseDataWrapper {

NativeScript/runtime/Interop.mm

@@ -431,7 +431,7 @@ inline bool isBool() {
             MethodCallbackWrapper* userData = new MethodCallbackWrapper(isolate, poCallback, 1, argsCount, blockTypeEncoding);
             blockPtr = Interop::CreateBlock(1, argsCount, blockTypeEncoding, ArgConverter::MethodCallback, userData);
 
-            BlockWrapper* wrapper = new BlockWrapper((void*)blockPtr, blockTypeEncoding);
+            BlockWrapper* wrapper = new BlockWrapper((void*)blockPtr, blockTypeEncoding, false);
             tns::SetValue(isolate, arg.As<v8::Function>(), wrapper);
         }
 
@@ -947,7 +947,7 @@ inline bool isBool() {
             return callback;
         }
 
-        BlockWrapper* blockWrapper = new BlockWrapper(block, typeEncoding);
+        BlockWrapper* blockWrapper = new BlockWrapper(block, typeEncoding, true);
         Local<External> ext = External::New(isolate, blockWrapper);
         Local<v8::Function> callback;
 

NativeScript/runtime/ObjectManager.mm

@@ -103,7 +103,9 @@
         }
         case WrapperType::Block: {
             BlockWrapper* blockWrapper = static_cast<BlockWrapper*>(wrapper);
-            std::free(blockWrapper->Block());
+            if(!blockWrapper->OwnsBlock()) {
+                std::free(blockWrapper->Block());
+            }
             break;
         }
         case WrapperType::Reference: {