From b3f85fd2e43b1d69384396a6d99fa2135dbf46e3 Mon Sep 17 00:00:00 2001 From: Florian Weimer Date: Mon, 14 Aug 2017 12:57:50 +0200 Subject: [PATCH] _dl_init: Remove internal_function attribute The function is called from the i386 startup code, which needs minor adjustments due to the changed ABI. --- ChangeLog | 6 ++++++ elf/dl-init.c | 1 - sysdeps/generic/ldsodefs.h | 2 +- sysdeps/i386/dl-machine.h | 11 +++++++---- 4 files changed, 14 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 74ad96d800..ac21eae713 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2017-08-14 Florian Weimer + * elf/dl-init.c (_dl_init): Remove internal_function. + * sysdeps/generic/ldsodefs.h (_dl_init): Likewise. + * sysdeps/i386/dl-machine.h (RTLD_START): Adjust call to _dl_init. + +2017-08-14 Florian Weimer + * elf/rtld.c (_dl_start): Remove internal_function. * sysdeps/i386/dl-machine.h (RTLD_START): Adjust call to _dl_start. diff --git a/elf/dl-init.c b/elf/dl-init.c index 5c5f3de365..fb3734590b 100644 --- a/elf/dl-init.c +++ b/elf/dl-init.c @@ -75,7 +75,6 @@ call_init (struct link_map *l, int argc, char **argv, char **env) void -internal_function _dl_init (struct link_map *main_map, int argc, char **argv, char **env) { ElfW(Dyn) *preinit_array = main_map->l_info[DT_PREINIT_ARRAY]; diff --git a/sysdeps/generic/ldsodefs.h b/sysdeps/generic/ldsodefs.h index 4540096688..49e673dd24 100644 --- a/sysdeps/generic/ldsodefs.h +++ b/sysdeps/generic/ldsodefs.h @@ -964,7 +964,7 @@ extern int _dl_check_map_versions (struct link_map *map, int verbose, /* Initialize the object in SCOPE by calling the constructors with ARGC, ARGV, and ENV as the parameters. */ extern void _dl_init (struct link_map *main_map, int argc, char **argv, - char **env) internal_function attribute_hidden; + char **env) attribute_hidden; /* Call the finalizer functions of all shared objects whose initializer functions have completed. */ diff --git a/sysdeps/i386/dl-machine.h b/sysdeps/i386/dl-machine.h index 924de953b7..2e17eba5c0 100644 --- a/sysdeps/i386/dl-machine.h +++ b/sysdeps/i386/dl-machine.h @@ -177,17 +177,20 @@ _dl_start_user:\n\ # switch stacks if it moves these contents over.\n\ " RTLD_START_SPECIAL_INIT "\n\ # Load the parameters again.\n\ - # (eax, edx, ecx, *--esp) = (_dl_loaded, argc, argv, envp)\n\ + # (eax, edx, ecx, esi) = (_dl_loaded, argc, argv, envp)\n\ movl _rtld_local@GOTOFF(%ebx), %eax\n\ leal 8(%esp,%edx,4), %esi\n\ leal 4(%esp), %ecx\n\ movl %esp, %ebp\n\ # Make sure _dl_init is run with 16 byte aligned stack.\n\ andl $-16, %esp\n\ - pushl %eax\n\ - pushl %eax\n\ + subl $12, %esp\n\ pushl %ebp\n\ + # Arguments for _dl_init.\n\ pushl %esi\n\ + pushl %ecx\n\ + pushl %edx\n\ + pushl %eax\n\ # Clear %ebp, so that even constructors have terminated backchain.\n\ xorl %ebp, %ebp\n\ # Call the function to run the initializers.\n\ @@ -195,7 +198,7 @@ _dl_start_user:\n\ # Pass our finalizer function to the user in %edx, as per ELF ABI.\n\ leal _dl_fini@GOTOFF(%ebx), %edx\n\ # Restore %esp _start expects.\n\ - movl (%esp), %esp\n\ + movl 16(%esp), %esp\n\ # Jump to the user's entry point.\n\ jmp *%edi\n\ .previous\n\ -- 2.11.4.GIT