Browse Source

for executing init array functions, use function type with prototype

this is for consistency with the way it's done in in the dynamic
linker, avoiding a deprecated C feature (non-prototype function
types), and improving code generation. GCC unnecessarily uses the
variadic calling convention (e.g. clearing rax on x86_64) when making
a call where the argument types are not known for compatibility with
wrong code which calls variadic functions this way. (C on the other
hand is clear that such calls have undefined behavior.)
Rich Felker 7 years ago
parent
commit
b3516058eb
1 changed files with 1 additions and 1 deletions
  1. 1 1
      src/env/__libc_start_main.c

+ 1 - 1
src/env/__libc_start_main.c

@@ -58,7 +58,7 @@ static void libc_start_init(void)
 	_init();
 	uintptr_t a = (uintptr_t)&__init_array_start;
 	for (; a<(uintptr_t)&__init_array_end; a+=sizeof(void(*)()))
-		(*(void (**)())a)();
+		(*(void (**)(void))a)();
 }
 
 weak_alias(libc_start_init, __libc_start_init);