diff options
| author | Florian Weimer <fweimer@redhat.com> | 2024-01-02 14:36:17 +0100 |
|---|---|---|
| committer | Florian Weimer <fweimer@redhat.com> | 2024-01-02 14:39:31 +0100 |
| commit | bd9f194c34333c0148fc0a793b8e68e2399f27cb (patch) | |
| tree | fe75a8bb6d891a768e06f35bbffbb39ed18101cf | |
| parent | 9b90e763dbee9899c5521481db6d4e92772ef526 (diff) | |
| download | glibc-bd9f194c34333c0148fc0a793b8e68e2399f27cb.tar.gz glibc-bd9f194c34333c0148fc0a793b8e68e2399f27cb.tar.xz glibc-bd9f194c34333c0148fc0a793b8e68e2399f27cb.zip | |
libio: Check remaining buffer size in _IO_wdo_write (bug 31183)
The multibyte character needs to fit into the remaining buffer space,
not the already-written buffer space. Without the fix, we were never
moving the write pointer from the start of the buffer, always using
the single-character fallback buffer.
Fixes commit 04b76b5aa8b2d1d19066e42dd1 ("Don't error out writing
a multibyte character to an unbuffered stream (bug 17522)").
(cherry picked from commit ecc7c3deb9f347649c2078fcc0f94d4cedf92d60)
| -rw-r--r-- | NEWS | 1 | ||||
| -rw-r--r-- | libio/wfileops.c | 2 |
2 files changed, 2 insertions, 1 deletions
diff --git a/NEWS b/NEWS index 3e7d071557..c0d9993372 100644 --- a/NEWS +++ b/NEWS @@ -55,6 +55,7 @@ The following bugs are resolved with this release: [30745] Slight bug in cache info codes for x86 [30842] Stack read overflow in getaddrinfo in no-aaaa mode (CVE-2023-4527) [30843] potential use-after-free in getcanonname (CVE-2023-4806) + [31183] Wide stream buffer size reduced MB_LEN_MAX bytes after bug 17522 fix [31184] FAIL: elf/tst-tlsgap [31185] Incorrect thread point access in _dl_tlsdesc_undefweak and _dl_tlsdesc_dynamic diff --git a/libio/wfileops.c b/libio/wfileops.c index cecea21c1c..e81924529e 100644 --- a/libio/wfileops.c +++ b/libio/wfileops.c @@ -55,7 +55,7 @@ _IO_wdo_write (FILE *fp, const wchar_t *data, size_t to_do) char mb_buf[MB_LEN_MAX]; char *write_base, *write_ptr, *buf_end; - if (fp->_IO_write_ptr - fp->_IO_write_base < sizeof (mb_buf)) + if (fp->_IO_buf_end - fp->_IO_write_ptr < sizeof (mb_buf)) { /* Make sure we have room for at least one multibyte character. */ |