rtld: Fix ELF filter implementation
authorJohn Marino <draco@marino.st>
Sun, 25 Mar 2012 10:28:13 +0000 (12:28 +0200)
committerJohn Marino <draco@marino.st>
Sun, 25 Mar 2012 11:27:31 +0000 (13:27 +0200)
Vertatim from FreeBSD SVN 233231 (20 March 2012):

Do not relocate twice an object which happens to be needed by loaded
binary (or dso) and some filtee opened due to symbol resolution when
relocating need objects.  Record the state of the relocation
processing in Obj_Entry and short-circuit relocate_objects() if
current object already processed.

Do not call constructors for filtees loaded during the early
relocation processing before image is initialized enough to run
user-provided code.  Filtees are loaded using dlopen_object(), which
normally performs relocation and initialization.  If filtee is
lazy-loaded during the relocation of dso needed by the main object,
dlopen_object() runs too earlier, when most runtime services are not
yet ready.

Postpone the constructors call to the time when main binary and
depended libraries constructors are run, passing the new flag
RTLD_LO_EARLY to dlopen_object().  Symbol lookups callers inform
symlook_* functions about early stage of initialization with
SYMLOOK_EARLY.  Pass flags through all functions participating in
object relocation.

Use the opportunity and fix flags argument to find_symdef() in
arch-specific reloc.c to use proper name SYMLOOK_IN_PLT instead of
true, which happen to have the same numeric value.

libexec/rtld-elf/i386/reloc.c
libexec/rtld-elf/rtld.c
libexec/rtld-elf/rtld.h
libexec/rtld-elf/x86_64/reloc.c

index 30d9c39..159f77d 100644 (file)
@@ -84,6 +84,7 @@ do_copy_relocations(Obj_Entry *dstobj)
            size = dstsym->st_size;
            symlook_init(&req, name);
            req.ventry = fetch_ventry(dstobj, ELF_R_SYM(rel->r_info));
+           req.flags = SYMLOOK_EARLY;
 
            for (srcobj = dstobj->next;  srcobj != NULL;  srcobj = srcobj->next) {
                res = symlook_obj(&req, srcobj);
@@ -120,7 +121,8 @@ init_pltgot(Obj_Entry *obj)
 
 /* Process the non-PLT relocations. */
 int
-reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
+reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, int flags,
+    RtldLockState *lockstate)
 {
        const Elf_Rel *rellim;
        const Elf_Rel *rel;
@@ -152,7 +154,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -171,7 +173,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -201,7 +203,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -221,7 +223,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    Elf_Addr add;
 
                    def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -254,7 +256,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -268,7 +270,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -287,7 +289,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
 done:
        if (cache != NULL)
            free(cache);
-       return(r);
+       return (r);
 }
 
 /* Process the PLT relocations. */
@@ -323,7 +325,7 @@ reloc_plt(Obj_Entry *obj)
 
 /* Relocate the jump slots in an object. */
 int
-reloc_jmpslots(Obj_Entry *obj, RtldLockState *lockstate)
+reloc_jmpslots(Obj_Entry *obj, int flags, RtldLockState *lockstate)
 {
     const Elf_Rel *rellim;
     const Elf_Rel *rel;
@@ -339,8 +341,8 @@ reloc_jmpslots(Obj_Entry *obj, RtldLockState *lockstate)
        switch (ELF_R_TYPE(rel->r_info)) {
        case R_386_JMP_SLOT:
          where = (Elf_Addr *)(obj->relocbase + rel->r_offset);
-         def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj, true, NULL,
-             lockstate);
+         def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj,
+               SYMLOOK_IN_PLT | flags, NULL, lockstate);
          if (def == NULL)
              return (-1);
          if (ELF_ST_TYPE(def->st_info) == STT_GNU_IFUNC) {
@@ -391,7 +393,7 @@ reloc_iresolve(Obj_Entry *obj, RtldLockState *lockstate)
 }
 
 int
-reloc_gnu_ifunc(Obj_Entry *obj, RtldLockState *lockstate)
+reloc_gnu_ifunc(Obj_Entry *obj, int flags, RtldLockState *lockstate)
 {
     const Elf_Rel *rellim;
     const Elf_Rel *rel;
@@ -407,8 +409,8 @@ reloc_gnu_ifunc(Obj_Entry *obj, RtldLockState *lockstate)
        switch (ELF_R_TYPE(rel->r_info)) {
        case R_386_JMP_SLOT:
          where = (Elf_Addr *)(obj->relocbase + rel->r_offset);
-         def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj, true, NULL,
-             lockstate);
+         def = find_symdef(ELF_R_SYM(rel->r_info), obj, &defobj,
+               SYMLOOK_IN_PLT | flags, NULL, lockstate);
          if (def == NULL)
              return (-1);
          if (ELF_ST_TYPE(def->st_info) != STT_GNU_IFUNC)
index d08b62c..c70a7d9 100644 (file)
@@ -120,9 +120,10 @@ static void objlist_push_head(Objlist *, Obj_Entry *);
 static void objlist_push_tail(Objlist *, Obj_Entry *);
 static void objlist_remove(Objlist *, Obj_Entry *);
 static void *path_enumerate(const char *, path_enum_proc, void *);
-static int relocate_objects(Obj_Entry *, bool, Obj_Entry *, RtldLockState *);
+static int relocate_objects(Obj_Entry *, bool, Obj_Entry *, int,
+    RtldLockState *);
 static int resolve_objects_ifunc(Obj_Entry *first, bool bind_now,
-    RtldLockState *lockstate);
+    int flags, RtldLockState *lockstate);
 static int rtld_dirname(const char *, char *);
 static int rtld_dirname_abs(const char *, char *);
 static void rtld_exit(void);
@@ -628,7 +629,8 @@ resident_skip1:
      */
 
     if (relocate_objects(obj_main,
-      ld_bind_now != NULL && *ld_bind_now != '\0', &obj_rtld, NULL) == -1)
+      ld_bind_now != NULL && *ld_bind_now != '\0',
+      &obj_rtld, SYMLOOK_EARLY, NULL) == -1)
        die();
 
     dbg("doing copy relocations");
@@ -677,7 +679,8 @@ resident_skip2:
 
     dbg("resolving ifuncs");
     if (resolve_objects_ifunc(obj_main,
-      ld_bind_now != NULL && *ld_bind_now != '\0', NULL) == -1)
+      ld_bind_now != NULL && *ld_bind_now != '\0', SYMLOOK_EARLY,
+      NULL) == -1)
        die();
 
     /*
@@ -1810,7 +1813,7 @@ init_rtld(caddr_t mapbase, Elf_Auxinfo **aux_info)
         * that symbols can be found.
         */
 
-       relocate_objects(&objtmp, true, &objtmp, NULL);
+       relocate_objects(&objtmp, true, &objtmp, 0, NULL);
     }
 
     /* Initialize the object list. */
@@ -1863,6 +1866,7 @@ initlist_add_neededs(Needed_Entry *needed, Objlist *list)
 static void
 initlist_add_objects(Obj_Entry *obj, Obj_Entry **tail, Objlist *list)
 {
+
     if (obj->init_scanned || obj->init_done)
        return;
     obj->init_scanned = true;
@@ -1874,6 +1878,10 @@ initlist_add_objects(Obj_Entry *obj, Obj_Entry **tail, Objlist *list)
     /* Recursively process the needed objects. */
     if (obj->needed != NULL)
        initlist_add_neededs(obj->needed, list);
+    if (obj->needed_filtees != NULL)
+       initlist_add_neededs(obj->needed_filtees, list);
+    if (obj->needed_aux_filtees != NULL)
+       initlist_add_neededs(obj->needed_aux_filtees, list);
 
     /* Add the object to the init list. */
     if (obj->preinit_array != (Elf_Addr)NULL || obj->init != (Elf_Addr)NULL ||
@@ -2408,13 +2416,17 @@ objlist_remove(Objlist *list, Obj_Entry *obj)
  */
 static int
 relocate_objects(Obj_Entry *first, bool bind_now, Obj_Entry *rtldobj,
-    RtldLockState *lockstate)
+    int flags, RtldLockState *lockstate)
 {
     Obj_Entry *obj;
 
     for (obj = first;  obj != NULL;  obj = obj->next) {
+       if (obj->relocated)
+           continue;
+       obj->relocated = true;
        if (obj != rtldobj)
            dbg("relocating \"%s\"", obj->path);
+
        if (obj->symtab == NULL || obj->strtab == NULL ||
          !(obj->valid_hash_sysv || obj->valid_hash_gnu)) {
            _rtld_error("%s: Shared object has no run-time symbol table",
@@ -2433,7 +2445,7 @@ relocate_objects(Obj_Entry *first, bool bind_now, Obj_Entry *rtldobj,
        }
 
        /* Process the non-PLT relocations. */
-       if (reloc_non_plt(obj, rtldobj, lockstate))
+       if (reloc_non_plt(obj, rtldobj, flags, lockstate))
                return -1;
 
        /*
@@ -2468,7 +2480,7 @@ relocate_objects(Obj_Entry *first, bool bind_now, Obj_Entry *rtldobj,
            return -1;
        /* Relocate the jump slots if we are doing immediate binding. */
        if (obj->bind_now || bind_now)
-           if (reloc_jmpslots(obj, lockstate) == -1)
+           if (reloc_jmpslots(obj, flags, lockstate) == -1)
                return -1;
 
        /*
@@ -2507,35 +2519,39 @@ relocate_objects(Obj_Entry *first, bool bind_now, Obj_Entry *rtldobj,
  * consistent with how GNU does it.
  */
 static int
-resolve_object_ifunc(Obj_Entry *obj, bool bind_now, RtldLockState *lockstate)
+resolve_object_ifunc(Obj_Entry *obj, bool bind_now, int flags,
+    RtldLockState *lockstate)
 {
        if (obj->irelative && reloc_iresolve(obj, lockstate) == -1)
                return (-1);
        if ((obj->bind_now || bind_now) && obj->gnu_ifunc &&
-           reloc_gnu_ifunc(obj, lockstate) == -1)
+           reloc_gnu_ifunc(obj, flags, lockstate) == -1)
                return (-1);
        return (0);
 }
 
 static int
-resolve_objects_ifunc(Obj_Entry *first, bool bind_now, RtldLockState *lockstate)
+resolve_objects_ifunc(Obj_Entry *first, bool bind_now, int flags,
+    RtldLockState *lockstate)
 {
        Obj_Entry *obj;
 
        for (obj = first;  obj != NULL;  obj = obj->next) {
-               if (resolve_object_ifunc(obj, bind_now, lockstate) == -1)
+               if (resolve_object_ifunc(obj, bind_now, flags, lockstate) == -1)
                        return (-1);
        }
        return (0);
 }
 
 static int
-initlist_objects_ifunc(Objlist *list, bool bind_now, RtldLockState *lockstate)
+initlist_objects_ifunc(Objlist *list, bool bind_now, int flags,
+    RtldLockState *lockstate)
 {
        Objlist_Entry *elm;
 
        STAILQ_FOREACH(elm, list, link) {
-               if (resolve_object_ifunc(elm->obj, bind_now, lockstate) == -1)
+               if (resolve_object_ifunc(elm->obj, bind_now, flags,
+                   lockstate) == -1)
                        return (-1);
        }
        return (0);
@@ -2751,17 +2767,30 @@ dlopen_object(const char *name, Obj_Entry *refobj, int lo_flags, int mode)
            objlist_push_tail(&list_global, obj);
        if (*old_obj_tail != NULL) {            /* We loaded something new. */
            assert(*old_obj_tail == obj);
-           result = load_needed_objects(obj, lo_flags & RTLD_LO_DLOPEN);
+           result = load_needed_objects(obj,
+               lo_flags & (RTLD_LO_DLOPEN | RTLD_LO_EARLY));
            init_dag(obj);
            ref_dag(obj);
            if (result != -1)
                result = rtld_verify_versions(&obj->dagmembers);
            if (result != -1 && ld_tracing)
                goto trace;
-           if (result == -1 || (relocate_objects(obj, (mode & RTLD_MODEMASK)
-             == RTLD_NOW, &obj_rtld, &lockstate)) == -1) {
+           if (result == -1 || (relocate_objects(obj,
+            (mode & RTLD_MODEMASK) == RTLD_NOW, &obj_rtld,
+             (lo_flags & RTLD_LO_EARLY) ? SYMLOOK_EARLY : 0,
+             &lockstate)) == -1) {
                dlopen_cleanup(obj);
                obj = NULL;
+           } else if (lo_flags & RTLD_LO_EARLY) {
+               /*
+                * Do not call the init functions for early loaded
+                * filtees.  The image is still not initialized enough
+                * for them to work.
+                *
+                * Our object is found by the global object list and
+                * will be ordered among all init calls done right
+                * before transferring control to main.
+                */
            } else {
                /* Make list of init functions to call. */
                initlist_add_objects(obj, &obj->next, &initlist);
@@ -2795,6 +2824,7 @@ dlopen_object(const char *name, Obj_Entry *refobj, int lo_flags, int mode)
     map_stacks_exec(&lockstate);
 
     if (initlist_objects_ifunc(&initlist, (mode & RTLD_MODEMASK) == RTLD_NOW,
+      (lo_flags & RTLD_LO_EARLY) ? SYMLOOK_EARLY : 0,
       &lockstate) == -1) {
        objlist_clear(&initlist);
        dlopen_cleanup(obj);
@@ -2802,8 +2832,10 @@ dlopen_object(const char *name, Obj_Entry *refobj, int lo_flags, int mode)
        return (NULL);
     }
 
-    /* Call the init functions. */
-    objlist_call_init(&initlist, &lockstate);
+    if (!(lo_flags & RTLD_LO_EARLY)) {
+       /* Call the init functions. */
+       objlist_call_init(&initlist, &lockstate);
+    }
     objlist_clear(&initlist);
     lock_release(rtld_bind_lock, &lockstate);
     return obj;
@@ -3624,7 +3656,7 @@ symlook_obj(SymLook *req, const Obj_Entry *obj)
 {
     DoneList donelist;
     SymLook req1;
-    int res, mres;
+    int flags, res, mres;
 
     /*
      * There is at least one valid hash at this point, and we prefer to use
@@ -3637,7 +3669,8 @@ symlook_obj(SymLook *req, const Obj_Entry *obj)
 
     if (mres == 0) {
        if (obj->needed_filtees != NULL) {
-           load_filtees(__DECONST(Obj_Entry *, obj), 0, req->lockstate);
+           flags = (req->flags & SYMLOOK_EARLY) ? RTLD_LO_EARLY : 0;
+           load_filtees(__DECONST(Obj_Entry *, obj), flags, req->lockstate);
            donelist_init(&donelist);
            symlook_init_from_req(&req1, req);
            res = symlook_needed(&req1, obj->needed_filtees, &donelist);
@@ -3648,7 +3681,8 @@ symlook_obj(SymLook *req, const Obj_Entry *obj)
            return (res);
        }
        if (obj->needed_aux_filtees != NULL) {
-           load_filtees(__DECONST(Obj_Entry *, obj), 0, req->lockstate);
+           flags = (req->flags & SYMLOOK_EARLY) ? RTLD_LO_EARLY : 0;
+           load_filtees(__DECONST(Obj_Entry *, obj), flags, req->lockstate);
            donelist_init(&donelist);
            symlook_init_from_req(&req1, req);
            res = symlook_needed(&req1, obj->needed_aux_filtees, &donelist);
index 6f8f735..c9be7b3 100644 (file)
@@ -227,6 +227,7 @@ typedef struct Struct_Obj_Entry {
 
     bool mainprog : 1;         /* True if this is the main program */
     bool rtld : 1;             /* True if this is the dynamic linker */
+    bool relocated : 1;                /* True if processed by relocate_objects() */
     bool textrel : 1;          /* True if there are relocations to text seg */
     bool symbolic : 1;         /* True if generated with "-Bsymbolic" */
     bool bind_now : 1;         /* True if all relocations should be made first */
@@ -269,6 +270,7 @@ typedef struct Struct_Obj_Entry {
 #define SYMLOOK_IN_PLT 0x01    /* Lookup for PLT symbol */
 #define SYMLOOK_DLSYM  0x02    /* Return newest versioned symbol. Used by
                                   dlsym. */
+#define        SYMLOOK_EARLY   0x04    /* Symlook is done during initialization. */
 
 /* Flags for load_object(). */
 #define        RTLD_LO_NOLOAD  0x01    /* dlopen() specified RTLD_NOLOAD. */
@@ -276,6 +278,8 @@ typedef struct Struct_Obj_Entry {
 #define        RTLD_LO_TRACE   0x04    /* Only tracing. */
 #define        RTLD_LO_NODELETE 0x08   /* Loaded object cannot be closed. */
 #define        RTLD_LO_FILTEES 0x10    /* Loading filtee. */
+#define        RTLD_LO_EARLY   0x20    /* Do not call ctors, postpone it to the
+                                  initialization during the image start. */
 
 /*
  * Symbol cache entry used during relocation to avoid multiple lookups
@@ -364,11 +368,12 @@ const Ver_Entry *fetch_ventry(const Obj_Entry *obj, unsigned long);
  * MD function declarations.
  */
 int do_copy_relocations(Obj_Entry *);
-int reloc_non_plt(Obj_Entry *, Obj_Entry *, struct Struct_RtldLockState *);
+int reloc_non_plt(Obj_Entry *, Obj_Entry *, int flags,
+    struct Struct_RtldLockState *);
 int reloc_plt(Obj_Entry *);
-int reloc_jmpslots(Obj_Entry *, struct Struct_RtldLockState *);
+int reloc_jmpslots(Obj_Entry *, int flags, struct Struct_RtldLockState *);
 int reloc_iresolve(Obj_Entry *, struct Struct_RtldLockState *);
-int reloc_gnu_ifunc(Obj_Entry *, struct Struct_RtldLockState *);
+int reloc_gnu_ifunc(Obj_Entry *, int flags, struct Struct_RtldLockState *);
 void allocate_initial_tls(Obj_Entry *);
 
 #endif /* } */
index 96544a6..ca216f6 100644 (file)
@@ -85,6 +85,7 @@ do_copy_relocations(Obj_Entry *dstobj)
            size = dstsym->st_size;
            symlook_init(&req, name);
            req.ventry = fetch_ventry(dstobj, ELF_R_SYM(rela->r_info));
+           req.flags = SYMLOOK_EARLY;
 
            for (srcobj = dstobj->next;  srcobj != NULL;  srcobj = srcobj->next) {
                res = symlook_obj(&req, srcobj);
@@ -121,7 +122,8 @@ init_pltgot(Obj_Entry *obj)
 
 /* Process the non-PLT relocations. */
 int
-reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
+reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, int flags,
+    RtldLockState *lockstate)
 {
        const Elf_Rela *relalim;
        const Elf_Rela *rela;
@@ -154,7 +156,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -173,7 +175,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -203,7 +205,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -217,7 +219,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -248,7 +250,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -280,7 +282,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -294,7 +296,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -308,7 +310,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
                    const Obj_Entry *defobj;
 
                    def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
-                     false, cache, lockstate);
+                     flags, cache, lockstate);
                    if (def == NULL)
                        goto done;
 
@@ -333,7 +335,7 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate)
 done:
        if (cache != NULL)
            free(cache);
-       return(r);
+       return (r);
 }
 
 /* Process the PLT relocations. */
@@ -369,7 +371,7 @@ reloc_plt(Obj_Entry *obj)
 
 /* Relocate the jump slots in an object. */
 int
-reloc_jmpslots(Obj_Entry *obj, RtldLockState *lockstate)
+reloc_jmpslots(Obj_Entry *obj, int flags, RtldLockState *lockstate)
 {
     const Elf_Rela *relalim;
     const Elf_Rela *rela;
@@ -385,8 +387,8 @@ reloc_jmpslots(Obj_Entry *obj, RtldLockState *lockstate)
        switch (ELF_R_TYPE(rela->r_info)) {
        case R_X86_64_JMP_SLOT:
          where = (Elf_Addr *)(obj->relocbase + rela->r_offset);
-         def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj, true, NULL,
-             lockstate);
+         def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
+               SYMLOOK_IN_PLT | flags, NULL, lockstate);
          if (def == NULL)
              return (-1);
          if (ELF_ST_TYPE(def->st_info) == STT_GNU_IFUNC) {
@@ -441,7 +443,7 @@ reloc_iresolve(Obj_Entry *obj, RtldLockState *lockstate)
 }
 
 int
-reloc_gnu_ifunc(Obj_Entry *obj, RtldLockState *lockstate)
+reloc_gnu_ifunc(Obj_Entry *obj, int flags, RtldLockState *lockstate)
 {
     const Elf_Rela *relalim;
     const Elf_Rela *rela;
@@ -457,8 +459,8 @@ reloc_gnu_ifunc(Obj_Entry *obj, RtldLockState *lockstate)
        switch (ELF_R_TYPE(rela->r_info)) {
        case R_X86_64_JMP_SLOT:
          where = (Elf_Addr *)(obj->relocbase + rela->r_offset);
-         def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj, true, NULL,
-             lockstate);
+         def = find_symdef(ELF_R_SYM(rela->r_info), obj, &defobj,
+               SYMLOOK_IN_PLT | flags, NULL, lockstate);
          if (def == NULL)
              return (-1);
          if (ELF_ST_TYPE(def->st_info) != STT_GNU_IFUNC)