]> git.ipfire.org Git - thirdparty/glibc.git/commit
ungetc: Fix backup buffer leak on program exit [BZ #27821]
authorSiddhesh Poyarekar <siddhesh@sourceware.org>
Wed, 14 Aug 2024 01:08:49 +0000 (21:08 -0400)
committerSiddhesh Poyarekar <siddhesh@sourceware.org>
Wed, 28 Aug 2024 22:44:07 +0000 (18:44 -0400)
commit5259c8f0521662bcbf4ebb5e1ef803e8acd00cd2
tree9e4e5a0b62fc8b980779d61307c1a6f0f1712934
parent810fa2488c140d7ae3c239fa01ec2dd61199cdc0
ungetc: Fix backup buffer leak on program exit [BZ #27821]

If a file descriptor is left unclosed and is cleaned up by _IO_cleanup
on exit, its backup buffer remains unfreed, registering as a leak in
valgrind.  This is not strictly an issue since (1) the program should
ideally be closing the stream once it's not in use and (2) the program
is about to exit anyway, so keeping the backup buffer around a wee bit
longer isn't a real problem.  Free it anyway to keep valgrind happy
when the streams in question are the standard ones, i.e. stdout, stdin
or stderr.

Also, the _IO_have_backup macro checks for _IO_save_base,
which is a roundabout way to check for a backup buffer instead of
directly looking for _IO_backup_base.  The roundabout check breaks when
the main get area has not been used and user pushes a char into the
backup buffer with ungetc.  Fix this to use the _IO_backup_base
directly.

Signed-off-by: Siddhesh Poyarekar <siddhesh@sourceware.org>
Reviewed-by: Carlos O'Donell <carlos@redhat.com>
(cherry picked from commit 3e1d8d1d1dca24ae90df2ea826a8916896fc7e77)
(cherry picked from commit b9f72bd5de931eac39219018c2fa319a449bb2cf)
libio/genops.c
libio/libioP.h
stdio-common/Makefile
stdio-common/tst-ungetc-leak.c [new file with mode: 0644]