patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 8 Packages Projects Releases Wiki Activity

tests: sprintf: Avoid buffer overrun

Browse source 
fwrite parameters are "size_t size" and "size_t nmemb",
when writing a string we should set sizeof(char) and len(string).
The test is doing it wrongly and making the function read more
memory than it should.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
This commit is contained in:
Flavio Ceolin 2024-01-17 16:02:54 -08:00 committed by Anas Nashif
parent 4163ea0207
commit 417747886c
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tests/lib/sprintf/src/main.c
View file

@ -909,10 +909,10 @@ ZTEST(sprintf, test_fwrite)
ret = fwrite("This 3", 0, 4, stdout);
zassert_equal(ret, 0, "fwrite failed!");
ret = fwrite("This 3", 4, 4, stdout);
ret = fwrite("This 3", 1, 4, stdout);
zassert_equal(ret, 4, "fwrite failed!");
ret = fwrite("This 3", 4, 4, stdin);
ret = fwrite("This 3", 1, 4, stdin);
zassert_equal(ret, 0, "fwrite failed!");
}