fix(runtime): CreateCBaseObjectArray malloc

entity.go

@@ -23,14 +23,17 @@ func getEntityData(e *C.struct_baseObject) (typ entity.BaseObjectType, ptr unsaf
 }
 
 func newBaseObjectArray[T entity.BaseObject](arr C.struct_array) []T {
-	return cutil.NewArrayFunc[*C.struct_baseObject, T](unsafe.Pointer(arr.ptr), int(arr.size), func(item *C.struct_baseObject) T {
+	entities := cutil.NewArrayFunc[*C.struct_baseObject, T](unsafe.Pointer(arr.ptr), int(arr.size), func(item *C.struct_baseObject) T {
 		v, err := factory.GetBaseObject[T](getEntityData(item))
 		C.free(unsafe.Pointer(item))
 		if err != nil {
 			altlog.Errorln(fmt.Sprintf("[Go] newBaseObjectArray: %s", err.Error()))
 		}
 		return v
 	})
+	// TODO: not sure, but i think we are leaking memory here: arr is allocated in cpp runtime using new CBaseObject*[arr.size];
+	// C.free(arr.ptr) crashes and logs double free :|
+	return entities
 }
 
 func Players() []entity.Player {

runtime/src/GoRuntime.h

@@ -55,10 +55,11 @@ namespace Go {
             CArray arr;
             arr.size = objects.size();
 
-            // TODO: test if this works
+            // TODO: optimize so we dont have to call malloc in for loop
             auto entities = new CBaseObject*[arr.size];
 
             for (uint64_t i = 0; i < arr.size; i++) {
+                entities[i] = (CBaseObject*) malloc(sizeof(CBaseObject));
                 GetCBaseObject(objects[i], entities[i]);
             }