Re: [PATCH] loader: Check if the preloader overlaps the reserved range on the Mac.

[Sebastian Lackner]

Am Mi., 5. Dez. 2018 um 23:22 Uhr schrieb Ken Thomases :
> @@ -562,6 +598,11 @@ void *wld_start( void *stack, int *is_unix_thread )
>          p++;
>      }
> - LOAD_POSIX_DYLD_FUNC( dlopen );
> - LOAD_POSIX_DYLD_FUNC( dlsym );
> - LOAD_POSIX_DYLD_FUNC( dladdr );
> - LOAD_MACHO_DYLD_FUNC( _dyld_get_image_slide );
> - /* reserve memory that Wine needs */
>    if (reserve) preload_reserve( reserve );
>    for (i = 0; preload_info[i].size; i++)
>
> @@ -576,11 +617,6 @@ void *wld_start( void *stack, int *is_unix_thread )
>      if (!map_region( &builtin_dlls ))
>          builtin_dlls.size = 0;
> - LOAD_POSIX_DYLD_FUNC( dlopen );
> - LOAD_POSIX_DYLD_FUNC( dlsym );
> - LOAD_POSIX_DYLD_FUNC( dladdr );
> - LOAD_MACHO_DYLD_FUNC( _dyld_get_image_slide );
> - /* load the main binary */
>    if (!(mod = pdlopen( argv[1], RTLD_NOW )))
>        fatal_error( "%s: could not load binary\n", argv[1] );
>
> --
> 2.10.2

Hello Ken,

I am not sure anymore if the system libs (dlopen, etc.) are already
loaded by the kernel, or if they are loaded on the first
_dyld_func_lookup() call. Could you maybe check that? In the second
case, changing the order means that there is a higher risk of running
into some address space conflicts.

Best regards,
Sebastian