Skip to content

Commit de79ea7

Browse files
committed
fix(unit-test): libc_builtin_test issues
- uninitialized buffer pointers (crashes) - match integer constant size with printf specifier Signed-off-by: Peter Tatrai <[email protected]>
1 parent 7b724e2 commit de79ea7

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

tests/unit/libc-builtin/libc_builtin_test.cc

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ TEST_F(LibcBuiltinTest, printf)
185185
va_list.add(20); //%zd
186186
va_list.add(20); //%ld
187187

188-
va_list.add(20L); //%jd
188+
va_list.add(intmax_t(20)); //%jd
189189

190190
testing::internal::CaptureStdout();
191191

@@ -323,7 +323,7 @@ TEST_F(LibcBuiltinTest, printf)
323323

324324
TEST_F(LibcBuiltinTest, sprintf)
325325
{
326-
const char *buf;
326+
char buf[200] = {0};
327327
const char *str = "Hello Wrold";
328328
const char *str_sig = "c";
329329
const char *str_f = "20, 3.140000, Hello World";
@@ -508,7 +508,7 @@ TEST_F(LibcBuiltinTest, memcmp)
508508
TEST_F(LibcBuiltinTest, memcpy)
509509
{
510510
const char *src = "Hell World";
511-
char *dest;
511+
char dest[sizeof(src)] = {0};
512512

513513
AppData src_app{ dummy_exec_env.get(), src };
514514
AppData dest_app{ dummy_exec_env.get(), dest };
@@ -535,7 +535,7 @@ TEST_F(LibcBuiltinTest, memcpy)
535535
TEST_F(LibcBuiltinTest, memmove)
536536
{
537537
const char *src = "Hell World";
538-
char *dest;
538+
char dest[sizeof(src)] = {0};
539539

540540
AppData src_app{ dummy_exec_env.get(), src };
541541
AppData dest_app{ dummy_exec_env.get(), dest };
@@ -673,7 +673,7 @@ TEST_F(LibcBuiltinTest, strncmp)
673673
TEST_F(LibcBuiltinTest, strcpy)
674674
{
675675
char *src = (char *)"Hello World!";
676-
char *dest;
676+
char dest[sizeof(src)] = {0};
677677

678678
AppData src_app{ dummy_exec_env.get(), src };
679679
AppData dest_app{ dummy_exec_env.get(), dest };
@@ -696,7 +696,7 @@ TEST_F(LibcBuiltinTest, strcpy)
696696
TEST_F(LibcBuiltinTest, strncpy)
697697
{
698698
char *src = (char *)"Hello World!";
699-
char *dest;
699+
char dest[sizeof(src)] = {0};
700700

701701
AppData src_app{ dummy_exec_env.get(), src };
702702
AppData dest_app{ dummy_exec_env.get(), dest };
@@ -1295,7 +1295,7 @@ TEST_F(LibcBuiltinTest, isalnum)
12951295
TEST_F(LibcBuiltinTest, emscripten_memcpy_big)
12961296
{
12971297
const char *src = "Hell World";
1298-
char *dest;
1298+
char dest[sizeof(src)] = {0};
12991299

13001300
AppData src_app{ dummy_exec_env.get(), src };
13011301
AppData dest_app{ dummy_exec_env.get(), dest };

0 commit comments

Comments
 (0)