[PATCH] misc: Fix right-justification in strfmon (bug 34510, CVE-2026-19499)
Florian Weimer
fweimer@redhat.com
Tue Aug 11 14:20:45 GMT 2026
The memmove call did not take into account that __printf_buffer_pad
updated the buffer pointers.
Fixes commit e88b9f0e5cc50cab57a299dc7efe1a4eb385161d
("stdio-common: Convert vfprintf and related functions to buffers"),
which went into glibc 2.37.
---
stdlib/Makefile | 1 +
stdlib/strfmon_l.c | 5 +++--
stdlib/tst-strfmon-bug34510.c | 33 +++++++++++++++++++++++++++++++++
3 files changed, 37 insertions(+), 2 deletions(-)
diff --git a/stdlib/Makefile b/stdlib/Makefile
index addf7dc99f..16948eb512 100644
--- a/stdlib/Makefile
+++ b/stdlib/Makefile
@@ -347,6 +347,7 @@ tests := \
tst-stdc_leading_zeros \
tst-stdc_trailing_ones \
tst-stdc_trailing_zeros \
+ tst-strfmon-bug34510 \
tst-strfmon_l \
tst-strfrom \
tst-strfrom-locale \
diff --git a/stdlib/strfmon_l.c b/stdlib/strfmon_l.c
index f864289480..c39babaeae 100644
--- a/stdlib/strfmon_l.c
+++ b/stdlib/strfmon_l.c
@@ -549,7 +549,8 @@ __vstrfmon_l_buffer (struct __printf_buffer *buf, locale_t loc,
/* Now test whether the output width is filled. */
if (buf->write_ptr - startp < width)
{
- size_t pad_width = width - (buf->write_ptr - startp);
+ size_t written_width = buf->write_ptr - startp;
+ size_t pad_width = width - written_width;
__printf_buffer_pad (buf, ' ', pad_width);
if (__printf_buffer_has_failed (buf))
/* Implies length check. */
@@ -558,7 +559,7 @@ __vstrfmon_l_buffer (struct __printf_buffer *buf, locale_t loc,
Otherwise move the field contents in place. */
if (!left)
{
- memmove (startp + pad_width, startp, buf->write_ptr - startp);
+ memmove (startp + pad_width, startp, written_width);
memset (startp, ' ', pad_width);
}
}
diff --git a/stdlib/tst-strfmon-bug34510.c b/stdlib/tst-strfmon-bug34510.c
new file mode 100644
index 0000000000..b187bde1f4
--- /dev/null
+++ b/stdlib/tst-strfmon-bug34510.c
@@ -0,0 +1,33 @@
+/* Test handling of right-padding in strfmon (bug 34510, CVE-2026-19499).
+ Copyright (C) 2026 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+
+ The GNU C Library is free software; you can redistribute it and/or
+ modify it under the terms of the GNU Lesser General Public
+ License as published by the Free Software Foundation; either
+ version 2.1 of the License, or (at your option) any later version.
+
+ The GNU C Library is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ Lesser General Public License for more details.
+
+ You should have received a copy of the GNU Lesser General Public
+ License along with the GNU C Library; if not, see
+ <https://www.gnu.org/licenses/>. */
+
+#include <monetary.h>
+#include <errno.h>
+#include <support/check.h>
+#include <support/next_to_fault.h>
+
+static int
+do_test (void)
+{
+ struct support_next_to_fault ntf = support_next_to_fault_allocate (100);
+ TEST_COMPARE (strfmon (ntf.buffer, ntf.length, "%100n", 1.23), -1);
+ TEST_COMPARE (errno, E2BIG);
+ return 0;
+}
+
+#include <support/test-driver.c>
base-commit: 25a8ce5d563b98ef5a35a5207bdef927e0f0969d
More information about the Libc-alpha
mailing list