aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFlorian Weimer <fweimer@redhat.com>2024-01-02 14:36:17 +0100
committerAndreas K. Hüttel <dilfridge@gentoo.org>2024-01-31 17:24:51 +0100
commit32f6b0cd3d35a4b8ff091e6b8277ee9aaa593567 (patch)
tree1610bb61cb960376699d9cc7e90bddeddc286d76
parentgetaddrinfo: translate ENOMEM to EAI_MEMORY (bug 31163) (diff)
downloadglibc-32f6b0cd3d35a4b8ff091e6b8277ee9aaa593567.tar.gz
glibc-32f6b0cd3d35a4b8ff091e6b8277ee9aaa593567.tar.bz2
glibc-32f6b0cd3d35a4b8ff091e6b8277ee9aaa593567.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) (cherry picked from commit bd9f194c34333c0148fc0a793b8e68e2399f27cb)
-rw-r--r--NEWS1
-rw-r--r--libio/wfileops.c2
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. */