From f59c3033e678ced61bc8c78e8ac9fbee31ef0210 Mon Sep 17 00:00:00 2001 From: Bohdan Date: Fri, 1 Mar 2019 01:52:49 +0200 Subject: Fix memory leaks (#1925) Fixes #1921 Signed-off-by: Bogdan Vaneev --- src/wasm/wasm.cpp | 12 ++++++++++++ 1 file changed, 12 insertions(+) (limited to 'src/wasm/wasm.cpp') diff --git a/src/wasm/wasm.cpp b/src/wasm/wasm.cpp index 847df5ce6..c12481760 100644 --- a/src/wasm/wasm.cpp +++ b/src/wasm/wasm.cpp @@ -866,6 +866,7 @@ void Module::addExport(Export* curr) { exportsMap[curr->name] = curr; } +// TODO(@warchant): refactor all usages to use variant with unique_ptr void Module::addFunction(Function* curr) { if (!curr->name.is()) { Fatal() << "Module::addFunction: empty name"; @@ -877,6 +878,17 @@ void Module::addFunction(Function* curr) { functionsMap[curr->name] = curr; } +void Module::addFunction(std::unique_ptr curr) { + if (!curr->name.is()) { + Fatal() << "Module::addFunction: empty name"; + } + if (getFunctionOrNull(curr->name)) { + Fatal() << "Module::addFunction: " << curr->name << " already exists"; + } + functionsMap[curr->name] = curr.get(); + functions.push_back(std::move(curr)); +} + void Module::addGlobal(Global* curr) { if (!curr->name.is()) { Fatal() << "Module::addGlobal: empty name"; -- cgit v1.2.3