From: Wei Yang <wei.y...@ucloud.cn>

sizeof(BLOCK_NAME_END) is actually 2.  it can work
because there's a space after ,. but It is better to
replace it with strlen so it will work without a space too.

Signed-off-by: Wei Yang <wei.y...@ucloud.cn>
CC: Michael S. Tsirkin <m...@redhat.com>
CC: qemu-devel@nongnu.org
---
 tests/bios-tables-test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c
index 88dbf97853..d1f31b19d3 100644
--- a/tests/bios-tables-test.c
+++ b/tests/bios-tables-test.c
@@ -352,7 +352,7 @@ static GString *normalize_asl(gchar *asl_code)
         block_name = g_strstr_len(asl->str, asl->len, BLOCK_NAME_END);
         g_assert(block_name);
         asl = g_string_erase(asl, 0,
-                             block_name + sizeof(BLOCK_NAME_END) - asl->str);
+                             block_name + strlen(BLOCK_NAME_END) - asl->str);
     }
 
     return asl;
-- 
2.11.0



Reply via email to