summaryrefslogtreecommitdiff
path: root/tests/Makefile
diff options
context:
space:
mode:
authorjvoisin2026-04-30 18:06:56 +0200
committerjvoisin2026-04-30 18:06:56 +0200
commitd6105aba5fd791e8d3f069e771517cdb947b5604 (patch)
treebbafa423c172a0a0d63edf3aa0e0334fd798f085 /tests/Makefile
parent7fecafe015505c0ebd47780050118ff789a9ae3f (diff)
Fix mbsnrtowcs
mbsnrtowcs writes up to __wn wide characters into wchar_t *__d. The destination capacity is __b / sizeof(wchar_t) wide characters, but the else branch clamps __n (source byte limit) to __b (destination byte size). __wn (the actual output count) is passed through unclamped. Example: __b=8 (dest holds 2 wchar_t), __n=100, __wn=25. The else branch applies (25 <= 100/4), clamps source to 8 bytes, but passes __wn=25 — the function can write 25 wchar_t (100 bytes) into an 8-byte buffer. The first branch is also wrong: it divides __b (bytes) by sizeof(wchar_t) to get wchar_t capacity, which is correct for the destination — but the condition __wn > __n / sizeof(wchar_t) uses integer division that can produce incorrect routing between branches. The fix mirrors the already-correct mbsrtowcs pattern: clamp __wn (the output wide-char count) to the destination's wchar_t capacity, and pass __n (source byte limit) through unchanged.
Diffstat (limited to 'tests/Makefile')
-rw-r--r--tests/Makefile2
1 files changed, 2 insertions, 0 deletions
diff --git a/tests/Makefile b/tests/Makefile
index 9fc8287..6904b2d 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -50,6 +50,8 @@ RUNTIME_TARGETS= \
50 test_mempcpy_static_write \ 50 test_mempcpy_static_write \
51 test_memset_dynamic_write \ 51 test_memset_dynamic_write \
52 test_memset_static_write \ 52 test_memset_static_write \
53 test_mbsnrtowcs_dynamic \
54 test_mbsnrtowcs_static \
53 test_poll_dynamic \ 55 test_poll_dynamic \
54 test_poll_static \ 56 test_poll_static \
55 test_ppoll_dynamic \ 57 test_ppoll_dynamic \