git: c6d4339208b4 - stable/14 - rtld symlook_obj: move common code to check filtees into helper

[Konstantin Belousov]

The branch stable/14 has been updated by kib:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=c6d4339208b4cade3cfb1642943da91ae31ed6ae

commit c6d4339208b4cade3cfb1642943da91ae31ed6ae
Author:     Konstantin Belousov <k...@freebsd.org>
AuthorDate: 2024-02-13 00:48:42 +0000
Commit:     Konstantin Belousov <k...@freebsd.org>
CommitDate: 2024-02-18 10:01:46 +0000

    rtld symlook_obj: move common code to check filtees into helper
    
    (cherry picked from commit 9ea864b54b57f2d0125860fb923f8db52b20eac2)
---
 libexec/rtld-elf/rtld.c | 31 +++++++++++++++++++------------
 1 file changed, 19 insertions(+), 12 deletions(-)

diff --git a/libexec/rtld-elf/rtld.c b/libexec/rtld-elf/rtld.c
index 37c016299a66..380b2696f41c 100644
--- a/libexec/rtld-elf/rtld.c
+++ b/libexec/rtld-elf/rtld.c
@@ -4680,6 +4680,20 @@ symlook_needed(SymLook *req, const Needed_Entry *needed, 
DoneList *dlp)
     return (ESRCH);
 }
 
+static int
+symlook_obj_load_filtees(SymLook *req, SymLook *req1, const Obj_Entry *obj,
+    Needed_Entry *needed)
+{
+       DoneList donelist;
+       int flags;
+
+       flags = (req->flags & SYMLOOK_EARLY) != 0 ? RTLD_LO_EARLY : 0;
+       load_filtees(__DECONST(Obj_Entry *, obj), flags, req->lockstate);
+       donelist_init(&donelist);
+       symlook_init_from_req(req1, req);
+       return (symlook_needed(req1, needed, &donelist));
+}
+
 /*
  * Search the symbol table of a single shared object for a symbol of
  * the given name and version, if requested.  Returns a pointer to the
@@ -4692,9 +4706,8 @@ symlook_needed(SymLook *req, const Needed_Entry *needed, 
DoneList *dlp)
 int
 symlook_obj(SymLook *req, const Obj_Entry *obj)
 {
-    DoneList donelist;
     SymLook req1;
-    int flags, res, mres;
+    int res, mres;
 
     /*
      * If there is at least one valid hash at this point, we prefer to
@@ -4709,11 +4722,8 @@ symlook_obj(SymLook *req, const Obj_Entry *obj)
 
     if (mres == 0) {
        if (obj->needed_filtees != NULL) {
-           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);
+           res = symlook_obj_load_filtees(req, &req1, obj,
+               obj->needed_filtees);
            if (res == 0) {
                req->sym_out = req1.sym_out;
                req->defobj_out = req1.defobj_out;
@@ -4721,11 +4731,8 @@ symlook_obj(SymLook *req, const Obj_Entry *obj)
            return (res);
        }
        if (obj->needed_aux_filtees != NULL) {
-           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);
+           res = symlook_obj_load_filtees(req, &req1, obj,
+               obj->needed_aux_filtees);
            if (res == 0) {
                req->sym_out = req1.sym_out;
                req->defobj_out = req1.defobj_out;