]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
cheri: elf: elfptr_t fixes for preinit/init/fini array
authorSzabolcs Nagy <szabolcs.nagy@arm.com>
Thu, 7 Apr 2022 17:40:25 +0000 (18:40 +0100)
committerSzabolcs Nagy <szabolcs.nagy@arm.com>
Fri, 5 Aug 2022 18:45:19 +0000 (19:45 +0100)
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).

csu/libc-start.c
elf/dl-close.c
elf/dl-fini.c
elf/dl-init.c

index 543560f36c33b07a1fbe1b7e4578374fe8007b1f..2726fbdbec515bc214efa8306dc588ac4816401e 100644 (file)
@@ -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);
     }
index bcd6e206e94b1c67cbe0b884f82eed847ccd228e..a20cf8cc36f7c4ec8668e75545dde76e765fdc7f 100644 (file)
@@ -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]) ();
index 030b1fcbcdc3018d7874ca4760eacbfb5fa66760..040bda28b265c5fc9c64c32eee57864bc10547f5 100644 (file)
@@ -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]) ();
                        }
index deefeb099a856d1f590626f6c5fb5727b94e11d8..4c8aaae7e0c2d21b8154fe01f51929e9a1f5fa0c 100644 (file)
@@ -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);
     }