]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
libio: Disable vtable validation in case of interposition [BZ #23313]
authorFlorian Weimer <fweimer@redhat.com>
Tue, 26 Jun 2018 08:24:52 +0000 (10:24 +0200)
committerFlorian Weimer <fweimer@redhat.com>
Thu, 28 Nov 2019 15:21:26 +0000 (16:21 +0100)
(cherry picked from commit c402355dfa7807b8e0adb27c009135a7e2b9f1b0)

ChangeLog
libio/vtables.c

index 9a8840b96819e30d2beac97429206e5bd0ce540d..94788a98c68594db30ca517da90b9a6e4e7c3325 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2018-06-26  Florian Weimer  <fweimer@redhat.com>
+
+       [BZ #23313]
+       * libio/vtables.c (check_stdfiles_vtables): New ELF constructor.
+
 2019-11-01  Dragan Mladjenovic  <dmladjenovic@wavecomp.com>
 
        * sysdeps/unix/sysv/linux/mips/Makefile
index 41b48db98c71c84a2305a7400d56e5e7c6992343..a11226ab17c584d614ef6741e46288787dd6fdaf 100644 (file)
@@ -70,3 +70,19 @@ _IO_vtable_check (void)
 
   __libc_fatal ("Fatal error: glibc detected an invalid stdio handle\n");
 }
+
+/* Some variants of libstdc++ interpose _IO_2_1_stdin_ etc. and
+   install their own vtables directly, without calling _IO_init or
+   other functions.  Detect this by looking at the vtables values
+   during startup, and disable vtable validation in this case.  */
+#ifdef SHARED
+__attribute__ ((constructor))
+static void
+check_stdfiles_vtables (void)
+{
+  if (_IO_2_1_stdin_.vtable != &_IO_file_jumps
+      || _IO_2_1_stdout_.vtable != &_IO_file_jumps
+      || _IO_2_1_stderr_.vtable != &_IO_file_jumps)
+    IO_set_accept_foreign_vtables (&_IO_vtable_check);
+}
+#endif