dont use smart pointers to transfer vector data

since `wasm_functype_new` will transfer data ownerships of both `params`
and `resutls` to the new `wasm_functype_t`, we should not
- delete `wasm_valtype_t` of `wasm_valtype_vec_t`
- use `wasm_functype_delete` to release members of `wasm_functype_t`

a good example will be found here:
[multi.c](https://github.com/WebAssembly/wasm-c-api/blob/master/example/multi.c)

another thing is the runtime() should return a string as same as the one in envoy.yaml

update wamr code release to 0.1-beta

Signed-off-by: liam <[email protected]>

Author: lum1n0us

bazel/external/wamr.BUILD

@@ -13,11 +13,11 @@ filegroup(
 cmake(
     name = "libiwasm",
     cache_entries = {
-        "CMAKE_EXPORT_COMPILE_COMMANDS": "On",
         "WAMR_BUILD_AOT": "0",
         "WAMR_BUILD_SIMD": "0",
+        "WAMR_BUILD_MULTI_MODULE": "1",
+        "WAMR_BUILD_LIBC_WASI": "0",
     },
     lib_source = ":srcs",
     out_shared_libs = ["libiwasm.so"],
-    working_directory = "product-mini/platforms/linux"
 )

bazel/repositories.bzl

@@ -39,8 +39,9 @@ def proxy_wasm_cpp_host_repositories():
     http_archive(
         name = "wamr",
         build_file = "@proxy_wasm_cpp_host//bazel/external:wamr.BUILD",
-        sha256 = "f819b9ec866a12086233e578044dd8297e6be86f0f17807b16124f78a3652d4f",
-        url = "https://github.com/lum1n0us/wasm-micro-runtime/releases/download/WAMR-01-29-2021/source.zip",
+        sha256 = "3f9c993cf81a573cd733b7d97fa15ed757351881e502aad22ff873371ee55202",
+        strip_prefix = "wasm-micro-runtime-0.1-beta",
+        url = "https://github.com/lum1n0us/wasm-micro-runtime/archive/refs/tags/v0.1-beta.tar.gz",
     )
 
     http_archive(

src/wamr/wamr.cc

@@ -56,11 +56,11 @@ class Wamr : public WasmVm {
 public:
   Wamr() {}
 
-  std::string_view runtime() override { return "wamr"; }
+  std::string_view runtime() override { return "envoy.wasm.runtime.wamr"; }
   std::string_view getPrecompiledSectionName() override { return ""; }
 
   Cloneable cloneable() override { return Cloneable::NotCloneable; }
-  std::unique_ptr<WasmVm> clone() override { return nullptr; };
+  std::unique_ptr<WasmVm> clone() override { return nullptr; }
 
   AbiVersion getAbiVersion() override;
 
@@ -433,9 +433,6 @@ void convertArgsTupleToValTypesImpl(wasm_valtype_vec_t *types, std::index_sequen
   auto ps = std::array<wasm_valtype_t *, std::tuple_size<T>::value>{
       convertArgToValTypePtr<typename std::tuple_element<I, T>::type>()...};
   wasm_valtype_vec_new(types, size, ps.data());
-  for (auto i = ps.begin(); i < ps.end(); i++) { // TODO(mathetake): better way to handle?
-    wasm_valtype_delete(*i);
-  }
 }
 
 template <typename T, typename Is = std::make_index_sequence<std::tuple_size<T>::value>>
@@ -444,17 +441,17 @@ void convertArgsTupleToValTypes(wasm_valtype_vec_t *types) {
 }
 
 template <typename R, typename T> WasmFunctypePtr newWasmNewFuncType() {
-  WasmValtypeVec params, results;
-  convertArgsTupleToValTypes<T>(params.get());
-  convertArgsTupleToValTypes<std::tuple<R>>(results.get());
-  return wasm_functype_new(params.get(), results.get());
+  wasm_valtype_vec_t params, results;
+  convertArgsTupleToValTypes<T>(&params);
+  convertArgsTupleToValTypes<std::tuple<R>>(&results);
+  return wasm_functype_new(&params, &results);
 }
 
 template <typename T> WasmFunctypePtr newWasmNewFuncType() {
-  WasmValtypeVec params, results;
-  convertArgsTupleToValTypes<T>(params.get());
-  convertArgsTupleToValTypes<std::tuple<>>(results.get());
-  return wasm_functype_new(params.get(), results.get());
+  wasm_valtype_vec_t params, results;
+  convertArgsTupleToValTypes<T>(&params);
+  convertArgsTupleToValTypes<std::tuple<>>(&results);
+  return wasm_functype_new(&params, &results);
 }
 
 template <typename... Args>