John Baldwin wrote: > On Tuesday 20 June 2006 16:07, Dmitry Pryanishnikov wrote: >> Hello! >> >> On Tue, 20 Jun 2006, John Baldwin wrote: >>>>> This patch is somewhat redundant. Adding >>>>> >>>>> __weak_reference(dlsym, _dlsym); >>>>> >>>>> is enough. >>>> Indeed, this also works for me. It would be nice to see this trivial >>>> addition committed. >>> I agree. Do you have the __weak_reference() patch handy? >> Sure, here is the diff against HEAD: >> >> --- libexec/rtld-elf/rtld.c.orig Tue Mar 28 10:17:21 2006 >> +++ libexec/rtld-elf/rtld.c Tue Jun 20 22:35:17 2006 >> _at__at_ -137,6 +137,8 _at__at_ >> >> void r_debug_state(struct r_debug*, struct link_map*); >> >> +__weak_reference(dlsym, _dlsym); >> + >> /* >> * Data declarations. >> */ >> >> I'm not sure whether it's style(9)-compliant, it just works. > > Well, this patch was shot down. Would it be possible to have the > pluginwrapper provide the _dlsym weak reference itself? > I already try. That's not possible. I don't know why :/ but i think that's because something with linux emulation/load shared libraries. Regards -- Marcus Alves Grando marcus(at)corp.grupos.com.br | Grupos Internet S/A mnag(at)FreeBSD.org | FreeBSD.orgReceived on Tue Jun 20 2006 - 19:24:11 UTC
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:38:57 UTC