git: 1bce4afb06a9 - stable/13 - rtld: make token substitution table-driven
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Fri, 18 Feb 2022 02:01:29 UTC
The branch stable/13 has been updated by kib:
URL: https://cgit.FreeBSD.org/src/commit/?id=1bce4afb06a9c82a0d41dbbd5f6784cb273540e3
commit 1bce4afb06a9c82a0d41dbbd5f6784cb273540e3
Author: Konstantin Belousov <kib@FreeBSD.org>
AuthorDate: 2022-02-11 13:44:36 +0000
Commit: Konstantin Belousov <kib@FreeBSD.org>
CommitDate: 2022-02-18 02:01:24 +0000
rtld: make token substitution table-driven
(cherry picked from commit 55abf23dd36b2fa1499bd6806ce4c9510f7a4ee5)
---
libexec/rtld-elf/rtld.c | 27 ++++++++++++++++++++-------
1 file changed, 20 insertions(+), 7 deletions(-)
diff --git a/libexec/rtld-elf/rtld.c b/libexec/rtld-elf/rtld.c
index 5819bf18e4b5..c2fa85a27224 100644
--- a/libexec/rtld-elf/rtld.c
+++ b/libexec/rtld-elf/rtld.c
@@ -1179,10 +1179,22 @@ origin_subst_one(Obj_Entry *obj, char *real, const char *kw,
return (res);
}
+static const struct {
+ const char *kw;
+ bool pass_obj;
+ const char *subst;
+} tokens[] = {
+ { .kw = "$ORIGIN", .pass_obj = true, .subst = NULL },
+ { .kw = "$OSNAME", .pass_obj = false, .subst = uts.sysname },
+ { .kw = "$OSREL", .pass_obj = false, .subst = uts.release },
+ { .kw = "$PLATFORM", .pass_obj = false, .subst = uts.machine },
+};
+
static char *
origin_subst(Obj_Entry *obj, const char *real)
{
- char *res1, *res2, *res3, *res4;
+ char *res;
+ int i;
if (obj == NULL || !trust)
return (xstrdup(real));
@@ -1192,13 +1204,14 @@ origin_subst(Obj_Entry *obj, const char *real)
return (NULL);
}
}
+
/* __DECONST is safe here since without may_free real is unchanged */
- res1 = origin_subst_one(obj, __DECONST(char *, real), "$ORIGIN", NULL,
- false);
- res2 = origin_subst_one(NULL, res1, "$OSNAME", uts.sysname, true);
- res3 = origin_subst_one(NULL, res2, "$OSREL", uts.release, true);
- res4 = origin_subst_one(NULL, res3, "$PLATFORM", uts.machine, true);
- return (res4);
+ res = __DECONST(char *, real);
+ for (i = 0; i < (int)nitems(tokens); i++) {
+ res = origin_subst_one(tokens[i].pass_obj ? obj : NULL,
+ res, tokens[i].kw, tokens[i].subst, i == 0);
+ }
+ return (res);
}
void