diff options
author | Szabolcs Nagy <szabolcs.nagy@arm.com> | 2022-04-07 18:40:25 +0100 |
---|---|---|
committer | Szabolcs Nagy <szabolcs.nagy@arm.com> | 2022-08-05 19:45:19 +0100 |
commit | d63bc8be874f42090f3a7a51441221863b83f149 (patch) | |
tree | b861bc8a1a69be5f8f10b9fce611dd2438e4db7f | |
parent | d257d001c3f16ccc4e4692201d3c03f67a0a9b18 (diff) | |
download | glibc-d63bc8be874f42090f3a7a51441221863b83f149.tar.gz glibc-d63bc8be874f42090f3a7a51441221863b83f149.tar.xz glibc-d63bc8be874f42090f3a7a51441221863b83f149.zip |
cheri: elf: elfptr_t fixes for preinit/init/fini array
According to the ELF spec: "Each element of this array is a pointer to a function to be executed by the dynamic linker." "Note that the address of a function need not be the same as a pointer to a function as defined by the processor supplement." so these should be accessed via uintptr_t type instead of ElfW(Addr).
-rw-r--r-- | csu/libc-start.c | 4 | ||||
-rw-r--r-- | elf/dl-close.c | 6 | ||||
-rw-r--r-- | elf/dl-fini.c | 6 | ||||
-rw-r--r-- | elf/dl-init.c | 12 |
4 files changed, 14 insertions, 14 deletions
diff --git a/csu/libc-start.c b/csu/libc-start.c index 543560f36c..2726fbdbec 100644 --- a/csu/libc-start.c +++ b/csu/libc-start.c @@ -139,8 +139,8 @@ call_init (int argc, char **argv, char **env) if (init_array != NULL) { unsigned int jm - = l->l_info[DT_INIT_ARRAYSZ]->d_un.d_val / sizeof (ElfW(Addr)); - ElfW(Addr) *addrs = (void *) (init_array->d_un.d_ptr + l->l_addr); + = l->l_info[DT_INIT_ARRAYSZ]->d_un.d_val / sizeof (elfptr_t); + elfptr_t *addrs = (void *) (init_array->d_un.d_ptr + l->l_addr); for (unsigned int j = 0; j < jm; ++j) ((dl_init_t) addrs[j]) (argc, argv, env); } diff --git a/elf/dl-close.c b/elf/dl-close.c index bcd6e206e9..a20cf8cc36 100644 --- a/elf/dl-close.c +++ b/elf/dl-close.c @@ -119,11 +119,11 @@ call_destructors (void *closure) if (map->l_info[DT_FINI_ARRAY] != NULL) { - ElfW(Addr) *array = - (ElfW(Addr) *) (map->l_addr + elfptr_t *array = + (elfptr_t *) (map->l_addr + map->l_info[DT_FINI_ARRAY]->d_un.d_ptr); unsigned int sz = (map->l_info[DT_FINI_ARRAYSZ]->d_un.d_val - / sizeof (ElfW(Addr))); + / sizeof (elfptr_t)); while (sz-- > 0) ((fini_t) array[sz]) (); diff --git a/elf/dl-fini.c b/elf/dl-fini.c index 030b1fcbcd..040bda28b2 100644 --- a/elf/dl-fini.c +++ b/elf/dl-fini.c @@ -133,11 +133,11 @@ _dl_fini (void) /* First see whether an array is given. */ if (l->l_info[DT_FINI_ARRAY] != NULL) { - ElfW(Addr) *array = - (ElfW(Addr) *) (l->l_addr + elfptr_t *array = + (elfptr_t *) (l->l_addr + l->l_info[DT_FINI_ARRAY]->d_un.d_ptr); unsigned int i = (l->l_info[DT_FINI_ARRAYSZ]->d_un.d_val - / sizeof (ElfW(Addr))); + / sizeof (elfptr_t)); while (i-- > 0) ((fini_t) array[i]) (); } diff --git a/elf/dl-init.c b/elf/dl-init.c index deefeb099a..4c8aaae7e0 100644 --- a/elf/dl-init.c +++ b/elf/dl-init.c @@ -61,11 +61,11 @@ call_init (struct link_map *l, int argc, char **argv, char **env) { unsigned int j; unsigned int jm; - ElfW(Addr) *addrs; + elfptr_t *addrs; - jm = l->l_info[DT_INIT_ARRAYSZ]->d_un.d_val / sizeof (ElfW(Addr)); + jm = l->l_info[DT_INIT_ARRAYSZ]->d_un.d_val / sizeof (elfptr_t); - addrs = (ElfW(Addr) *) (init_array->d_un.d_ptr + l->l_addr); + addrs = (elfptr_t *) (init_array->d_un.d_ptr + l->l_addr); for (j = 0; j < jm; ++j) ((dl_init_t) addrs[j]) (argc, argv, env); } @@ -88,16 +88,16 @@ _dl_init (struct link_map *main_map, int argc, char **argv, char **env) /* Don't do anything if there is no preinit array. */ if (__builtin_expect (preinit_array != NULL, 0) && preinit_array_size != NULL - && (i = preinit_array_size->d_un.d_val / sizeof (ElfW(Addr))) > 0) + && (i = preinit_array_size->d_un.d_val / sizeof (elfptr_t)) > 0) { - ElfW(Addr) *addrs; + elfptr_t *addrs; unsigned int cnt; if (__glibc_unlikely (GLRO(dl_debug_mask) & DL_DEBUG_IMPCALLS)) _dl_debug_printf ("\ncalling preinit: %s\n\n", DSO_FILENAME (main_map->l_name)); - addrs = (ElfW(Addr) *) (preinit_array->d_un.d_ptr + main_map->l_addr); + addrs = (elfptr_t *) (preinit_array->d_un.d_ptr + main_map->l_addr); for (cnt = 0; cnt < i; ++cnt) ((dl_init_t) addrs[cnt]) (argc, argv, env); } |