From 94702a4309c6b25cf87a18c729fda029f8eb6da1 Mon Sep 17 00:00:00 2001 From: Romain Vimont Date: Sat, 4 Dec 2021 09:10:40 +0100 Subject: [PATCH] Fix memset() size in tests The memset() size was 1 byte too long. It was harmless because the last 'a' was overwritten by '\0`. --- app/tests/test_control_msg_serialize.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/tests/test_control_msg_serialize.c b/app/tests/test_control_msg_serialize.c index 6fed2438..c7f464c8 100644 --- a/app/tests/test_control_msg_serialize.c +++ b/app/tests/test_control_msg_serialize.c @@ -54,7 +54,7 @@ static void test_serialize_inject_text_long(void) { struct control_msg msg; msg.type = CONTROL_MSG_TYPE_INJECT_TEXT; char text[CONTROL_MSG_INJECT_TEXT_MAX_LENGTH + 1]; - memset(text, 'a', sizeof(text)); + memset(text, 'a', CONTROL_MSG_INJECT_TEXT_MAX_LENGTH); text[CONTROL_MSG_INJECT_TEXT_MAX_LENGTH] = '\0'; msg.inject_text.text = text;