Fwd: dlopen uclibc patch

Bernhard Reutner-Fischer rep.dot.nop at gmail.com
Thu Mar 21 22:08:56 UTC 2013


On 21 March 2013 21:02:03 Bernd Schmidt <bernds at codesourcery.com> wrote:
> Another patch from Nathan.  Ok to commit?

That's better, yes. Please install.
Thanks,
>
> Bernd
> ==
>
> This patch fixes some fragileness in dlopen/do_dlopen wrapper & worker
> pair.
> do_dlopen contains __builtin_return_address to determine from whence it was
> called, and uses that to determine which dynamic object's data it should
> use to
> start the search.  (In the bug I was tracking, this related to whether the
> application's RPATH was used or not.)  For that to work, it has to have
> been
> inlined into the wrapper function.
>
> As it happens, it wasn't being inlined.  That's an unfortunate compiler
> behaviour, but it isn't wrong and shouldn't have caused dlopen to fail.
>
> This patch changes things so the wrapper function determines the return
> address,
> and passes it to the worker.  If the worker's inlined, the generated
> code should
> be exactly the same as before.
>
> nathan
> --
> Nathan Sidwell
>
>
>


Sent with AquaMail for Android
http://www.aqua-mail.com




More information about the uClibc mailing list