mii/wasmer
1
0
Fork 0
mirror of https://github.com/mii443/wasmer.git synced 2025-12-07 13:18:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix wasm_instance_exports exports memory leak in C API tests

Browse Source
This commit is contained in:
Mark McCaskey
2021-03-18 08:30:11 -07:00
parent 1a56ff2cc8
commit 5650e12f40
5 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
lib/c-api/src/wasm_c_api/externals/global.rs vendored
View File

@@ -145,6 +145,7 @@ mod tests {
wasm_instance_delete(instance);
wasm_byte_vec_delete(&wasm_bytes);
wasm_byte_vec_delete(&wat);
wasm_extern_vec_delete(&exports);
wasm_store_delete(store);
wasm_engine_delete(engine);

1
lib/c-api/src/wasm_c_api/externals/mod.rs vendored
View File

@@ -179,6 +179,7 @@ mod tests {
assert(wasm_extern_kind(function_copy) == WASM_EXTERN_FUNC);
wasm_extern_delete(function_copy);
wasm_extern_vec_delete(&exports);
wasm_instance_delete(instance);
wasm_module_delete(module);
wasm_byte_vec_delete(&wasm);

2
lib/c-api/src/wasm_c_api/instance.rs
View File

@@ -154,6 +154,7 @@ pub unsafe extern "C" fn wasm_instance_delete(_instance: Option<Box<wasm_instanc
/// assert(wasm_extern_kind(exports.data[3]) == WASM_EXTERN_MEMORY);
///
/// // Free everything.
/// wasm_extern_vec_delete(&exports);
/// wasm_instance_delete(instance);
/// wasm_module_delete(module);
/// wasm_byte_vec_delete(&wasm);
@@ -290,6 +291,7 @@ mod tests {
assert(results[0].of.i32 == 2);
// Free everything.
wasm_extern_vec_delete(&exports);
wasm_instance_delete(instance);
wasm_func_delete(sum_function);
wasm_functype_delete(sum_type);

3
lib/c-api/src/wasm_c_api/unstable/middlewares/metering.rs
View File

@@ -117,7 +117,8 @@
//! // There is 0 point left… they are exhausted.
//! assert(wasmer_metering_points_are_exhausted(instance) == true);
//! }
//!
//!
//! wasm_extern_vec_delete(&exports);
//! wasm_instance_delete(instance);
//! wasm_module_delete(module);
//! wasm_store_delete(store);