about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFlorian Weimer <fweimer@redhat.com>2020-05-18 15:21:04 +0200
committerFlorian Weimer <fweimer@redhat.com>2020-05-18 15:30:09 +0200
commit7b5bfe77836442b9aeb75cc520f0d1eb7f82be67 (patch)
tree042242b904ee724c8beab0f6f0a816f095fcd09f
parentaa70d0563256b8ea053203177f756bca33b5cf37 (diff)
downloadglibc-7b5bfe77836442b9aeb75cc520f0d1eb7f82be67.tar.gz
glibc-7b5bfe77836442b9aeb75cc520f0d1eb7f82be67.tar.xz
glibc-7b5bfe77836442b9aeb75cc520f0d1eb7f82be67.zip
elf: Assert that objects are relocated before their constructors run
If we try to run constructors before relocation, this is always
a dynamic linker bug.  An assert is easier to notice than a call
via an invalid function pointer (which may not even produce a valid
call stack).

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
-rw-r--r--elf/dl-init.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/elf/dl-init.c b/elf/dl-init.c
index 1234611a1c..518824e8a5 100644
--- a/elf/dl-init.c
+++ b/elf/dl-init.c
@@ -16,6 +16,7 @@
    License along with the GNU C Library; if not, see
    <https://www.gnu.org/licenses/>.  */
 
+#include <assert.h>
 #include <stddef.h>
 #include <ldsodefs.h>
 #include <elf-initfini.h>
@@ -28,6 +29,11 @@ typedef void (*init_t) (int, char **, char **);
 static void
 call_init (struct link_map *l, int argc, char **argv, char **env)
 {
+  /* If the object has not been relocated, this is a bug.  The
+     function pointers are invalid in this case.  (Executables do not
+     need relocation, and neither do proxy objects.)  */
+  assert (l->l_real->l_relocated || l->l_real->l_type == lt_executable);
+
   if (l->l_init_called)
     /* This object is all done.  */
     return;