(nuttx-apps) branch master updated: drivertest_uart: The string was not truncated because one more character was read

[xiaoxiang]

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git


The following commit(s) were added to refs/heads/master by this push:
     new 2e0b57768 drivertest_uart: The string was not truncated because one 
more character was read
2e0b57768 is described below

commit 2e0b5776804547bfa4df5bd5b34627b84c04475d
Author: zhangshoukui <zhangshou...@xiaomi.com>
AuthorDate: Thu Nov 7 21:17:18 2024 +0800

    drivertest_uart: The string was not truncated because one more character 
was read
    
        #0 0x43c7443a in strnlen nuttx/libs/libc/string/lib_strnlen.c:42
        #1 0x43c698aa in vsprintf_internal 
nuttx/libs/libc/stdio/lib_libvsprintf.c:933
        #2 0x43c6ca80 in lib_vsprintf 
nuttx/libs/libc/stdio/lib_libvsprintf.c:1383
        #3 0x4409c0bd in vsnprintf nuttx/libs/libc/stdio/lib_vsnprintf.c:72
        #4 0x47137cd0 in vcmocka_print_error 
apps/testing/cmocka/cmocka/src/cmocka.c:2097
        #5 0x47139573 in cmocka_print_error 
apps/testing/cmocka/cmocka/src/cmocka.c:2422
        #6 0x471376ff in string_equal_display_error 
apps/testing/cmocka/cmocka/src/cmocka.c:1410
        #7 0x471379a0 in _assert_string_equal 
apps/testing/cmocka/cmocka/src/cmocka.c:1952
        #8 0x4433d972 in read_default 
apps/testing/drivertest/drivertest_uart.c:242
        #9 0x4713c6cd in cmocka_run_one_test_or_fixture 
apps/testing/cmocka/cmocka/src/cmocka.c:3029
        #10 0x4713d487 in cmocka_run_one_tests 
apps/testing/cmocka/cmocka/src/cmocka.c:3143
        #11 0x4713f2ca in _cmocka_run_group_tests 
apps/testing/cmocka/cmocka/src/cmocka.c:3294
        #12 0x443444c6 in cmocka_driver_uart_main 
apps/testing/drivertest/drivertest_uart.c:358
        #13 0x4409a472 in nxtask_startup nuttx/libs/libc/sched/task_startup.c:72
        #14 0x43dc92e7 in nxtask_start nuttx/sched/task/task_start.c:116
        #15 0x43e31f00 in pre_start nuttx/arch/sim/src/sim/sim_initialstate.c:52
    
    Signed-off-by: zhangshoukui <zhangshou...@xiaomi.com>
---
 testing/drivertest/drivertest_uart.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testing/drivertest/drivertest_uart.c 
b/testing/drivertest/drivertest_uart.c
index d26d07005..093d002c3 100644
--- a/testing/drivertest/drivertest_uart.c
+++ b/testing/drivertest/drivertest_uart.c
@@ -226,7 +226,7 @@ static void read_default(FAR void **state)
 
   while (cnt < sizeof(DEFAULT_CONTENT) - 1)
     {
-      ssize_t n = read(test_state->fd, buffer + cnt, buffer_size - cnt);
+      ssize_t n = read(test_state->fd, buffer + cnt, buffer_size - cnt - 1);
 
       assert_true(n >= 0);
       if (n == 0)