christoph/wamr
1
Fork 0
Code Releases Activity

Fix potential memory leak in insert_native_symbol (#3712)

Browse Source
This commit is contained in:
Wenyong Huang
2024-08-14 18:11:18 +08:00
committed by GitHub
parent cce48a0943
commit 740f499e9c
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
core/iwasm/compilation/aot_llvm.c
View File

@ -3288,6 +3288,7 @@ insert_native_symbol(AOTCompContext *comp_ctx, const char *symbol, int32 idx)
bh_assert(strlen(symbol) <= sizeof(sym->symbol)); bh_assert(strlen(symbol) <= sizeof(sym->symbol));
ret = snprintf(sym->symbol, sizeof(sym->symbol), "%s", symbol); ret = snprintf(sym->symbol, sizeof(sym->symbol), "%s", symbol);
if (ret < 0 || ret + 1 > (int)sizeof(sym->symbol)) { if (ret < 0 || ret + 1 > (int)sizeof(sym->symbol)) {
wasm_runtime_free(sym);
aot_set_last_error_v("symbol name too long: %s", symbol); aot_set_last_error_v("symbol name too long: %s", symbol);
return false; return false;
} }