about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFlorian Weimer <fweimer@redhat.com>2017-08-14 12:57:50 +0200
committerFlorian Weimer <fweimer@redhat.com>2017-08-14 15:08:48 +0200
commitb3f85fd2e43b1d69384396a6d99fa2135dbf46e3 (patch)
treea2f23e7f5247f1737c29ee348b23df26f92342f8
parent630bf4916f10bdba28a99480ef32170b977ea5fc (diff)
downloadglibc-b3f85fd2e43b1d69384396a6d99fa2135dbf46e3.tar.gz
glibc-b3f85fd2e43b1d69384396a6d99fa2135dbf46e3.tar.xz
glibc-b3f85fd2e43b1d69384396a6d99fa2135dbf46e3.zip
_dl_init: Remove internal_function attribute
The function is called from the i386 startup code, which needs minor
adjustments due to the changed ABI.
-rw-r--r--ChangeLog6
-rw-r--r--elf/dl-init.c1
-rw-r--r--sysdeps/generic/ldsodefs.h2
-rw-r--r--sysdeps/i386/dl-machine.h11
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  <fweimer@redhat.com>
 
+	* 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  <fweimer@redhat.com>
+
 	* 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\