On 21 March 2013 21:02:03 Bernd Schmidt <ber...@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


_______________________________________________
uClibc mailing list
uClibc@uclibc.org
http://lists.busybox.net/mailman/listinfo/uclibc

Reply via email to