From 73914caa9b05cd702232aa719410b44143385061 Mon Sep 17 00:00:00 2001 From: YAMAMOTO Takashi Date: Fri, 1 Dec 2023 19:29:15 +0900 Subject: [PATCH] core/iwasm/interpreter/wasm_loader.c: remove an extra validation (#2845) * Empty names are spec-wise valid. * As we ignore unknown custom sections anyway, it's safe to accept empty names here. * Currently, the problem is not exposed on our CI because the wabt version used there is a bit old. --- core/iwasm/interpreter/wasm_loader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/iwasm/interpreter/wasm_loader.c b/core/iwasm/interpreter/wasm_loader.c index 61e30be7..1286aa0a 100644 --- a/core/iwasm/interpreter/wasm_loader.c +++ b/core/iwasm/interpreter/wasm_loader.c @@ -2708,7 +2708,7 @@ load_user_section(const uint8 *buf, const uint8 *buf_end, WASMModule *module, read_leb_uint32(p, p_end, name_len); - if (name_len == 0 || p + name_len > p_end) { + if (p + name_len > p_end) { set_error_buf(error_buf, error_buf_size, "unexpected end"); return false; }