mirror of
https://git.musl-libc.org/git/musl
synced 2025-01-23 22:52:23 +03:00
fix buffer overflow in mbsrtowcs
issue reported by Michael Forney: "If wn becomes 0 after processing a chunk of 4, mbsrtowcs currently continues on, wrapping wn around to -1, causing the rest of the string to be processed. This resulted in buffer overruns if there was only space in ws for wn wide characters." the original patch submitted added an additional check for !wn after the loop; to avoid extra branching, I instead just changed the wn>=4 check to wn>=5 to ensure that at least one slot remains after the word-at-a-time loop runs. this should not slow down the tail processing on real-world usage, since an extra slot that can't be processed in the word-at-a-time loop is needed for the null termination anyway.
This commit is contained in:
parent
e1f1df9c74
commit
211264e46a
@ -59,7 +59,7 @@ resume0:
|
||||
return wn0;
|
||||
}
|
||||
if (*s-1u < 0x7f && (uintptr_t)s%4 == 0) {
|
||||
while (wn>=4 && !(( *(uint32_t*)s | *(uint32_t*)s-0x01010101) & 0x80808080)) {
|
||||
while (wn>=5 && !(( *(uint32_t*)s | *(uint32_t*)s-0x01010101) & 0x80808080)) {
|
||||
*ws++ = *s++;
|
||||
*ws++ = *s++;
|
||||
*ws++ = *s++;
|
||||
|
Loading…
Reference in New Issue
Block a user